On Thu, 20 Mar 2025 18:02:49 GMT, Artur Barashev <[email protected]> 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
