On Fri, 3 Apr 2026 11:57:29 GMT, Daniel Fuchs <[email protected]> wrote:
>> Some more TestNG -> JUnit conversion for java/net tests. >> This change converts tests under: >> >> test/jdk/java/net/DatagramSocketImp/ >> test/jdk/java/net/MulticastSocket/ >> test/jdk/java/net/Socket/ >> test/jdk/java/net/SocketImpl/ > > Daniel Fuchs has updated the pull request incrementally with one additional > commit since the last revision: > > Fixed two mistakes test/jdk/java/net/SocketImpl/TestDefaultBehavior.java line 28: > 26: * @bug 8224477 > 27: * @summary Basic test for java.net.SocketImpl default behavior > 28: * @run junit TestDefaultBehavior I notice you didn't change this to use $(test.main.classes}. Was this overlooked? Also, just wondering what was the reason for the change elsewhere? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30564#discussion_r3032669863
