author | alanb |
Fri, 10 Feb 2017 09:03:55 +0000 | |
changeset 43665 | 4bb003cad9b9 |
parent 42307 | cefc81dc1d52 |
child 44520 | 0553e129e0ec |
permissions | -rw-r--r-- |
36508 | 1 |
/* |
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
2 |
* Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved. |
36508 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. Oracle designates this |
|
8 |
* particular file as subject to the "Classpath" exception as provided |
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
|
10 |
* |
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
15 |
* accompanied this code). |
|
16 |
* |
|
17 |
* You should have received a copy of the GNU General Public License version |
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
20 |
* |
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
22 |
* or visit www.oracle.com if you need additional information or have any |
|
23 |
* questions. |
|
24 |
*/ |
|
25 |
||
26 |
/* |
|
27 |
* @test |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
28 |
* @summary class p1.c1 defined in m1x tries to access p2.c2 defined in unnamed module. |
40631
ed82623d7831
8157957: ClassNotFoundException: jdk.test.lib.JDKToolFinder
ctornqvi
parents:
38306
diff
changeset
|
29 |
* @library /test/lib |
38152
80e5da81fb2c
8154258: [TESTBUG] Various serviceability tests fail compilation
dsamersoff
parents:
36508
diff
changeset
|
30 |
* @modules java.base/jdk.internal.misc |
36508 | 31 |
* @modules java.base/jdk.internal.module |
32 |
* @compile myloaders/MyDiffClassLoader.java |
|
33 |
* @compile p2/c2.java |
|
34 |
* @compile p1/c1.java |
|
35 |
* @compile p1/c1ReadEdgeDiffLoader.java |
|
36 |
* @compile p1/c1Loose.java |
|
37 |
* @run main/othervm -Xbootclasspath/a:. DiffCL_Umod |
|
38 |
*/ |
|
39 |
||
40 |
import static jdk.test.lib.Asserts.*; |
|
41 |
||
42 |
import java.lang.module.Configuration; |
|
43 |
import java.lang.module.ModuleDescriptor; |
|
44 |
import java.lang.module.ModuleFinder; |
|
45 |
import java.lang.reflect.Layer; |
|
46 |
import java.lang.reflect.Module; |
|
47 |
import java.util.HashMap; |
|
48 |
import java.util.Map; |
|
49 |
import java.util.Set; |
|
50 |
import myloaders.MyDiffClassLoader; |
|
51 |
||
52 |
// |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
53 |
// ClassLoader1 --> defines m1x --> packages p1 |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
54 |
// package p1 in m1x is exported unqualifiedly |
36508 | 55 |
// |
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
56 |
// class p1.c1 defined in m1x tries to access p2.c2 defined in |
36508 | 57 |
// in unnamed module. |
58 |
// |
|
59 |
// Three access attempts occur in this test: |
|
60 |
// 1. The first access is not allowed because a strict module |
|
61 |
// cannot read an unnamed module. |
|
62 |
// 2. In this scenario a strict module establishes readability |
|
63 |
// to the particular unnamed module it is trying to access. |
|
64 |
// Access is allowed. |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
65 |
// 3. Module m1x in the test_looseModuleLayer() method |
36508 | 66 |
// is transitioned to a loose module, access |
67 |
// to all unnamed modules is allowed. |
|
68 |
// |
|
69 |
public class DiffCL_Umod { |
|
70 |
||
71 |
// Create Layers over the boot layer to test different |
|
72 |
// accessing scenarios of a named module to an unnamed module. |
|
73 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
74 |
// Module m1x is a strict module and has not established |
36508 | 75 |
// readability to an unnamed module that p2.c2 is defined in. |
76 |
public void test_strictModuleLayer() throws Throwable { |
|
77 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
78 |
// Define module: m1x |
36508 | 79 |
// Can read: java.base |
80 |
// Packages: p1 |
|
81 |
// Packages exported: p1 is exported unqualifiedly |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
82 |
ModuleDescriptor descriptor_m1x = |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
83 |
ModuleDescriptor.newModule("m1x") |
36508 | 84 |
.requires("java.base") |
85 |
.exports("p1") |
|
86 |
.build(); |
|
87 |
||
88 |
// Set up a ModuleFinder containing all modules for this layer. |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
89 |
ModuleFinder finder = ModuleLibrary.of(descriptor_m1x); |
36508 | 90 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
91 |
// Resolves "m1x" |
36508 | 92 |
Configuration cf = Layer.boot() |
93 |
.configuration() |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
94 |
.resolve(finder, ModuleFinder.of(), Set.of("m1x")); |
36508 | 95 |
|
96 |
MyDiffClassLoader.loader1 = new MyDiffClassLoader(); |
|
97 |
MyDiffClassLoader.loader2 = new MyDiffClassLoader(); |
|
98 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
99 |
// map module m1x to class loader. |
36508 | 100 |
// class c2 will be loaded in an unnamed module/loader2 |
101 |
// to achieve differing class loaders. |
|
102 |
Map<String, ClassLoader> map = new HashMap<>(); |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
103 |
map.put("m1x", MyDiffClassLoader.loader1); |
36508 | 104 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
105 |
// Create Layer that contains m1x |
36508 | 106 |
Layer layer = Layer.boot().defineModules(cf, map::get); |
107 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
108 |
assertTrue(layer.findLoader("m1x") == MyDiffClassLoader.loader1); |
36508 | 109 |
assertTrue(layer.findLoader("java.base") == null); |
110 |
||
111 |
// now use the same loader to load class p1.c1 |
|
112 |
Class p1_c1_class = MyDiffClassLoader.loader1.loadClass("p1.c1"); |
|
113 |
||
114 |
// Attempt access |
|
115 |
try { |
|
116 |
p1_c1_class.newInstance(); |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
117 |
throw new RuntimeException("Test Failed, strict module m1x should not be able " + |
36508 | 118 |
"to access public type p2.c2 defined in unnamed module"); |
119 |
} catch (IllegalAccessError e) { |
|
120 |
} |
|
121 |
} |
|
122 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
123 |
// Module m1x is a strict module and has established |
36508 | 124 |
// readability to an unnamed module that p2.c2 is defined in. |
125 |
public void test_strictModuleUnnamedReadableLayer() throws Throwable { |
|
126 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
127 |
// Define module: m1x |
36508 | 128 |
// Can read: java.base |
129 |
// Packages: p1 |
|
130 |
// Packages exported: p1 is exported unqualifiedly |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
131 |
ModuleDescriptor descriptor_m1x = |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
132 |
ModuleDescriptor.newModule("m1x") |
36508 | 133 |
.requires("java.base") |
134 |
.exports("p1") |
|
135 |
.build(); |
|
136 |
||
137 |
// Set up a ModuleFinder containing all modules for this layer. |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
138 |
ModuleFinder finder = ModuleLibrary.of(descriptor_m1x); |
36508 | 139 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
140 |
// Resolves "m1x" |
36508 | 141 |
Configuration cf = Layer.boot() |
142 |
.configuration() |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
143 |
.resolve(finder, ModuleFinder.of(), Set.of("m1x")); |
36508 | 144 |
|
145 |
MyDiffClassLoader.loader1 = new MyDiffClassLoader(); |
|
146 |
MyDiffClassLoader.loader2 = new MyDiffClassLoader(); |
|
147 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
148 |
// map module m1x to class loader. |
36508 | 149 |
// class c2 will be loaded in an unnamed module/loader2 |
150 |
// to achieve differing class loaders. |
|
151 |
Map<String, ClassLoader> map = new HashMap<>(); |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
152 |
map.put("m1x", MyDiffClassLoader.loader1); |
36508 | 153 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
154 |
// Create Layer that contains m1x |
36508 | 155 |
Layer layer = Layer.boot().defineModules(cf, map::get); |
156 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
157 |
assertTrue(layer.findLoader("m1x") == MyDiffClassLoader.loader1); |
36508 | 158 |
assertTrue(layer.findLoader("java.base") == null); |
159 |
||
160 |
// now use the same loader to load class p1.c1ReadEdgeDiffLoader |
|
161 |
Class p1_c1_class = MyDiffClassLoader.loader1.loadClass("p1.c1ReadEdgeDiffLoader"); |
|
162 |
||
163 |
try { |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
164 |
// Read edge between m1x and the unnamed module that loads p2.c2 is established in |
36508 | 165 |
// c1ReadEdgeDiffLoader's ctor before attempting access. |
166 |
p1_c1_class.newInstance(); |
|
167 |
} catch (IllegalAccessError e) { |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
168 |
throw new RuntimeException("Test Failed, module m1x has established readability to p2/c2 loader's " + |
36508 | 169 |
"unnamed module, access should be allowed: " + e.getMessage()); |
170 |
} |
|
171 |
} |
|
172 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
173 |
// Module m1x is a loose module and thus can read all unnamed modules. |
36508 | 174 |
public void test_looseModuleLayer() throws Throwable { |
175 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
176 |
// Define module: m1x |
36508 | 177 |
// Can read: java.base |
178 |
// Packages: p1 |
|
179 |
// Packages exported: p1 is exported unqualifiedly |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
180 |
ModuleDescriptor descriptor_m1x = |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
181 |
ModuleDescriptor.newModule("m1x") |
36508 | 182 |
.requires("java.base") |
183 |
.exports("p1") |
|
184 |
.build(); |
|
185 |
||
186 |
// Set up a ModuleFinder containing all modules for this layer. |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
187 |
ModuleFinder finder = ModuleLibrary.of(descriptor_m1x); |
36508 | 188 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
189 |
// Resolves "m1x" |
36508 | 190 |
Configuration cf = Layer.boot() |
191 |
.configuration() |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
192 |
.resolve(finder, ModuleFinder.of(), Set.of("m1x")); |
36508 | 193 |
|
194 |
MyDiffClassLoader.loader1 = new MyDiffClassLoader(); |
|
195 |
MyDiffClassLoader.loader2 = new MyDiffClassLoader(); |
|
196 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
197 |
// map module m1x to class loader. |
36508 | 198 |
// class c2 will be loaded in an unnamed module/loader2 |
199 |
// to achieve differing class loaders. |
|
200 |
Map<String, ClassLoader> map = new HashMap<>(); |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
201 |
map.put("m1x", MyDiffClassLoader.loader1); |
36508 | 202 |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
203 |
// Create Layer that contains m1x |
36508 | 204 |
Layer layer = Layer.boot().defineModules(cf, map::get); |
205 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
206 |
assertTrue(layer.findLoader("m1x") == MyDiffClassLoader.loader1); |
36508 | 207 |
assertTrue(layer.findLoader("java.base") == null); |
208 |
||
209 |
// now use the same loader to load class p1.c1Loose |
|
210 |
Class p1_c1_class = MyDiffClassLoader.loader1.loadClass("p1.c1Loose"); |
|
211 |
||
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
212 |
// change m1x to read all unnamed modules |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
213 |
Module m1x = layer.findModule("m1x").get(); |
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
214 |
jdk.internal.module.Modules.addReadsAllUnnamed(m1x); |
36508 | 215 |
|
216 |
try { |
|
217 |
p1_c1_class.newInstance(); |
|
218 |
} catch (IllegalAccessError e) { |
|
43665
4bb003cad9b9
8173393: Module system implementation refresh (2/2017)
alanb
parents:
42307
diff
changeset
|
219 |
throw new RuntimeException("Test Failed, loose module m1x should be able to access " + |
36508 | 220 |
"public type p2.c2 defined in unnamed module: " + e.getMessage()); |
221 |
} |
|
222 |
} |
|
223 |
||
224 |
public static void main(String args[]) throws Throwable { |
|
225 |
DiffCL_Umod test = new DiffCL_Umod(); |
|
226 |
test.test_strictModuleLayer(); // access denied |
|
227 |
test.test_strictModuleUnnamedReadableLayer(); // access allowed |
|
228 |
test.test_looseModuleLayer(); // access allowed |
|
229 |
} |
|
230 |
} |