8081347: Add @modules to jdk_core tests

Co-authored-by: Alexander Kulyakthin <alexander.kulyakhtin@oracle.com>
Co-authored-by: Alan Bateman <alan.bateman@oracle.com>
Reviewed-by: alanb, joehw, lancea

jdk/test/com/sun/corba/transport/KeepAliveSockets.java

@@ -26,6 +26,7 @@
  * @bug 8017195
  * @summary Introduce option to setKeepAlive parameter on CORBA sockets
  *
+ * @modules java.corba/com.sun.corba.se.impl.orb java.corba/com.sun.corba.se.impl.transport
  * @run main/othervm KeepAliveSockets
  * @run main/othervm -Dcom.sun.CORBA.transport.enableTcpKeepAlive KeepAliveSockets
  * @run main/othervm -Dcom.sun.CORBA.transport.enableTcpKeepAlive=true KeepAliveSockets

jdk/test/com/sun/crypto/provider/Cipher/AES/TestAESCiphers/TestAESWithProviderChange.java

@@ -27,6 +27,7 @@ import java.security.Security;
  * @bug 8044193
  * @summary Test AES ciphers with different modes and padding schemes after
  *  remove then add provider.
+ * @modules java.base/com.sun.crypto.provider
  * @run main/othervm/policy=testAES.policy TestAESWithProviderChange
  */
 

jdk/test/com/sun/crypto/provider/Cipher/KeyWrap/XMLEncKAT.java

@@ -24,9 +24,10 @@
 /*
  * @test
  * @bug 5008159 5008156
- * @run main XMLEncKAT
  * @summary Verify that the two key wrap ciphers, i.e. "DESedeWrap"
  * and "AESWrap", work as expected.
+ * @modules java.base/sun.misc
+ * @run main XMLEncKAT
  * @author Valerie Peng
  */
 import java.security.Key;

jdk/test/com/sun/crypto/provider/TLS/TestKeyMaterial.java

@@ -26,6 +26,7 @@
  * @bug 6313661
  * @summary Known-answer-test for TlsKeyMaterial generator
  * @author Andreas Sterbenz
+ * @modules java.base/sun.security.internal.spec
  */
 
 import java.io.*;

jdk/test/com/sun/crypto/provider/TLS/TestMasterSecret.java

@@ -26,6 +26,8 @@
  * @bug 6313661
  * @summary Known-answer-test for TlsMasterSecret generator
  * @author Andreas Sterbenz
+ * @modules java.base/sun.security.internal.interfaces
+ *          java.base/sun.security.internal.spec
  */
 
 import java.io.*;

jdk/test/com/sun/crypto/provider/TLS/TestPRF.java

@@ -26,6 +26,7 @@
  * @bug 6313661
  * @summary Basic known-answer-test for TlsPrf
  * @author Andreas Sterbenz
+ * @modules java.base/sun.security.internal.spec
  */
 
 import java.io.*;

jdk/test/com/sun/crypto/provider/TLS/TestPRF12.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 6313661
+ * @modules java.base/sun.security.internal.spec
  * @summary Basic known-answer-test for TlsPrf 12
  *
  * Vector obtained from the IETF TLS working group mailing list:

jdk/test/com/sun/crypto/provider/TLS/TestPremaster.java

@@ -26,6 +26,7 @@
  * @bug 6313661
  * @summary Basic tests for TlsRsaPremasterSecret generator
  * @author Andreas Sterbenz
+ * @modules java.base/sun.security.internal.spec
  */
 
 import java.security.Security;

jdk/test/com/sun/jndi/cosnaming/CNNameParser.java

@@ -25,6 +25,7 @@
  * @bug 4238914
  * @summary Tests that JNDI/COS naming parser supports the syntax
  * defined in the new INS standard.
+ * @modules java.corba/com.sun.jndi.cosnaming
  */
 
 import javax.naming.*;

jdk/test/com/sun/jndi/cosnaming/IiopUrlIPv6.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 5042453
  * @summary Ipv6 address throws Non-numeric port number error
+ * @modules java.corba/com.sun.jndi.cosnaming
  */
 
 import com.sun.jndi.cosnaming.*;

jdk/test/com/sun/jndi/dns/Parser.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 8035105
  * @summary DNS resource record parsing
+ * @modules jdk.naming.dns/com.sun.jndi.dns
  */
 
 import com.sun.jndi.dns.ResourceRecord;

jdk/test/com/sun/jndi/ldap/Base64Test.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 8035807
  * @summary Confirm that old and new Base64 encodings are compatible.
