RFR: 8294881: test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose003/TestDescription.java fails [v2]

[Chris Plummer]

On Thu, 27 Oct 2022 03:41:34 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> Change Thread.resume() to error reporting and add some time to complete thread.
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
> 
>   All dispose tests fixed.

test/hotspot/jtreg/vmTestbase/nsk/jdi/VirtualMachine/dispose/dispose002a.java line 130:

> 128:                              instruction = pipe.readln();
> 129:                              if (instruction.equals("check_done")) {
> 130:                                  if (Utils.isAlive(test_thread)) {

We also have `jdk.test.lib.Utils` being referenced as `Utils` in the main test. Maybe `JDIUtils` would be better.

Also, `isAlive` is a bit generic sounding. It would be nice if the name conveyed the relationship to waiting after a dispose. Maybe `isAliveAfterVMDispose`. At least then the user knows there's something special about the "is alive" check being made.

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

PR: https://git.openjdk.org/jdk/pull/10877