RFR(M): 8078896: Add @modules as needed to the jdk_svc tests

[Mandy Chung]

On 05/05/2015 01:33 PM, Mandy Chung wrote:
>
>
> On 05/05/2015 02:04 AM, Yekaterina Kantserova wrote:
>> Hi,
>>
>> Could I please have a review of this fix.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8078896
>> webrev: http://cr.openjdk.java.net/~ykantser/8078896
>>
>
> com.sun.management has been moved to jdk.management module.  The patch 
> for JDK-8042901 is just integrated in jdk9/dev today.  Most, if not 
> all, test/com/sun/management tests need updates to require @modules 
> jdk.management.
>
> There are several test/java/lang/management tests dependng on both 
> java.lang.management and com.sun.management.  For tests only using 
> java.lang.management API, @modules java.management is adequate.
>
> test/com/sun/jdi/InterfaceMethodsTest.java
> test/com/sun/jdi/InterruptHangTest.java
>    Since you have updated the end year of the copyright header in most 
> tests, I spot a couple and so mention it to you.
>
> Otherwise looks good.  Thanks for doing this.

About the test selection, one typical aspect of svc tests is to run a j* 
tool in a child process (e.g. jinfo, jstack, jstat, jstatd,jcmd, jps etc 
that are in jdk.jcmd module).   I would expect all test/sun/tools/jcmd 
tests should have @modules jdk.jcmd and similiar for other sun/tools/j* 
tests.  Are you thinking to come back in the next round of test 
selection update?

Mandy