TestScaffold combines test options and jtreg vm options. Before JDK-8304834 it incorrectly passed test jvm args as part of targetApp and test passed. But now it parse then into targetVMArgs and then add VM options.
TestScaffold should prepend vm options so test options overrides jtreg vm options.
diff --git a/test/jdk/com/sun/jdi/TestScaffold.java b/test/jdk/com/sun/jdi/TestScaffold.java
index 72fa4f6079d..0560bdfa367 100644
--- a/test/jdk/com/sun/jdi/TestScaffold.java
+++ b/test/jdk/com/sun/jdi/TestScaffold.java
@@ -548,7 +548,7 @@ abstract public class TestScaffold extends TargetAdapter {
public void connect(String args[]) {
ArgInfo argInfo = parseArgs(args);
- argInfo.targetVMArgs += VMConnection.getDebuggeeVMOptions();
+ argInfo.targetVMArgs = VMConnection.getDebuggeeVMOptions() + argInfo.targetVMArgs;
connection = new VMConnection(argInfo.connectorSpec,
argInfo.traceFlags);
Might be it also makes sense to not run this test with Xcomp. It doesn't make a sense. The test sets -Xbatch.
TestScaffold should prepend vm options so test options overrides jtreg vm options.
diff --git a/test/jdk/com/sun/jdi/TestScaffold.java b/test/jdk/com/sun/jdi/TestScaffold.java
index 72fa4f6079d..0560bdfa367 100644
--- a/test/jdk/com/sun/jdi/TestScaffold.java
+++ b/test/jdk/com/sun/jdi/TestScaffold.java
@@ -548,7 +548,7 @@ abstract public class TestScaffold extends TargetAdapter {
public void connect(String args[]) {
ArgInfo argInfo = parseArgs(args);
- argInfo.targetVMArgs += VMConnection.getDebuggeeVMOptions();
+ argInfo.targetVMArgs = VMConnection.getDebuggeeVMOptions() + argInfo.targetVMArgs;
connection = new VMConnection(argInfo.connectorSpec,
argInfo.traceFlags);
Might be it also makes sense to not run this test with Xcomp. It doesn't make a sense. The test sets -Xbatch.