Integrated: 8307778: com/sun/jdi/cds tests fail with jtreg's Virtual test thread factory

[Jaikiran Pai]

On Wed, 15 May 2024 05:59:56 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change which removes the `test/jdk/com/sun/jdi/cds/` tests from being problem listed when jtreg launches these tests through a virtual thread?
> 
> These tests aren't actually incompatible with virtual threads. The real issue is that in https://bugs.openjdk.org/browse/JDK-8305608, a test infrastructure class `test/jdk/com/sun/jdi/VMConnection.java` was changed to use the `test.class.path` system property instead of the previously used `test.classes`.
> 
> That change missed out updating the `test/jdk/com/sun/jdi/cds/` tests to pass along the classpath through the `test.class.path` system property. As a result these tests still use the old `test.classes` system property to pass around the test classpath and thus causes these tests to fail. The reason why this only impacts virtual threads is noted by Chris in this JBS comment https://bugs.openjdk.org/browse/JDK-8305608?focusedId=14572100&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14572100.
> 
> The commit in this PR updates the `CDSJDITest` to pass along `test.class.path` instead of `test.classes`. The `CDSJDITest` is only used in the tests under `test/jdk/com/sun/jdi/cds/`, so nothing else should be impacted by this change.
> 
> I ran these changes against both launching with platform thread as well as virtual thread and these tests now pass in both these cases.

This pull request has now been integrated.

Changeset: fe8a2aff
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/fe8a2aff3129b515c2a0f3ab96f5e3ad6cef7b70
Stats:     5 lines in 2 files changed: 0 ins; 3 del; 2 mod

8307778: com/sun/jdi/cds tests fail with jtreg's Virtual test thread factory

Reviewed-by: dholmes, sspitsyn, cjplummer, lmesnik

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

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