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

Mandy Chung mandy.chung at oracle.com
Tue May 5 20:33:27 UTC 2015



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.

Mandy


More information about the serviceability-dev mailing list