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