8028141: test/sun/management/jmxremote/bootstrap/LocalManagementTest|CustomLauncherTest.java failing again
authorrriggs
Tue, 19 Nov 2013 13:20:07 -0500
changeset 21822 12f13ba496cd
parent 21821 2fb0dc66167a
child 21823 2100a5feec29
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
jdk/test/sun/management/jmxremote/bootstrap/CustomLauncherTest.java
jdk/test/sun/management/jmxremote/bootstrap/LocalManagementTest.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<String> port = new AtomicReference<>();
             final AtomicReference<String> pid = new AtomicReference<>();
@@ -169,7 +170,7 @@
 
             ProcessBuilder client = ProcessTools.createJavaProcessBuilder(
                 "-cp",
-                TEST_CLASSES +
+                TEST_CLASSPATH +
                     File.pathSeparator +
                     TEST_JDK +
                     File.separator +
--- 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<String> 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 +