Hi all, I have migrated these classes to JUnit: - `jdk/net/ExtendedSocketOption/AsynchronousSocketChannelNAPITest.java` - `jdk/net/ExtendedSocketOption/DatagramChannelNAPITest.java` - `jdk/net/ExtendedSocketOption/DatagramSocketNAPITest.java` - `jdk/net/ExtendedSocketOption/SocketChannelNAPITest.java` - `jdk/net/ExtendedSocketOption/SocketNAPITest.java`
The former `@BeforeTest` has been converted to a static `@BeforeAll`, as this better reflects the intended scope. Additionally, I noticed an issue in the `testSocket` method where `initialRun` remained `true` for all tests (and conversely remained `false` in `SocketNAPITest`). I have corrected this so that the intended consistency check is properly exercised. Please let me know if this change introduces any unintended side effects. Thanks. --------- - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). ------------- Commit messages: - 8381862: Refactor NAPI tests to use JUnit instead of TestNG Changes: https://git.openjdk.org/jdk/pull/30964/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=30964&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8381862 Stats: 139 lines in 5 files changed: 25 ins; 48 del; 66 mod Patch: https://git.openjdk.org/jdk/pull/30964.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30964/head:pull/30964 PR: https://git.openjdk.org/jdk/pull/30964
