RFR: 8244976: vmTestbase/nsk/jdi/Event/request/request001.java doesn' initialize eName

[Serguei Spitsyn]

On Sat, 23 Jul 2022 19:41:07 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> Fixed tests that use eName to print event name to print the class name. Plus some minor refactoring.

I've added a couple of nits.
Otherwise, it looks good.
Thanks,
Serguei

test/hotspot/jtreg/vmTestbase/nsk/jdi/Event/request/request001.java line 553:

> 551:                     log3("ERROR: else clause in detecting type of event1");
> 552:                     testExitCode = FAILED;
> 553:                     throw new JDITestRuntimeException("** unexpected event **");

It is better to print the event type (or, at least, something) in this case.

test/hotspot/jtreg/vmTestbase/nsk/jdi/EventIterator/nextEvent/nextevent001.java line 543:

> 541:                     log3("ERROR: else clause in detecting type of event1");
> 542:                     testExitCode = FAILED;
> 543:                     throw new JDITestRuntimeException("** unexpected event **");

The same as commented above.

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

Marked as reviewed by sspitsyn (Reviewer).

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