> May I get a review for this change that converts `java/net/DatagramSocket` 
> TestNG tests to use JUnit.
> 
> Some of these tests now take advantage from the fact that Closeable arguments 
> passed to a ParameterizedTest are automatically closed by JUnit.

Daniel Fuchs has updated the pull request incrementally with one additional 
commit since the last revision:

  Apply suggestion from @Marcono1234
  
  Co-authored-by: Marcono1234 <[email protected]>

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/30502/files
  - new: https://git.openjdk.org/jdk/pull/30502/files/151124b4..a6b17125

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=30502&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30502&range=00-01

  Stats: 6 lines in 1 file changed: 0 ins; 4 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/30502.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/30502/head:pull/30502

PR: https://git.openjdk.org/jdk/pull/30502

Reply via email to