+ * @modules java.base/sun.misc
  */
 
 import java.io.*;

jdk/test/com/sun/jndi/ldap/LdapName/Case.java

@@ -26,6 +26,7 @@
  * @bug 4278094
  * @summary Ensure that setValuesCaseSensitive() does not leave name
  *      in an invalid state.
+ * @modules java.naming/com.sun.jndi.ldap
  */
 
 import com.sun.jndi.ldap.LdapName;

jdk/test/com/sun/jndi/ldap/LdapName/UnescapeTest.java

@@ -26,6 +26,7 @@
  * @bug 4892070
  * @summary java gets hung in
  *      com.sun.jndi.ldap.LdapName$TypeAndValue.unescapeValue()
+ * @modules java.naming/com.sun.jndi.ldap
  */
 
 import com.sun.jndi.ldap.LdapName;

jdk/test/com/sun/jndi/ldap/LdapURLOptionalFields.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 8074761
  * @summary RFC-2255 allows attribute, scope and filter to be empty.
+ * @modules java.naming/com.sun.jndi.ldap
  */
 
 import com.sun.jndi.ldap.LdapURL;

jdk/test/com/sun/jndi/ldap/LdapUnicodeURL.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 6961765
  * @summary Double byte characters corrupted in DN for LDAP referrals
+ * @modules java.naming/com.sun.jndi.ldap
  */
 
 import com.sun.jndi.ldap.LdapURL;

jdk/test/com/sun/jndi/rmi/registry/RegistryContext/ContextWithNullProperties.java

@@ -25,6 +25,8 @@
  * @test
  * @bug 6676075
  * @summary RegistryContext (com.sun.jndi.url.rmi.rmiURLContext) coding problem
+ * @modules jdk.naming.rmi/com.sun.jndi.rmi.registry java.rmi/sun.rmi.registry
+ *     java.rmi/sun.rmi.server java.rmi/sun.rmi.transport java.rmi/sun.rmi.transport.tcp
  * @library ../../../../../../java/rmi/testlibrary
  * @build TestLibrary
  * @run main ContextWithNullProperties

jdk/test/com/sun/jndi/rmi/registry/RegistryContext/UnbindIdempotent.java

@@ -25,7 +25,9 @@
  * @test
  * @bug 4278121
  * @summary Ensure that calling unbind() on an unbound name returns
- *      successfully.
+ *     successfully.
+ * @modules java.rmi/sun.rmi.registry java.rmi/sun.rmi.server
+ *     java.rmi/sun.rmi.transport java.rmi/sun.rmi.transport.tcp
  * @library ../../../../../../java/rmi/testlibrary
  * @build TestLibrary
  * @run main UnbindIdempotent

jdk/test/com/sun/management/VMOptionOpenDataTest.java

@@ -31,6 +31,8 @@ import sun.management.MappedMXBeanType;
  * @test
  * @bug     8042901
  * @summary Check that MappedMXBeanType.toOpenTypeData supports VMOption
