# HG changeset patch # User rriggs # Date 1384885207 18000 # Node ID 12f13ba496cd2fe47209ab2868b1a6a07723af8a # Parent 2fb0dc66167ab85967515578257f007345ce19ec 8028141: test/sun/management/jmxremote/bootstrap/LocalManagementTest|CustomLauncherTest.java failing again Summary: Correct to use the test.class.path instead of test.classes Reviewed-by: alanb, chegar diff -r 2fb0dc66167a -r 12f13ba496cd jdk/test/sun/management/jmxremote/bootstrap/CustomLauncherTest.java --- a/jdk/test/sun/management/jmxremote/bootstrap/CustomLauncherTest.java Tue Nov 19 10:19:31 2013 -0800 +++ b/jdk/test/sun/management/jmxremote/bootstrap/CustomLauncherTest.java Tue Nov 19 13:20:07 2013 -0500 @@ -39,12 +39,13 @@ * @test * @bug 6434402 8004926 * @library /lib/testlibrary + * @build jdk.testlibrary.ProcessTools * @build TestManager TestApplication CustomLauncherTest * @run main/othervm CustomLauncherTest * @author Jaroslav Bachorik */ public class CustomLauncherTest { - private static final String TEST_CLASSES = System.getProperty("test.classes"); + private static final String TEST_CLASSPATH = System.getProperty("test.class.path"); private static final String TEST_JDK = System.getProperty("test.jdk"); private static final String TEST_SRC = System.getProperty("test.src"); @@ -82,7 +83,7 @@ } public static void main(String[] args) throws Exception { - if (TEST_CLASSES == null || TEST_CLASSES.isEmpty()) { + if (TEST_CLASSPATH == null || TEST_CLASSPATH.isEmpty()) { System.out.println("Test is designed to be run from jtreg only"); return; } @@ -139,8 +140,8 @@ System.out.println("========================="); System.out.println(" launcher : " + LAUNCHER); System.out.println(" libjvm : " + libjvmPath.toString()); - System.out.println(" classpath : " + TEST_CLASSES); - ProcessBuilder server = new ProcessBuilder(LAUNCHER, libjvmPath.toString(), TEST_CLASSES, "TestApplication"); + System.out.println(" classpath : " + TEST_CLASSPATH); + ProcessBuilder server = new ProcessBuilder(LAUNCHER, libjvmPath.toString(), TEST_CLASSPATH, "TestApplication"); final AtomicReference port = new AtomicReference<>(); final AtomicReference pid = new AtomicReference<>(); @@ -169,7 +170,7 @@ ProcessBuilder client = ProcessTools.createJavaProcessBuilder( "-cp", - TEST_CLASSES + + TEST_CLASSPATH + File.pathSeparator + TEST_JDK + File.separator + diff -r 2fb0dc66167a -r 12f13ba496cd jdk/test/sun/management/jmxremote/bootstrap/LocalManagementTest.java --- a/jdk/test/sun/management/jmxremote/bootstrap/LocalManagementTest.java Tue Nov 19 10:19:31 2013 -0800 +++ b/jdk/test/sun/management/jmxremote/bootstrap/LocalManagementTest.java Tue Nov 19 13:20:07 2013 -0500 @@ -41,6 +41,7 @@ * without connection or username/password details. * TestManager will attempt a connection to the address obtained from * both agent properties and jvmstat buffer. + * @build jdk.testlibrary.ProcessTools * @build TestManager TestApplication * @run main/othervm/timeout=300 LocalManagementTest */ @@ -48,7 +49,7 @@ import jdk.testlibrary.ProcessTools; public class LocalManagementTest { - private static final String TEST_CLASSES = System.getProperty("test.classes"); + private static final String TEST_CLASSPATH = System.getProperty("test.class.path"); private static final String TEST_JDK = System.getProperty("test.jdk"); public static void main(String[] args) throws Exception { @@ -109,7 +110,7 @@ "com.sun.management.jmxremote.authenticate=false," + "com.sun.management.jmxremote.ssl=false", "-cp", - TEST_CLASSES, + TEST_CLASSPATH, "TestApplication", "-exit" ); @@ -163,7 +164,7 @@ private static boolean doTest(String testId, String arg) throws Exception { List args = new ArrayList<>(); args.add("-cp"); - args.add(TEST_CLASSES); + args.add(TEST_CLASSPATH); if (arg != null) { args.add(arg); @@ -202,7 +203,7 @@ ProcessBuilder client = ProcessTools.createJavaProcessBuilder( "-cp", - TEST_CLASSES + + TEST_CLASSPATH + File.pathSeparator + TEST_JDK + File.separator +