> 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 with a new target base due to a merge or a rebase. The pull request now contains seven commits: - Merge - Merge - Merge branch 'master' into junit-ds-8381359 - Review feedback - Apply suggestions from code review Co-authored-by: Marcono1234 <[email protected]> - Apply suggestion from @Marcono1234 Co-authored-by: Marcono1234 <[email protected]> - 8381359: Refactor java/net/DatagramSocket TestNG tests to use JUnit ------------- Changes: https://git.openjdk.org/jdk/pull/30502/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=30502&range=06 Stats: 348 lines in 10 files changed: 82 ins; 86 del; 180 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
