test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JarBuilder.java
branchJDK-8200758-branch
changeset 58648 3bf53ffa9ae7
parent 58416 f09bf58c1f17
child 58696 61c44899b4eb
--- a/test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JarBuilder.java	Wed Oct 16 09:57:23 2019 -0400
+++ b/test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JarBuilder.java	Wed Oct 16 10:32:08 2019 -0400
@@ -62,15 +62,16 @@
 
     public void create() {
         TKit.withTempDirectory("jar-workdir", workDir -> {
-            Executor.Result javacReply = new Executor()
-                    .setExecutable(JavaTool.JAVAC)
-                    .addArguments("-d", workDir.toString())
-                    .addPathArguments(sourceFiles).execute();
-            javacReply.assertExitCodeIsZero();
+            if (!sourceFiles.isEmpty()) {
+                new Executor()
+                        .setToolProvider(JavaTool.JAVAC)
+                        .addArguments("-d", workDir.toString())
+                        .addPathArguments(sourceFiles)
+                        .execute().assertExitCodeIsZero();
+            }
             Path tmpJar = workDir.resolve("foo.jar");
             Executor jarExe = new Executor();
-            jarExe.setExecutable(JavaTool.JAR).addArguments("-c", "-v", "-f",
-                    tmpJar.toString());
+            jarExe.setToolProvider(JavaTool.JAR).addArguments("-c", "-f", tmpJar.toString());
             if (moduleVersion != null) {
                 jarExe.addArguments(String.format("--module-version=%s",
                         moduleVersion));
@@ -79,8 +80,7 @@
                 jarExe.addArguments("-e", mainClass);
             }
             jarExe.addArguments("-C", workDir.toString(), ".");
-            javacReply = jarExe.execute();
-            javacReply.assertExitCodeIsZero();
+            jarExe.execute().assertExitCodeIsZero();
             outputJar.getParentFile().mkdirs();
             Files.copy(tmpJar, outputJar.toPath(), REPLACE_EXISTING);
         });