Review Request for 8081347: Add @modules to jdk_core tests

[Lance Andersen]

Hi Mandy,

The webrev and the diff below seem fine.

Best
Lance
On May 28, 2015, at 1:07 AM, Mandy Chung <mandy.chung at oracle.com> wrote:

> An update to this patch:
> 
> Joe pointed out that javax/sql/testng has a TEST.properties file.  I now move the @modules from test/javax/sql/testng individual tests to the modules key in TEST.properties.
> 
> $ hg diff test/javax/sql/testng
> diff --git a/test/javax/sql/testng/TEST.properties b/test/javax/sql/testng/TEST.properties
> --- a/test/javax/sql/testng/TEST.properties
> +++ b/test/javax/sql/testng/TEST.properties
> @@ -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
> 
> Mandy
> 
>> On May 27, 2015, at 4:57 PM, Mandy Chung <mandy.chung at oracle.com> 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:
>>  http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8081347/webrev.00/
>> 
>> Thanks
>> Mandy
> 



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