RFR: 8330535: Update nsk/jdb tests to use driver instead of othervm

Chris Plummer cjplummer at openjdk.org
Fri Aug 9 15:25:33 UTC 2024


On Fri, 9 Aug 2024 02:25:33 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> The nsk/jdb tests updated to use driver instead of othervm. The nsk/share/jdb code updated to set correct classpath and  throw Exceptions instead of exit.
> 
> Testing by running tests with different flags and in different CI modes.
> 
> The 'shouldPass()' might be changed to throw SkippedException but I am going to do this separately.

test/hotspot/jtreg/vmTestbase/nsk/share/jdb/JdbTest.java line 237:

> 235:             t.printStackTrace(out);
> 236:             out.println("TEST FAILED");
> 237:             throw new RuntimeException("TEST FAILED with " + t);

Why not just pass `t` as the `cause` argument?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20518#discussion_r1711668683


More information about the serviceability-dev mailing list