+ * @modules java.management/sun.management
+ *          jdk.management/com.sun.management
  * @author  Shanliang Jiang
  */
 public class VMOptionOpenDataTest {

jdk/test/com/sun/net/ssl/SSLSecurity/ProviderTest.java

@@ -24,6 +24,7 @@
 /*
  * @test
  * @bug 4667976
+ * @modules java.base/com.sun.net.ssl
  * @compile JavaxSSLContextImpl.java ComSSLContextImpl.java
  *      JavaxTrustManagerFactoryImpl.java ComTrustManagerFactoryImpl.java
  *      JavaxKeyManagerFactoryImpl.java ComKeyManagerFactoryImpl.java

jdk/test/com/sun/net/ssl/SSLSecurity/TruncateArray.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 4665824
  * @summary JSSE - ClassCastException with 1.4
+ * @modules java.base/com.sun.net.ssl
  */
 
 import com.sun.net.ssl.*;

jdk/test/com/sun/org/apache/xml/internal/security/TruncateHMAC.java

@@ -26,6 +26,10 @@
  * @bug 6824440 6858484
  * @summary Check that Apache XMLSec APIs will not accept HMAC truncation
  *    lengths less than minimum bound
+ * @modules java.xml.crypto/com.sun.org.apache.xml.internal.security
+ *          java.xml.crypto/com.sun.org.apache.xml.internal.security.c14n
+ *          java.xml.crypto/com.sun.org.apache.xml.internal.security.signature
+ *          java.xml.crypto/com.sun.org.apache.xml.internal.security.utils
  * @compile -XDignore.symbol.file TruncateHMAC.java
  * @run main TruncateHMAC
  */

jdk/test/com/sun/org/apache/xml/internal/security/exceptions/LocaleTest.java

@@ -26,6 +26,7 @@
  * @bug 6454215
  * @summary Make sure there are no runtime errors when throwing Apache XML
  *      Security exceptions in a non-US default locale.
+ * @modules java.xml.crypto/com.sun.org.apache.xml.internal.security.exceptions
  * @compile -XDignore.symbol.file LocaleTest.java
  * @run main LocaleTest
  */

jdk/test/com/sun/org/apache/xml/internal/security/transforms/ClassLoaderTest.java

@@ -25,6 +25,8 @@
  * @test
  * @author Sean Mullan
  * @bug 6461674 8009217 7147060
+ * @modules java.xml.crypto/com.sun.org.apache.xml.internal.security.exceptions
+ *          java.xml.crypto/com.sun.org.apache.xml.internal.security.transforms
  * @compile -XDignore.symbol.file ClassLoaderTest.java MyTransform.java
  * @run main/othervm ClassLoaderTest
  * @summary Ensure Transform.register works with transform implementations

jdk/test/com/sun/org/apache/xml/internal/security/utils/UnsyncByteArrayOutputStream/BufferOverflowTest.java

@@ -26,6 +26,7 @@
  * @bug 6954275
  * @summary Check that UnsyncByteArrayOutputStream does not
  *          throw ArrayIndexOutOfBoundsException
+ * @modules java.xml.crypto/com.sun.org.apache.xml.internal.security.utils
  * @compile -XDignore.symbol.file BufferOverflowTest.java
  * @run main BufferOverflowTest
  */

jdk/test/com/sun/security/sasl/ntlm/NTLMTest.java

@@ -25,6 +25,8 @@
  * @test
  * @bug 6911951 7150092
  * @summary NTLM should be a supported Java SASL mechanism
+ * @modules java.base/sun.misc
+ *          java.security.sasl
  */
 import java.io.IOException;
 import javax.security.sasl.*;

jdk/test/com/sun/tools/attach/BasicTests.java

@@ -37,6 +37,7 @@ import jdk.testlibrary.ProcessThread;
  * @test
  * @bug 6173612 6273707 6277253 6335921 6348630 6342019 6381757
  * @summary Basic unit tests for the VM attach mechanism.
+ * @modules jdk.jartool/sun.tools.jar
  * @library /lib/testlibrary
  * @modules java.instrument
  *          java.management

jdk/test/com/sun/tools/attach/PermissionTest.java

@@ -32,6 +32,7 @@ import jdk.testlibrary.ProcessThread;
  * @test
  * @bug 6173612 6273707 6277253 6335921 6348630 6342019 6381757
  * @summary Basic unit tests for the VM attach mechanism.
+ * @modules jdk.jartool/sun.tools.jar
  * @library /lib/testlibrary
  * @modules java.management
  *          jdk.attach

jdk/test/com/sun/tools/attach/ProviderTest.java

@@ -32,6 +32,7 @@ import com.sun.tools.attach.spi.AttachProvider;
  * @test
  * @bug 6173612 6273707 6277253 6335921 6348630 6342019 6381757
  * @summary Basic unit tests for the VM attach mechanism.
+ * @modules jdk.jartool/sun.tools.jar
  * @library /lib/testlibrary
  * @modules java.management
  *          jdk.attach

jdk/test/com/sun/tools/attach/StartManagementAgent.java

@@ -39,6 +39,7 @@ import jdk.testlibrary.Utils;
 /*
  * @test
  * @summary Test for VirtualMachine.startManagementAgent and VirtualMachine.startLocalManagementAgent
+ * @modules jdk.jartool/sun.tools.jar
  * @library /lib/testlibrary
  * @modules java.management
  *          jdk.attach

jdk/test/com/sun/tools/attach/TempDirTest.java

@@ -37,6 +37,7 @@ import jdk.testlibrary.ProcessThread;
  * @test
  * @bug 8033104
  * @summary Test to make sure attach and jvmstat works correctly when java.io.tmpdir is set
+ * @modules jdk.jartool/sun.tools.jar
  * @library /lib/testlibrary
  * @modules java.management
  *          jdk.jartool/sun.tools.jar

jdk/test/java/lang/Class/getSimpleName/GetSimpleNameTest.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 8057919
  * @summary Class.getSimpleName() should work for non-JLS compliant class names
+ * @modules java.base/jdk.internal.org.objectweb.asm
  */
 import jdk.internal.org.objectweb.asm.*;
 import static jdk.internal.org.objectweb.asm.Opcodes.*;

jdk/test/java/lang/ProcessBuilder/Basic.java

@@ -29,6 +29,7 @@
  *      4947220 7018606 7034570 4244896 5049299 8003488 8054494 8058464
  *      8067796
  * @summary Basic tests for Process and Environment Variable code
+ * @modules java.base/sun.misc
  * @run main/othervm/timeout=300 Basic
  * @run main/othervm/timeout=300 -Djdk.lang.Process.launchMechanism=fork Basic
  * @author Martin Buchholz

jdk/test/java/lang/ProcessBuilder/RedirectWithLongFilename.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 8072611
  * @summary ProcessBuilder Redirect to file appending on Windows should work with long file names
+ * @modules java.base/sun.misc
  * @author Thomas Stuefe
  */
 

jdk/test/java/lang/annotation/AnnotationsInheritanceOrderRedefinitionTest.java

@@ -27,6 +27,7 @@
  * @summary Test inheritance, order and class redefinition behaviour of RUNTIME
  *          class annotations
  * @author plevart
+ * @modules java.base/sun.reflect.annotation
  */
 
 import sun.reflect.annotation.AnnotationParser;

jdk/test/java/lang/invoke/8022701/MHIllegalAccess.java

@@ -26,7 +26,7 @@
  * @test
  * @bug 8022701
  * @summary Illegal access exceptions via methodhandle invocations threw wrong error.
- *
+ * @modules java.base/jdk.internal.org.objectweb.asm
  * @compile -XDignore.symbol.file BogoLoader.java InvokeSeveralWays.java MHIllegalAccess.java MethodSupplier.java
  * @run main/othervm MHIllegalAccess
  */

jdk/test/java/lang/invoke/RevealDirectTest.java

@@ -24,6 +24,7 @@
 /*
  * @test
  * @summary verify Lookup.revealDirect on a variety of input handles
+ * @modules java.base/sun.reflect
  * @compile -XDignore.symbol.file RevealDirectTest.java
  * @run junit/othervm -ea -esa test.java.lang.invoke.RevealDirectTest
  *

jdk/test/java/lang/invoke/VMAnonymousClass.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 8046903
  * @summary VM anonymous class members can't be statically invocable
+ * @modules java.base/sun.misc java.base/jdk.internal.org.objectweb.asm
  * @run junit test.java.lang.invoke.VMAnonymousClass
  */
 package test.java.lang.invoke;

jdk/test/java/lang/invoke/accessProtectedSuper/Test.java

@@ -26,7 +26,7 @@
  * @test
  * @bug 8022718
  * @summary Runtime accessibility checking: protected class, if extended, should be accessible from another package
- *
+ * @modules java.base/jdk.internal.org.objectweb.asm
  * @compile -XDignore.symbol.file BogoLoader.java MethodInvoker.java Test.java anotherpkg/MethodSupplierOuter.java
  * @run main/othervm Test
  */

jdk/test/java/lang/invoke/lambda/LambdaAccessControlDoPrivilegedTest.java

@@ -76,7 +76,7 @@ public class LambdaAccessControlDoPrivilegedTest extends LUtils {
         doprivJava.delete();
         doprivClass.delete();
         tr = doExec(JAVA_CMD.getAbsolutePath(),
-                    "-Xbootclasspath/p:foo.jar",
+                    "-Xbootclasspath/a:foo.jar",
                     "-cp", ".", "Bar");
         tr.assertZero("testDoPrivileged fails");
         barJava.delete();

jdk/test/java/lang/invoke/lambda/LambdaAsm.java

@@ -26,6 +26,8 @@
  * @bug 8027232
  * @summary ensures that j.l.i.InvokerByteCodeGenerator and ASM visitMethodInsn
  * generate  bytecodes with correct constant pool references
+ * @modules java.base/jdk.internal.org.objectweb.asm
+ *          jdk.compiler/com.sun.tools.classfile
  * @compile -XDignore.symbol.file LambdaAsm.java LUtils.java
  * @run main/othervm LambdaAsm
  */
@@ -133,7 +135,7 @@ public class LambdaAsm {
         int mcount = 0;
         try (DirectoryStream<Path> ds = newDirectoryStream(new File(".").toPath(),
                 // filter in lambda proxy classes
-                "A$I$$Lambda$?.class")) {
+                "A$I$$Lambda$*.class")) {
             for (Path p : ds) {
                 System.out.println(p.toFile());
                 ClassFile cf = ClassFile.read(p.toFile());

jdk/test/java/lang/invoke/lambda/LambdaStackTrace.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 8025636
  * @summary Synthetic frames should be hidden in exceptions
+ * @modules java.base/jdk.internal.org.objectweb.asm
  * @compile -XDignore.symbol.file LUtils.java LambdaStackTrace.java
  * @run main LambdaStackTrace
  */

jdk/test/java/lang/invoke/lookup/SpecialStatic.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 8032400
  * @summary JSR292: invokeSpecial: InternalError attempting to lookup a method
+ * @modules java.base/jdk.internal.org.objectweb.asm
  * @compile -XDignore.symbol.file SpecialStatic.java
  * @run junit test.java.lang.invoke.lookup.SpecialStatic
  */

jdk/test/java/lang/reflect/Generics/TestBadSignatures.java

@@ -26,6 +26,7 @@
  * @bug 6832374 7052898
  * @summary Test bad signatures get a GenericSignatureFormatError thrown.
  * @author Joseph D. Darcy
+ * @modules java.base/sun.reflect.generics.parser
  */
 
 import java.lang.reflect.*;

jdk/test/java/lang/reflect/Method/invoke/TestPrivateInterfaceMethodReflect.java

@@ -26,6 +26,7 @@
  * @bug 8026213
  * @summary Reflection support for private methods in interfaces
  * @author  Robert Field
+ * @modules java.base/jdk.internal.org.objectweb.asm
  * @run main TestPrivateInterfaceMethodReflect
  */
 

jdk/test/java/net/Authenticator/B4678055.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 4678055
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main B4678055

jdk/test/java/net/Authenticator/B4722333.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 4722333
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main B4722333

jdk/test/java/net/Authenticator/B4759514.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 4759514
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main B4759514

jdk/test/java/net/Authenticator/B4921848.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 4921848
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main/othervm -Dhttp.auth.preference=basic B4921848

jdk/test/java/net/Authenticator/B4962064.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 4962064
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main/othervm B4962064

jdk/test/java/net/Authenticator/B6870935.java

@@ -24,6 +24,7 @@
 /**
  * @test
  * @bug 6870935
+ * @modules java.base/sun.net.www
  * @run main/othervm -Dhttp.nonProxyHosts="" -Dhttp.auth.digest.validateProxy=true B6870935
  */
 

jdk/test/java/net/Inet4Address/textToNumericFormat.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 4749938
  * @summary Bug in the parsing IPv4 literal addresses
+ * @modules java.base/sun.net.spi.nameservice
  * @compile -XDignore.symbol.file=true DummyNameService.java DummyNameServiceDescriptor.java
  * @run main/othervm -Dsun.net.spi.nameservice.provider.1=dummy,oracle textToNumericFormat
 */

jdk/test/java/net/ProxySelector/LoopbackAddresses.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 4924226
  * @summary PIT: Can no launch jnlp application via 127.0.0.1 address on the web server
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build ClosedChannelList TestHttpServer HttpTransaction HttpCallback
  * @compile LoopbackAddresses.java

jdk/test/java/net/ProxySelector/ProxyTest.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 4696512
  * @summary HTTP client: Improve proxy server configuration and selection
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build ClosedChannelList TestHttpServer HttpTransaction HttpCallback
  * @compile ProxyTest.java

jdk/test/java/net/Socket/HttpProxy.java

@@ -25,6 +25,7 @@
  * @test
  * @bug 6370908
  * @summary Add support for HTTP_CONNECT proxy in Socket class
+ * @modules java.base/sun.net.www
  */
 
 import java.io.IOException;

jdk/test/java/net/URL/PerConnectionProxy.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 4920526
  * @summary Needs per connection proxy support for URLs
+ * @modules java.base/sun.net.www
  * @library ../../../sun/net/www/httptest/
  * @build ClosedChannelList TestHttpServer HttpTransaction HttpCallback
  * @compile PerConnectionProxy.java

jdk/test/java/net/URL/SpecifyHandler.java

@@ -26,6 +26,7 @@
  * @bug 4135031
  * @summary Test new URL contructors that allow specification of a
  *          URLStreamHandler protocol handler.
+ * @modules java.base/sun.net.www.protocol.file
  */
 import java.net.*;
 

jdk/test/java/net/URLConnection/B5052093.java

@@ -24,6 +24,7 @@
 /*
  * @test
  * @bug 5052093
+ * @modules java.base/sun.net.www java.base/sun.net.www.protocol.file
  * @library ../../../sun/net/www/httptest/
  * @build HttpCallback TestHttpServer ClosedChannelList HttpTransaction
  * @run main B5052093

jdk/test/java/net/URLConnection/HandleContentTypeWithAttrs.java

@@ -26,6 +26,7 @@
  * @bug 4160200
  * @summary Make sure URLConnection.getContnentHandler
  *     can handle MIME types with attributes
+ * @modules java.base/sun.net.www java.base/sun.net.www.content.text
  */
 import java.net.*;
 import java.io.*;

jdk/test/java/nio/file/Files/InterruptCopy.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 4313887 6993267
  * @summary Unit test for Sun-specific ExtendedCopyOption.INTERRUPTIBLE option
+ * @modules java.base/com.sun.nio.file
  * @library ..
  */
 

jdk/test/java/nio/file/Files/SBC.java

@@ -25,6 +25,7 @@
  * @bug 4313887
  * @summary Unit test for java.nio.file.Files.newByteChannel
  * @library ..
+ * @modules java.base/com.sun.nio.file
  */
 
 import java.nio.ByteBuffer;

jdk/test/java/nio/file/WatchService/FileTreeModifier.java

@@ -25,6 +25,7 @@
  * @bug 4313887 6838333
  * @summary Sanity test for Sun-specific FILE_TREE watch event modifier
  * @library ..
+ * @modules java.base/com.sun.nio.file
  */
 
 import java.nio.file.*;

jdk/test/java/nio/file/WatchService/SensitivityModifier.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 4313887
  * @summary Sanity test for Sun-specific sensitivity level watch event modifier
+ * @modules java.base/com.sun.nio.file
  * @library ..
  * @run main/timeout=240 SensitivityModifier
  * @key randomness

jdk/test/java/nio/file/WatchService/WithSecurityManager.java

@@ -24,6 +24,7 @@
 /* @test
  * @bug 4313887
  * @summary Unit test for Watchable#register's permission checks
+ * @modules java.base/com.sun.nio.file
  * @build WithSecurityManager
  * @run main/othervm WithSecurityManager denyAll.policy - fail
  * @run main/othervm WithSecurityManager denyAll.policy tree fail

jdk/test/java/rmi/Naming/DefaultRegistryPort.java

@@ -27,6 +27,10 @@
  * @summary change in default URL port causes regression in java.rmi.Naming
  * @author Dana Burns
  * @library ../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main/othervm DefaultRegistryPort
  */

jdk/test/java/rmi/Naming/LookupIPv6.java

@@ -26,6 +26,10 @@
  *          IPv6 addresses.
  * @bug 4402708
  * @library ../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main/othervm -Djava.net.preferIPv6Addresses=true LookupIPv6
  */

jdk/test/java/rmi/Naming/LookupNameWithColon.java

@@ -27,6 +27,10 @@
  *          containing embedded ':' characters.
  *
  * @library ../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main/othervm LookupNameWithColon
  */

jdk/test/java/rmi/Naming/RmiIsNoScheme.java

@@ -29,6 +29,10 @@
  * @author Ann Wollrath
  *
  * @library ../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main/othervm RmiIsNoScheme
  */

jdk/test/java/rmi/Naming/UnderscoreHost.java

@@ -29,6 +29,10 @@
   * @author Vinod Johnson
   *
   * @library ../testlibrary
+  * @modules java.rmi/sun.rmi.registry
+  *          java.rmi/sun.rmi.server
+  *          java.rmi/sun.rmi.transport
+  *          java.rmi/sun.rmi.transport.tcp
   * @build TestLibrary UnderscoreHost_Stub
   * @run main/othervm UnderscoreHost
  */

jdk/test/java/rmi/Naming/legalRegistryNames/LegalRegistryNames.java

@@ -27,6 +27,10 @@
  * @summary Naming assumes '/' is present in relative URL; change in URL causes regression
  * @author Dana Burns
  * @library ../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary Legal LegalRegistryNames_Stub
  * @run main/othervm LegalRegistryNames
  */

jdk/test/java/rmi/activation/Activatable/checkActivateRef/CheckActivateRef.java

@@ -36,6 +36,10 @@
  *          functionality is in place
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivateMe CheckActivateRef_Stub
  * @run main/othervm/policy=security.policy/timeout=240 -Djava.rmi.server.ignoreStubClasses=true CheckActivateRef
  * @run main/othervm/policy=security.policy/timeout=240 -Djava.rmi.server.ignoreStubClasses=false CheckActivateRef

jdk/test/java/rmi/activation/Activatable/checkAnnotations/CheckAnnotations.java

@@ -28,6 +28,10 @@
  * @author Laird Dornin; code borrowed from Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID MyRMI CheckAnnotations_Stub
  * @run main/othervm/policy=security.policy/timeout=480 CheckAnnotations
  */

jdk/test/java/rmi/activation/Activatable/checkImplClassLoader/CheckImplClassLoader.java

@@ -27,6 +27,10 @@
  * @author Laird Dornin; code borrowed from Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID
  *     MyRMI ActivatableImpl ActivatableImpl ActivatableImpl_Stub
  * @run main/othervm/policy=security.policy/timeout=150 CheckImplClassLoader

jdk/test/java/rmi/activation/Activatable/checkRegisterInLog/CheckRegisterInLog.java

@@ -27,6 +27,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     ActivateMe CheckRegisterInLog_Stub
  * @run main/othervm/policy=security.policy/timeout=240 CheckRegisterInLog

jdk/test/java/rmi/activation/Activatable/createPrivateActivable/CreatePrivateActivatable.java

@@ -27,6 +27,10 @@
  * @author Laird Dornin
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivateMe
  * @run main/othervm/policy=security.policy/timeout=240 CreatePrivateActivatable
  */

jdk/test/java/rmi/activation/Activatable/downloadParameterClass/DownloadParameterClass.java

@@ -31,6 +31,10 @@
  * @author Peter Jones (much code taken from Ann Wollrath's activation tests)
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     Foo FooReceiverImpl FooReceiverImpl_Stub Bar
  * @run main/othervm/policy=security.policy/timeout=240 DownloadParameterClass

jdk/test/java/rmi/activation/Activatable/elucidateNoSuchMethod/ElucidateNoSuchMethod.java

@@ -27,6 +27,10 @@
  * @author Laird Dornin
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivateMe ElucidateNoSuchMethod_Stub
  * @run main/othervm/policy=security.policy/timeout=240 ElucidateNoSuchMethod
  */

jdk/test/java/rmi/activation/Activatable/forceLogSnapshot/ForceLogSnapshot.java

@@ -27,6 +27,10 @@
  * @author Laird Dornin
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     ActivateMe ForceLogSnapshot_Stub
  * @run main/othervm/policy=security.policy/timeout=640 ForceLogSnapshot

jdk/test/java/rmi/activation/Activatable/inactiveGroup/InactiveGroup.java

@@ -29,6 +29,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary ActivateMe InactiveGroup_Stub
  * @run main/othervm/policy=security.policy/timeout=240 InactiveGroup
  */

jdk/test/java/rmi/activation/Activatable/lookupActivationSystem/LookupActivationSystem.java

@@ -29,6 +29,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  * @run main/othervm/timeout=240 LookupActivationSystem
  */

jdk/test/java/rmi/activation/Activatable/nestedActivate/NestedActivate.java

@@ -27,6 +27,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary ActivateMe NestedActivate_Stub
  * @run main/othervm/policy=security.policy/timeout=240 NestedActivate
  */

jdk/test/java/rmi/activation/Activatable/nonExistentActivatable/NonExistentActivatable.java

@@ -28,6 +28,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     ActivateMe NonExistentActivatable_Stub
  * @run main/othervm/policy=security.policy/timeout=240 NonExistentActivatable

jdk/test/java/rmi/activation/Activatable/restartCrashedService/RestartCrashedService.java

@@ -28,6 +28,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivateMe RestartCrashedService_Stub
  * @run main/othervm/policy=security.policy/timeout=240 RestartCrashedService
  */

jdk/test/java/rmi/activation/Activatable/restartLatecomer/RestartLatecomer.java

@@ -27,6 +27,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     RestartLatecomer RestartLatecomer_Stub
  * @run main/othervm/policy=security.policy/timeout=240 RestartLatecomer

jdk/test/java/rmi/activation/Activatable/restartService/RestartService.java

@@ -27,6 +27,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary ActivateMe RestartService_Stub
  * @run main/othervm/policy=security.policy/timeout=240 RestartService
  */

jdk/test/java/rmi/activation/Activatable/shutdownGracefully/ShutdownGracefully.java

@@ -28,6 +28,10 @@
  * @author Laird Dornin; code borrowed from Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID
  *     TestSecurityManager RegisteringActivatable ShutdownGracefully_Stub
  * @run main/othervm/policy=security.policy/timeout=700 ShutdownGracefully

jdk/test/java/rmi/activation/Activatable/unregisterInactive/UnregisterInactive.java

@@ -28,6 +28,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary ActivateMe UnregisterInactive_Stub
  * @run main/othervm/policy=security.policy/timeout=240 UnregisterInactive
  */

jdk/test/java/rmi/activation/ActivateFailedException/activateFails/ActivateFails.java

@@ -31,6 +31,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     ActivateMe ActivateFails_Stub ShutdownThread
  * @run main/othervm/java.security.policy=security.policy/timeout=240 ActivateFails

jdk/test/java/rmi/activation/ActivationGroup/downloadActivationGroup/DownloadActivationGroup.java

@@ -32,6 +32,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     DownloadActivationGroup MyActivationGroupImpl DownloadActivationGroup_Stub
  * @run main/othervm/policy=security.policy/timeout=240 DownloadActivationGroup

jdk/test/java/rmi/activation/ActivationGroupDesc/checkDefaultGroupName/CheckDefaultGroupName.java

@@ -30,6 +30,10 @@
  * @author Laird Dornin
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main CheckDefaultGroupName
  */

jdk/test/java/rmi/activation/ActivationSystem/activeGroup/IdempotentActiveGroup.java

@@ -29,6 +29,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  * @run main/othervm/policy=security.policy/timeout=480 IdempotentActiveGroup
  */

jdk/test/java/rmi/activation/ActivationSystem/modifyDescriptor/ModifyDescriptor.java

@@ -29,6 +29,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     ActivateMe ModifyDescriptor_Stub
  * @run main/othervm/policy=security.policy/timeout=240 ModifyDescriptor

jdk/test/java/rmi/activation/ActivationSystem/stubClassesPermitted/StubClassesPermitted.java

@@ -27,6 +27,11 @@
  * @author Laird Dornin
  *
  * @library ../../../testlibrary
+ * @modules java.base/sun.security.provider
+ *          java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     CanCreateStubs StubClassesPermitted_Stub
  * @run main/othervm/java.security.policy=security.policy/secure=java.lang.SecurityManager/timeout=240 StubClassesPermitted

jdk/test/java/rmi/activation/ActivationSystem/unregisterGroup/UnregisterGroup.java

@@ -28,6 +28,10 @@
  * @author Ann Wollrath
  *
  * @library ../../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary ActivateMe
  * @run main/othervm/policy=security.policy UnregisterGroup
  */

jdk/test/java/rmi/activation/CommandEnvironment/SetChildEnv.java

@@ -35,6 +35,10 @@
  * @author Adrian Colley
  *
  * @library ../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  *     Eliza Retireable Doctor Doctor_Stub
  * @run main/othervm/timeout=240/policy=security.policy

jdk/test/java/rmi/activation/log/LogTest.java

@@ -29,6 +29,7 @@
  * boundaries
  * @author Ann Wollrath
  *
+ * @modules java.rmi/sun.rmi.log
  * @run main/othervm/timeout=240 LogTest
  */
 

jdk/test/java/rmi/activation/rmidViaInheritedChannel/InheritedChannelNotServerSocket.java

@@ -29,6 +29,11 @@
  * @author Peter Jones
  *
  * @library ../../testlibrary
+ * @modules java.base/sun.nio.ch
+ *          java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  * @run main/othervm/timeout=240 InheritedChannelNotServerSocket
  */

jdk/test/java/rmi/activation/rmidViaInheritedChannel/RmidViaInheritedChannel.java

@@ -27,6 +27,11 @@
  * @author Ann Wollrath
  *
  * @library ../../testlibrary
+ * @modules java.base/sun.nio.ch
+ *          java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary RMID ActivationLibrary
  * @run main/othervm/timeout=240 RmidViaInheritedChannel
  */

jdk/test/java/rmi/dgc/VMID/CheckVMID.java

@@ -30,6 +30,10 @@
  * @author Ann Wollrath
  *
  * @library ../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary
  * @run main/othervm/policy=security.policy CheckVMID
  */

jdk/test/java/rmi/dgc/dgcImplInsulation/DGCImplInsulation.java

@@ -31,6 +31,10 @@
  * @author Peter Jones
  *
  * @library ../../testlibrary
+ * @modules java.rmi/sun.rmi.registry
+ *          java.rmi/sun.rmi.server
+ *          java.rmi/sun.rmi.transport
+ *          java.rmi/sun.rmi.transport.tcp
  * @build TestLibrary DGCImplInsulation_Stub
  * @run main/othervm/policy=security.policy DGCImplInsulation
  */