RFR: 8357282: Test vmTestbase/nsk/jvmti/AttachOnDemand/attach045/TestDescription.java times out after ClassNotFoundException
Leonid Mesnik
lmesnik at openjdk.org
Thu May 22 19:54:55 UTC 2025
On Wed, 21 May 2025 20:17:09 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
> The change fixes failure of vmTestbase/nsk/jvmti/AttachOnDemand/attach045/TestDescription.java test.
> The test was problem-listed in jdk 11 due intermittent timeouts, but in jdk15-16 changes in nsk framework made the test fail 100% (it wasn't noticed as the test was problemlisted)
> The problem is attach045 has "target" parameter which contain class name and arguments (all other tests have only class name), and value of the argument is passed in surrounding quotes, so target application fails with ClassNotFoundException.
> Now I cannot reproduce original test failure, so removing the test from problemlist
As I understand, this fix is not related to timeout. So test is going to fail. It might be makes a sense to remove it from problemlist to see how it works. However better to rename PR summary.
Changes requested by lmesnik (Reviewer).
test/hotspot/jtreg/vmTestbase/nsk/share/aod/AODRunnerArgParser.java line 109:
> 107: }
> 108:
> 109: String target = options.getProperty(TARGET_APP_PARAM);
I think it is better to remove -classPath from parameters and
use
${test.class.path}
directly in the test.
-------------
PR Review: https://git.openjdk.org/jdk/pull/25366#pullrequestreview-2862411340
PR Review: https://git.openjdk.org/jdk/pull/25366#pullrequestreview-2862413634
PR Review Comment: https://git.openjdk.org/jdk/pull/25366#discussion_r2103297797
More information about the serviceability-dev
mailing list