Review Request for 8081347: Add @modules to jdk_core tests

Mandy Chung mandy.chung at
Thu May 28 05:07:10 UTC 2015

An update to this patch:

Joe pointed out that javax/sql/testng has a file.  I now move the @modules from test/javax/sql/testng individual tests to the modules key in

$ hg diff test/javax/sql/testng
diff --git a/test/javax/sql/testng/ b/test/javax/sql/testng/
--- a/test/javax/sql/testng/
+++ b/test/javax/sql/testng/
@@ -2,3 +2,6 @@
 TestNG.dirs= .
 othervm.dirs= .
 lib.dirs = /java/sql/testng
+modules = java.sql.rowset/com.sun.rowset \
+          java.sql.rowset/com.sun.rowset.internal \
+          java.sql.rowset/com.sun.rowset.providers


> On May 27, 2015, at 4:57 PM, Mandy Chung <mandy.chung at> wrote:
> jtreg supports a new @modules to specify the module and the internal API that a test depends on.  This is to prepare when the module boundaries are enforced at runtime.  This initial patch is contributed by several of us including Alexander Kulyakhtin and Alan.
> Webrev at:
> Thanks
> Mandy

More information about the core-libs-dev mailing list