The ModuleReader implementation for exploded modules maps resource names to 
file paths. A small oversight is that it doesn't handle InvalidPathException 
which is thrown when the resource name maps to something that can't be parsed 
as a file path. This has a knock on impact to Class/ClassLoader getResourceXXX 
and other usages of ModuleReader.

The existing ModuleReaderTest is updated to include additional resource names 
to test. I've also moved `@Test` annotation to the test methods so that it's a 
bit clearer which are the test methods in this source file.

-------------

Commit messages:
 - Initial commit

Changes: https://git.openjdk.org/jdk/pull/12035/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12035&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8300228
  Stats: 55 lines in 2 files changed: 29 ins; 12 del; 14 mod
  Patch: https://git.openjdk.org/jdk/pull/12035.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12035/head:pull/12035

PR: https://git.openjdk.org/jdk/pull/12035

Reply via email to