On Tue, 28 Apr 2026 13:48:54 GMT, Daisuke Yamazaki <[email protected]> wrote:
>> 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). > > Daisuke Yamazaki has updated the pull request incrementally with one > additional commit since the last revision: > > Refactor NAPI tests to use ${test.main.class} for test execution test/jdk/jdk/net/ExtendedSocketOption/SocketNAPITest.java line 133: > 131: sID = s.getOption(SO_INCOMING_NAPI_ID); > 132: cID = c.getOption(SO_INCOMING_NAPI_ID); > 133: if(initialRun) { Missing space? (was already the case before) Suggestion: if (initialRun) { ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30964#discussion_r3164071274
