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
--- 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 +