8223054: [TESTBUG] Put graalJarsCP before existing classpath in GraalUnitTestLauncher

Reviewed-by: iignatyev, dlong
This commit is contained in:
Pengfei Li 2019-05-07 00:58:10 +00:00
parent e0dcd3200a
commit 58a1f6ac13

View File

@ -254,7 +254,11 @@ public class GraalUnitTestLauncher {
.collect(Collectors.joining(File.pathSeparator));
javaFlags.add("-cp");
javaFlags.add(String.join(File.pathSeparator, System.getProperty("java.class.path"), graalJarsCP));
// Existing classpath returned by System.getProperty("java.class.path") may contain another
// version of junit with which the jtreg tool is built. It may be incompatible with required
// junit version. So we put graalJarsCP before existing classpath when generating a new one
// to avoid incompatibility issues.
javaFlags.add(String.join(File.pathSeparator, graalJarsCP, System.getProperty("java.class.path")));
//
javaFlags.add("com.oracle.mxtool.junit.MxJUnitWrapper");