RFR 8212045: Add back tests removed from HashesTest.java and AddExportsTest.java
Lance Andersen
lance.andersen at oracle.com
Thu Oct 11 11:55:26 UTC 2018
Hi all,,
Catching up on this issue from my todo list which adds a test back to HashesTest.java and AddExportsTest.java which were removed as part of the Java EE module removal(The original changes to remove the tests can be found at: http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/ <http://cr.openjdk.java.net/~lancea/8190378/open_changes/tests_webrev/>)
The webrev can be found for the change at http://cr.openjdk.java.net/~lancea/8212045/webrev.00/index.html <http://cr.openjdk.java.net/~lancea/8212045/webrev.00/index.html>
To help make the review easier, here is a diff of the tests comparing the new changes to the tests prior to the Java EE module removal:
————
HashesTest.java diff
——————
$ hg diff -r 47216 test/jdk/tools/jmod/hashes/HashesTest.java
diff -r 71c04702a3d5 test/jdk/tools/jmod/hashes/HashesTest.java
--- a/test/jdk/tools/jmod/hashes/HashesTest.java Tue Sep 12 19:03:39 2017 +0200
+++ b/test/jdk/tools/jmod/hashes/HashesTest.java Thu Jun 14 12:34:52 2018 -0400
@@ -1,5 +1,5 @@
/**
- * Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2015, 2018, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -247,19 +247,17 @@
Path dest = Paths.get("test4");
HashesTest ht = new HashesTest(dest);
ht.makeModule("m1");
- ht.makeModule("java.xml.bind", "m1");
- ht.makeModule("java.xml.ws", "java.xml.bind");
- ht.makeModule("m2", "java.xml.ws");
+ ht.makeModule("java.compiler", "m1");
+ ht.makeModule("m2", "java.compiler");
ht.makeJmod("m1");
ht.makeJmod("m2");
- ht.makeJmod("java.xml.ws");
- ht.makeJmod("java.xml.bind",
+ ht.makeJmod("java.compiler",
"--module-path",
ht.lib.toString() + File.pathSeparator + mpath,
- "--hash-modules", "^java.xml.*|^m.*");
+ "--hash-modules", "java\\.(?!se)|^m.*");
- ht.checkHashes("java.xml.bind", "java.xml.ws", "m2");
+ ht.checkHashes("java.compiler", "m2");
}
@Test
—————————
AddExports.java Diff
———————
$ hg diff -r 47216 test/jdk/tools/launcher/modules/addexports/AddExportsTest.java
diff -r 71c04702a3d5 test/jdk/tools/launcher/modules/addexports/AddExportsTest.java
--- a/test/jdk/tools/launcher/modules/addexports/AddExportsTest.java Tue Sep 12 19:03:39 2017 +0200
+++ b/test/jdk/tools/launcher/modules/addexports/AddExportsTest.java Fri Jun 15 15:17:56 2018 -0400
@@ -24,7 +24,7 @@
/**
* @test
* @library /lib/testlibrary /test/lib
- * @modules java.transaction
+ * @modules java.compiler
* jdk.compiler
* @build AddExportsTest jdk.test.lib.compiler.CompilerUtils jdk.testlibrary.*
* @run testng AddExportsTest
@@ -58,7 +58,7 @@
private static final String TEST1_MODULE = "m1";
private static final String TEST1_MAIN_CLASS = "jdk.test1.Main";
- // test module m2 uses java.transaction internals
+ // test module m2 uses java.compiler internals
private static final String TEST2_MODULE = "m2";
private static final String TEST2_MAIN_CLASS = "jdk.test2.Main";
@@ -78,18 +78,18 @@
"--add-exports", "java.base/jdk.internal.misc=m1");
assertTrue(compiled, "module " + TEST1_MODULE + " did not compile");
- // javac -d upgrademods/java.transaction src/java.transaction/**
+ // javac -d upgrademods/java.compiler src/java.compiler/**
compiled = CompilerUtils.compile(
- SRC_DIR.resolve("java.transaction"),
- UPGRADE_MODS_DIRS.resolve("java.transaction"));
- assertTrue(compiled, "module java.transaction did not compile");
+ SRC_DIR.resolve("java.compiler"),
+ UPGRADE_MODS_DIRS.resolve("java.compiler"));
+ assertTrue(compiled, "module java.compiler did not compile");
// javac --upgrade-module-path upgrademods -d mods/m2 src/m2/**
compiled = CompilerUtils.compile(
SRC_DIR.resolve(TEST2_MODULE),
MODS_DIR.resolve(TEST2_MODULE),
"--upgrade-module-path", UPGRADE_MODS_DIRS.toString(),
- "--add-exports", "java.transaction/javax.transaction.internal=m2");
+ "--add-exports", "java.compiler/javax.tools.internal=m2");
assertTrue(compiled, "module " + TEST2_MODULE + " did not compile");
// javac -d mods/m3 src/m3/**
@@ -168,11 +168,11 @@
*/
public void testWithUpgradedModule() throws Exception {
- // java --add-exports java.transaction/javax.transaction.internal=m2
+ // java --add-exports java.compiler/javax.tools.internal=m2
// --upgrade-module-path upgrademods --module-path mods -m ...
String mid = TEST2_MODULE + "/" + TEST2_MAIN_CLASS;
int exitValue = executeTestJava(
- "--add-exports", "java.transaction/javax.transaction.internal=m2",
+ "--add-exports", "java.compiler/javax.tools.internal=m2",
"--upgrade-module-path", UPGRADE_MODS_DIRS.toString(),
"--module-path", MODS_DIR.toString(),
"-m", mid)
ljanders-mac:open ljanders$
--------------------
The mach 5 jdk tier1, tier2, and tier3 tests ran without error
Best
Lance
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list