# HG changeset patch # User anazarov # Date 1523394973 25200 # Node ID e6c069766d4624a6c9576b545acda4acc744fb73 # Parent 7c224ec572d08f45ec45d07f74228199a7709a67 8178867: tools/jlink/multireleasejar/JLinkMultiReleaseJarTest.java failed to clean up files Reviewed-by: alanb, jjg, psandoz diff -r 7c224ec572d0 -r e6c069766d46 test/jdk/tools/jlink/multireleasejar/JLinkMultiReleaseJarTest.java --- a/test/jdk/tools/jlink/multireleasejar/JLinkMultiReleaseJarTest.java Tue Apr 10 22:37:01 2018 +0200 +++ b/test/jdk/tools/jlink/multireleasejar/JLinkMultiReleaseJarTest.java Tue Apr 10 14:16:13 2018 -0700 @@ -52,7 +52,9 @@ import java.nio.file.Path; import java.nio.file.Paths; import java.nio.file.StandardCopyOption; +import java.util.ArrayList; import java.util.Arrays; +import java.util.List; import java.util.Set; import java.util.jar.JarFile; import java.util.spi.ToolProvider; @@ -134,14 +136,15 @@ } private void javac(Path source, Path destination, String srcpath) throws IOException { - String[] args = Stream.concat( - Stream.of("-d", destination.toString(), "--module-source-path", srcpath), - Files.walk(source) - .map(Path::toString) - .filter(s -> s.endsWith(".java")) - ).toArray(String[]::new); - int rc = JAVAC_TOOL.run(System.out, System.err, args); - Assert.assertEquals(rc, 0); + var args = Stream.of("-d", destination.toString(), "--module-source-path", srcpath); + try (Stream pathStream = Files.walk(source)) { + args = Stream.concat(args, + pathStream.map(Path::toString) + .filter(s -> s.endsWith(".java"))); + + int rc = JAVAC_TOOL.run(System.out, System.err, args.toArray(String[]::new)); + Assert.assertEquals(rc, 0); + } } @Test @@ -149,7 +152,7 @@ if (ignoreTest()) return; // use jlink to build image from multi-release jar - jlink("m1.jar", "myimage"); + jlink("m1.jar", "myimage"); // validate image Path jimage = userdir.resolve("myimage").resolve("lib").resolve("modules");