On Thu, 30 Apr 2026 05:06:42 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: > > Add space @PeyaPeyaPeyang Your change (at version 288fce1c82fc1cd1249cf15872ccb5db29e673f4) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30964#issuecomment-4353648344
