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