RFR: 8343791: Socket.connect API should document whether the socket will be closed when hostname resolution fails or another error occurs [v22]

Daniel Fuchs dfuchs at openjdk.org
Thu Nov 28 15:25:44 UTC 2024


On Thu, 28 Nov 2024 14:24:02 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Volkan Yazıcı has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Don't exceed 120 characters per line
>>  - Rework functional interfaces in tests
>
> test/jdk/java/net/Socket/ConnectFailTest.java line 105:
> 
>> 103:                 ConnectFailTest::createConnectedNioSocket,
>> 104:                 executable -> assertThrows(AlreadyConnectedException.class, executable));
>> 105:     }
> 
> I assume that testConnectedSocket and testConnectedNioSocket could be @ParameterizedTest so it's called with a new created Socket.

It would make them similar to the other test method but would be a bit strange since the test method would be called only once with a single parameter set.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22160#discussion_r1862376904


More information about the net-dev mailing list