On Fri, 9 Aug 2024 02:25:33 GMT, Leonid Mesnik <[email protected]> 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
