On Thu, 20 Mar 2025 18:02:49 GMT, Artur Barashev <abaras...@openjdk.org> wrote:

>> Fernando Guallini has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   latch await
>
> test/jdk/sun/security/ssl/SSLSocketImpl/SetClientMode.java line 110:
> 
>> 108:                 } catch (IllegalArgumentException iae) {
>> 109:                     System.out.println("succeeded, we can't set the 
>> client mode");
>> 110:                 }
> 
> I suggest using `runAndCheckException` from /test/lib here instead of 
> try/catch construct.

Sure, but using assertThrows instead. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23898#discussion_r2007442469

Reply via email to