On Thu, 31 Oct 2024 12:11:41 GMT, Kevin Walls <kev...@openjdk.org> wrote:
> Test update: fail when it hits an Exception.
>
> This test still references jmxmp and iiop, which are of course redundant,
> there are various tests that still do this.
> I am working on http, so we may revisit these tests in future to change their
> list of protocols.
>
> For now, I'd like to simply make this test fail if any of the protocols it
> tests have failures.
> Fix a few typos while we are here.
test/jdk/javax/management/remote/mandatory/connection/DeadLockTest.java line 54:
> 52: test(protocols[i]);
> 53: } catch (Exception e) {
> 54: fail = true; // any one protocol failure, fails the test
Suggestion:
fail = true; // any one protocol failure fails the test
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21804#discussion_r1825048045
