> 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 suggestions from code review
  
  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/a6b17125..13a71a76

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

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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