RFR: 8345176: Add tests to verify java.net.Socket constructors close the socket on failure [v2]

Volkan Yazıcı duke at openjdk.org
Fri Nov 29 13:24:17 UTC 2024


On Fri, 29 Nov 2024 12:29:41 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Volkan Yazıcı has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Further simplify the test
>
> test/jdk/java/net/Socket/CtorFailTest.java line 211:
> 
>> 209:             throw new UnsupportedOperationException();
>> 210:         }
>> 211: 
> 
> IIRC Junit should print the test method arguments before running a test. I'd suggest adding a toString here, so that when a test fails we know which scenario is failing.

That is indeed a (really) nice to have. Added in cf86230d44c1bbde7e67be86692fa36721ddd28c.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22439#discussion_r1863518965


More information about the net-dev mailing list