RFR: 8343378: Exceptions in javax/management DeadLockTest.java do not cause test failure

[Chris Plummer]

On Thu, 31 Oct 2024 12:11:41 GMT, Kevin Walls <kevinw at 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