test/jdk/tools/jpackage/helpers/JPackageHelper.java
author herrick
Thu, 26 Sep 2019 10:37:37 -0400
branchJDK-8200758-branch
changeset 58360 fd45b7e2c027
parent 58359 e065fd274bc1
child 58888 d802578912f3
permissions -rw-r--r--
8231382: Use main class from main module if available Submitted-by: almatvee Reviewed-by: herrick, asemenyuk
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     1
/*
57106
ea870b9ce89a 8216492: Update copyright of all new jpackage fils to 2019
kcr
parents: 57079
diff changeset
     2
 * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     4
 *
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     7
 * published by the Free Software Foundation.
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     8
 *
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    13
 * accompanied this code).
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    14
 *
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    18
 *
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    21
 * questions.
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    22
 */
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    23
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    24
import java.io.File;
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
    25
import java.io.IOException;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    26
import java.io.PrintWriter;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    27
import java.io.StringWriter;
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
    28
import java.io.BufferedWriter;
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
    29
import java.nio.file.FileVisitResult;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    30
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    31
import java.nio.file.Files;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    32
import java.nio.file.Path;
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
    33
import java.nio.file.SimpleFileVisitor;
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
    34
import java.nio.file.attribute.BasicFileAttributes;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    35
import java.util.ArrayList;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    36
import java.util.List;
57419
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
    37
import java.util.Arrays;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    38
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    39
import java.util.spi.ToolProvider;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    40
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
    41
public class JPackageHelper {
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    42
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    43
    private static final boolean VERBOSE = false;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    44
    private static final String OS = System.getProperty("os.name").toLowerCase();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    45
    private static final String JAVA_HOME = System.getProperty("java.home");
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
    46
    public static final String TEST_SRC_ROOT;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    47
    public static final String TEST_SRC;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    48
    private static final Path BIN_DIR = Path.of(JAVA_HOME, "bin");
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
    49
    private static final Path JPACKAGE;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    50
    private static final Path JAVAC;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    51
    private static final Path JAR;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    52
    private static final Path JLINK;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    53
58360
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    54
    public static class ModuleArgs {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    55
        private final String version;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    56
        private final String mainClass;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    57
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    58
        ModuleArgs(String version, String mainClass) {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    59
            this.version = version;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    60
            this.mainClass = mainClass;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    61
        }
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    62
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    63
        public String getVersion() {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    64
            return version;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    65
        }
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    66
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    67
        public String getMainClass() {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    68
            return mainClass;
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    69
        }
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    70
    }
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
    71
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    72
    static {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    73
        if (OS.startsWith("win")) {
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
    74
            JPACKAGE = BIN_DIR.resolve("jpackage.exe");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    75
            JAVAC = BIN_DIR.resolve("javac.exe");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    76
            JAR = BIN_DIR.resolve("jar.exe");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    77
            JLINK = BIN_DIR.resolve("jlink.exe");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    78
        } else {
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
    79
            JPACKAGE = BIN_DIR.resolve("jpackage");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    80
            JAVAC = BIN_DIR.resolve("javac");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    81
            JAR = BIN_DIR.resolve("jar");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    82
            JLINK = BIN_DIR.resolve("jlink");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    83
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    84
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    85
        // Figure out test src based on where we called
57438
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    86
        TEST_SRC = System.getProperty("test.src");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    87
        Path root = Path.of(TEST_SRC);
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    88
        Path apps = Path.of(TEST_SRC, "apps");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    89
        if (apps.toFile().exists()) {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    90
            // fine - test is at root
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
    91
        } else {
57438
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    92
             apps = Path.of(TEST_SRC, "..", "apps");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    93
             if (apps.toFile().exists()) {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    94
                 root = apps.getParent().normalize(); // test is 1 level down
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    95
             } else {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    96
                 apps = Path.of(TEST_SRC, "..", "..", "apps");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    97
                 if (apps.toFile().exists()) {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    98
                     root = apps.getParent().normalize(); // 2 levels down
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
    99
                 } else {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   100
                     apps = Path.of(TEST_SRC, "..", "..", "..", "apps");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   101
                     if (apps.toFile().exists()) {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   102
                         root = apps.getParent().normalize(); // 3 levels down
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   103
                     } else {
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   104
                         // if we ever have tests more than three levels
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   105
                         // down we need to add code here
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   106
                         throw new RuntimeException("we should never get here");
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   107
                     }
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   108
                 }
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   109
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   110
        }
57438
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   111
        TEST_SRC_ROOT = root.toString();
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   112
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   113
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   114
    static final ToolProvider JPACKAGE_TOOL =
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   115
            ToolProvider.findFirst("jpackage").orElseThrow(
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   116
            () -> new RuntimeException("jpackage tool not found"));
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   117
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   118
    public static int execute(File out, String... command) throws Exception {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   119
        if (VERBOSE) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   120
            System.out.print("Execute command: ");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   121
            for (String c : command) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   122
                System.out.print(c);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   123
                System.out.print(" ");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   124
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   125
            System.out.println();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   126
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   127
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   128
        ProcessBuilder builder = new ProcessBuilder(command);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   129
        if (out != null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   130
            builder.redirectErrorStream(true);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   131
            builder.redirectOutput(out);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   132
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   133
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   134
        Process process = builder.start();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   135
        return process.waitFor();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   136
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   137
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   138
    public static Process executeNoWait(File out, String... command) throws Exception {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   139
        if (VERBOSE) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   140
            System.out.print("Execute command: ");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   141
            for (String c : command) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   142
                System.out.print(c);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   143
                System.out.print(" ");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   144
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   145
            System.out.println();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   146
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   147
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   148
        ProcessBuilder builder = new ProcessBuilder(command);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   149
        if (out != null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   150
            builder.redirectErrorStream(true);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   151
            builder.redirectOutput(out);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   152
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   153
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   154
        return builder.start();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   155
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   156
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   157
    private static String[] getCommand(String... args) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   158
        String[] command;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   159
        if (args == null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   160
            command = new String[1];
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   161
        } else {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   162
            command = new String[args.length + 1];
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   163
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   164
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   165
        int index = 0;
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   166
        command[index] = JPACKAGE.toString();
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   167
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   168
        if (args != null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   169
            for (String arg : args) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   170
                index++;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   171
                command[index] = arg;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   172
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   173
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   174
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   175
        return command;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   176
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   177
57314
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   178
    public static void deleteRecursive(File path) throws IOException {
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   179
        if (!path.exists()) {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   180
            return;
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   181
        }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   182
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   183
        Path directory = path.toPath();
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   184
        Files.walkFileTree(directory, new SimpleFileVisitor<Path>() {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   185
            @Override
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   186
            public FileVisitResult visitFile(Path file,
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   187
                    BasicFileAttributes attr) throws IOException {
57438
4a31db8d42bd 8226599: use code coverage results to remove dead code
herrick
parents: 57419
diff changeset
   188
                file.toFile().setWritable(true);
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   189
                if (OS.startsWith("win")) {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   190
                    try {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   191
                        Files.setAttribute(file, "dos:readonly", false);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   192
                    } catch (Exception ioe) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   193
                        // just report and try to contune
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   194
                        System.err.println("IOException: " + ioe);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   195
                        ioe.printStackTrace(System.err);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   196
                    }
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   197
                }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   198
                Files.delete(file);
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   199
                return FileVisitResult.CONTINUE;
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   200
            }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   201
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   202
            @Override
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   203
            public FileVisitResult preVisitDirectory(Path dir,
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   204
                    BasicFileAttributes attr) throws IOException {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   205
                if (OS.startsWith("win")) {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   206
                    Files.setAttribute(dir, "dos:readonly", false);
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   207
                }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   208
                return FileVisitResult.CONTINUE;
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   209
            }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   210
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   211
            @Override
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   212
            public FileVisitResult postVisitDirectory(Path dir, IOException e)
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   213
                    throws IOException {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   214
                Files.delete(dir);
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   215
                return FileVisitResult.CONTINUE;
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   216
            }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   217
        });
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   218
    }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   219
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   220
    public static void deleteOutputFolder(String output) throws IOException {
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   221
        File outputFolder = new File(output);
57405
539d8b3f9e1e 8224597: create automated tests for platform create-app-image options
herrick
parents: 57395
diff changeset
   222
        System.out.println("deleteOutputFolder: " + outputFolder.getAbsolutePath());
57314
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   223
        try {
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   224
            deleteRecursive(outputFolder);
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   225
        } catch (IOException ioe) {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   226
            System.err.println("IOException: " + ioe);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   227
            ioe.printStackTrace(System.err);
57314
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   228
            deleteRecursive(outputFolder);
23da9e6f446f 8221641: Follow up code clean up for JDK-8221582
herrick
parents: 57241
diff changeset
   229
        }
57241
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   230
    }
e11f3bf34083 8219889: Update jpackage tests for JDK-8219678 changes
herrick
parents: 57151
diff changeset
   231
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   232
    public static String executeCLI(boolean retValZero, String... args) throws Exception {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   233
        int retVal;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   234
        File outfile = new File("output.log");
57414
6eda749d3117 8225428: CLI change to remove "mode", rename to "package", and build only one target
herrick
parents: 57405
diff changeset
   235
        String[] command = getCommand(args);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   236
        try {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   237
            retVal = execute(outfile, command);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   238
        } catch (Exception ex) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   239
            if (outfile.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   240
                System.err.println(Files.readString(outfile.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   241
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   242
            throw ex;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   243
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   244
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   245
        String output = Files.readString(outfile.toPath());
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   246
        if (retValZero) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   247
            if (retVal != 0) {
57414
6eda749d3117 8225428: CLI change to remove "mode", rename to "package", and build only one target
herrick
parents: 57405
diff changeset
   248
                System.err.println("command run:");
6eda749d3117 8225428: CLI change to remove "mode", rename to "package", and build only one target
herrick
parents: 57405
diff changeset
   249
                for (String s : command) { System.err.println(s); }
6eda749d3117 8225428: CLI change to remove "mode", rename to "package", and build only one target
herrick
parents: 57405
diff changeset
   250
                System.err.println("command output:");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   251
                System.err.println(output);
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   252
                throw new AssertionError("jpackage exited with error: " + retVal);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   253
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   254
        } else {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   255
            if (retVal == 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   256
                System.err.println(output);
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   257
                throw new AssertionError("jpackage exited without error: " + retVal);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   258
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   259
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   260
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   261
        if (VERBOSE) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   262
            System.out.println("output =");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   263
            System.out.println(output);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   264
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   265
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   266
        return output;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   267
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   268
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   269
    public static String executeToolProvider(boolean retValZero, String... args) throws Exception {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   270
        StringWriter writer = new StringWriter();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   271
        PrintWriter pw = new PrintWriter(writer);
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   272
        int retVal = JPACKAGE_TOOL.run(pw, pw, args);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   273
        String output = writer.toString();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   274
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   275
        if (retValZero) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   276
            if (retVal != 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   277
                System.err.println(output);
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   278
                throw new AssertionError("jpackage exited with error: " + retVal);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   279
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   280
        } else {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   281
            if (retVal == 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   282
                System.err.println(output);
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   283
                throw new AssertionError("jpackage exited without error");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   284
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   285
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   286
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   287
        if (VERBOSE) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   288
            System.out.println("output =");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   289
            System.out.println(output);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   290
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   291
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   292
        return output;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   293
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   294
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   295
    public static boolean isWindows() {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   296
        return (OS.contains("win"));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   297
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   298
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   299
    public static boolean isOSX() {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   300
        return (OS.contains("mac"));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   301
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   302
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   303
    public static boolean isLinux() {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   304
        return ((OS.contains("nix") || OS.contains("nux")));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   305
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   306
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   307
    public static void createHelloImageJar(String inputDir) throws Exception {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   308
        createJar(false, "Hello", "image", inputDir);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   309
    }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   310
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   311
    public static void createHelloImageJar() throws Exception {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   312
        createJar(false, "Hello", "image", "input");
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   313
    }
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   314
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   315
    public static void createHelloImageJarWithMainClass() throws Exception {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   316
        createJar(true, "Hello", "image", "input");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   317
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   318
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   319
    public static void createHelloInstallerJar() throws Exception {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   320
        createJar(false, "Hello", "installer", "input");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   321
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   322
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   323
    public static void createHelloInstallerJarWithMainClass() throws Exception {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   324
        createJar(true, "Hello", "installer", "input");
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   325
    }
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   326
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   327
    private static void createJar(boolean mainClassAttribute, String name,
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   328
        String testType, String inputDir) throws Exception {
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   329
        int retVal;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   330
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   331
        File input = new File(inputDir);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   332
        if (!input.exists()) {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   333
            input.mkdirs();
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   334
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   335
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   336
        Path src = Path.of(TEST_SRC_ROOT + File.separator + "apps"
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   337
                + File.separator + testType + File.separator + name + ".java");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   338
        Path dst = Path.of(name + ".java");
57405
539d8b3f9e1e 8224597: create automated tests for platform create-app-image options
herrick
parents: 57395
diff changeset
   339
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   340
        if (dst.toFile().exists()) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   341
            Files.delete(dst);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   342
        }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   343
        Files.copy(src, dst);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   344
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   345
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   346
        File javacLog = new File("javac.log");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   347
        try {
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   348
            retVal = execute(javacLog, JAVAC.toString(), name + ".java");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   349
        } catch (Exception ex) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   350
            if (javacLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   351
                System.err.println(Files.readString(javacLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   352
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   353
            throw ex;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   354
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   355
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   356
        if (retVal != 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   357
            if (javacLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   358
                System.err.println(Files.readString(javacLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   359
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   360
            throw new AssertionError("javac exited with error: " + retVal);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   361
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   362
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   363
        File jarLog = new File("jar.log");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   364
        try {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   365
            List<String> args = new ArrayList<>();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   366
            args.add(JAR.toString());
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   367
            args.add("-c");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   368
            args.add("-v");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   369
            args.add("-f");
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   370
            args.add(inputDir + File.separator + name.toLowerCase() + ".jar");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   371
            if (mainClassAttribute) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   372
                args.add("-e");
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   373
                args.add(name);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   374
            }
57079
c53a2eca0f57 8215036: Create initial set of tests for jpackage create-installer mode
herrick
parents: 57039
diff changeset
   375
            args.add(name + ".class");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   376
            retVal = execute(jarLog, args.stream().toArray(String[]::new));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   377
        } catch (Exception ex) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   378
            if (jarLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   379
                System.err.println(Files.readString(jarLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   380
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   381
            throw ex;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   382
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   383
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   384
        if (retVal != 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   385
            if (jarLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   386
                System.err.println(Files.readString(jarLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   387
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   388
            throw new AssertionError("jar exited with error: " + retVal);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   389
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   390
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   391
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   392
    public static void createHelloModule() throws Exception {
58305
d42b1f6960aa 8230651: Use version string from main module
herrick
parents: 57438
diff changeset
   393
        createModule("Hello.java", "input", "hello", null, true);
d42b1f6960aa 8230651: Use version string from main module
herrick
parents: 57438
diff changeset
   394
    }
d42b1f6960aa 8230651: Use version string from main module
herrick
parents: 57438
diff changeset
   395
58360
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   396
    public static void createHelloModule(ModuleArgs moduleArgs) throws Exception {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   397
        createModule("Hello.java", "input", "hello", moduleArgs, true);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   398
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   399
57324
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   400
    public static void createOtherModule() throws Exception {
58305
d42b1f6960aa 8230651: Use version string from main module
herrick
parents: 57438
diff changeset
   401
        createModule("Other.java", "input-other", "other", null, false);
57324
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   402
    }
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   403
58360
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   404
    private static void createModule(String javaFile, String inputDir, String aName,
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   405
            ModuleArgs moduleArgs, boolean createModularJar) throws Exception {
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   406
        int retVal;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   407
57324
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   408
        File input = new File(inputDir);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   409
        if (!input.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   410
            input.mkdir();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   411
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   412
57324
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   413
        File module = new File("module" + File.separator + "com." + aName);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   414
        if (!module.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   415
            module.mkdirs();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   416
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   417
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   418
        File javacLog = new File("javac.log");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   419
        try {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   420
            List<String> args = new ArrayList<>();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   421
            args.add(JAVAC.toString());
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   422
            args.add("-d");
57324
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   423
            args.add("module" + File.separator + "com." + aName);
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   424
            args.add(TEST_SRC_ROOT + File.separator + "apps" + File.separator
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   425
                    + "com." + aName + File.separator + "module-info.java");
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   426
            args.add(TEST_SRC_ROOT + File.separator + "apps"
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   427
                    + File.separator + "com." + aName + File.separator + "com"
c1d3935fbb79 8222406: Multiple arguments for the same option - aggragation broken
herrick
parents: 57314
diff changeset
   428
                    + File.separator + aName + File.separator + javaFile);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   429
            retVal = execute(javacLog, args.stream().toArray(String[]::new));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   430
        } catch (Exception ex) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   431
            if (javacLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   432
                System.err.println(Files.readString(javacLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   433
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   434
            throw ex;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   435
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   436
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   437
        if (retVal != 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   438
            if (javacLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   439
                System.err.println(Files.readString(javacLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   440
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   441
            throw new AssertionError("javac exited with error: " + retVal);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   442
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   443
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   444
        if (createModularJar) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   445
            File jarLog = new File("jar.log");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   446
            try {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   447
                List<String> args = new ArrayList<>();
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   448
                args.add(JAR.toString());
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   449
                args.add("--create");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   450
                args.add("--file");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   451
                args.add(inputDir + File.separator + "com." + aName + ".jar");
58360
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   452
                if (moduleArgs != null) {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   453
                    if (moduleArgs.getVersion() != null) {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   454
                        args.add("--module-version");
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   455
                        args.add(moduleArgs.getVersion());
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   456
                    }
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   457
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   458
                    if (moduleArgs.getMainClass()!= null) {
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   459
                        args.add("--main-class");
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   460
                        args.add(moduleArgs.getMainClass());
fd45b7e2c027 8231382: Use main class from main module if available
herrick
parents: 58359
diff changeset
   461
                    }
58305
d42b1f6960aa 8230651: Use version string from main module
herrick
parents: 57438
diff changeset
   462
                }
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   463
                args.add("-C");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   464
                args.add("module" + File.separator + "com." + aName);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   465
                args.add(".");
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   466
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   467
                retVal = execute(jarLog, args.stream().toArray(String[]::new));
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   468
            } catch (Exception ex) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   469
                if (jarLog.exists()) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   470
                    System.err.println(Files.readString(jarLog.toPath()));
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   471
                }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   472
                throw ex;
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   473
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   474
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   475
            if (retVal != 0) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   476
                if (jarLog.exists()) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   477
                    System.err.println(Files.readString(jarLog.toPath()));
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   478
                }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   479
                throw new AssertionError("jar exited with error: " + retVal);
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   480
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   481
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   482
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   483
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   484
    public static void createRuntime() throws Exception {
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   485
        List<String> moreArgs = new ArrayList<>();
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   486
        createRuntime(moreArgs);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   487
    }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   488
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   489
    public static void createRuntime(List<String> moreArgs) throws Exception {
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   490
        int retVal;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   491
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   492
        File jlinkLog = new File("jlink.log");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   493
        try {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   494
            List<String> args = new ArrayList<>();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   495
            args.add(JLINK.toString());
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   496
            args.add("--output");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   497
            args.add("runtime");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   498
            args.add("--add-modules");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   499
            args.add("java.base");
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   500
            args.addAll(moreArgs);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   501
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   502
            retVal = execute(jlinkLog, args.stream().toArray(String[]::new));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   503
        } catch (Exception ex) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   504
            if (jlinkLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   505
                System.err.println(Files.readString(jlinkLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   506
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   507
            throw ex;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   508
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   509
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   510
        if (retVal != 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   511
            if (jlinkLog.exists()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   512
                System.err.println(Files.readString(jlinkLog.toPath()));
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   513
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   514
            throw new AssertionError("jlink exited with error: " + retVal);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   515
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   516
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   517
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   518
    public static String listToArgumentsMap(List<String> arguments, boolean toolProvider) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   519
        if (arguments.isEmpty()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   520
            return "";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   521
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   522
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   523
        String argsStr = "";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   524
        for (int i = 0; i < arguments.size(); i++) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   525
            String arg = arguments.get(i);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   526
            argsStr += quote(arg, toolProvider);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   527
            if ((i + 1) != arguments.size()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   528
                argsStr += " ";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   529
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   530
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   531
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   532
        if (!toolProvider && isWindows()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   533
            if (argsStr.contains(" ")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   534
                if (argsStr.contains("\"")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   535
                    argsStr = escapeQuote(argsStr, toolProvider);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   536
                }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   537
                argsStr = "\"" + argsStr + "\"";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   538
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   539
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   540
        return argsStr;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   541
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   542
57395
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   543
    public static String[] cmdWithAtFilename(String [] cmd, int ndx, int len)
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   544
                throws IOException {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   545
        ArrayList<String> newAList = new ArrayList<>();
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   546
        String fileString = null;
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   547
        for (int i=0; i<cmd.length; i++) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   548
            if (i == ndx) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   549
                newAList.add("@argfile.cmds");
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   550
                fileString = cmd[i];
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   551
            } else if (i > ndx && i < ndx + len) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   552
                fileString += " " + cmd[i];
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   553
            } else {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   554
                newAList.add(cmd[i]);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   555
            }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   556
        }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   557
        if (fileString != null) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   558
            Path path = new File("argfile.cmds").toPath();
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   559
            try (BufferedWriter bw = Files.newBufferedWriter(path);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   560
                    PrintWriter out = new PrintWriter(bw)) {
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   561
                out.println(fileString);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   562
            }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   563
        }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   564
        return newAList.toArray(new String[0]);
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   565
    }
521c02b9eed0 8224130: create additional automated tests for create-app-image
herrick
parents: 57324
diff changeset
   566
57419
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   567
    public static String [] splitAndFilter(String output) {
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   568
        if (output == null) {
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   569
            return null;
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   570
        }
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   571
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   572
        String[] result = output.split("\n");
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   573
        if (result == null || result.length == 0) {
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   574
            return result;
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   575
        }
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   576
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   577
        List<String> origList = new ArrayList(Arrays.asList(result));
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   578
        List<String> newlist = new ArrayList();
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   579
        origList.stream().filter((str) ->
58359
e065fd274bc1 8230649 : Make jpackage tool an experimental feature
herrick
parents: 58305
diff changeset
   580
                (!str.startsWith("Picked up") &&
e065fd274bc1 8230649 : Make jpackage tool an experimental feature
herrick
parents: 58305
diff changeset
   581
                 !str.startsWith("WARNING: Using experimental")))
e065fd274bc1 8230649 : Make jpackage tool an experimental feature
herrick
parents: 58305
diff changeset
   582
                .forEachOrdered((str) -> {
57419
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   583
            newlist.add(str);
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   584
        });
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   585
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   586
        return newlist.toArray(new String[newlist.size()]);
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   587
    }
44c2481650ce 8225092: Several jpackage tests failes when run with jcov enabled
herrick
parents: 57414
diff changeset
   588
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   589
    private static String quote(String in, boolean toolProvider) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   590
        if (in == null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   591
            return null;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   592
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   593
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   594
        if (in.isEmpty()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   595
            return "";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   596
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   597
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   598
        if (!in.contains("=")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   599
            // Not a property
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   600
            if (in.contains(" ")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   601
                in = escapeQuote(in, toolProvider);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   602
                return "\"" + in + "\"";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   603
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   604
            return in;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   605
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   606
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   607
        if (!in.contains(" ")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   608
            return in; // No need to quote
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   609
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   610
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   611
        int paramIndex = in.indexOf("=");
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   612
        if (paramIndex <= 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   613
            return in; // Something wrong, just skip quoting
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   614
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   615
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   616
        String param = in.substring(0, paramIndex);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   617
        String value = in.substring(paramIndex + 1);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   618
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   619
        if (value.length() == 0) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   620
            return in; // No need to quote
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   621
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   622
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   623
        value = escapeQuote(value, toolProvider);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   624
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   625
        return param + "=" + "\"" + value + "\"";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   626
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   627
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   628
    private static String escapeQuote(String in, boolean toolProvider) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   629
        if (in == null) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   630
            return null;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   631
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   632
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   633
        if (in.isEmpty()) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   634
            return "";
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   635
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   636
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   637
        if (in.contains("\"")) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   638
            // Use code points to preserve non-ASCII chars
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   639
            StringBuilder sb = new StringBuilder();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   640
            int codeLen = in.codePointCount(0, in.length());
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   641
            for (int i = 0; i < codeLen; i++) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   642
                int code = in.codePointAt(i);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   643
                // Note: No need to escape '\' on Linux or OS X
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   644
                // jpackage expects us to pass arguments and properties with
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   645
                // quotes and spaces as a map
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   646
                // with quotes being escaped with additional \ for
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   647
                // internal quotes.
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   648
                // So if we want two properties below:
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   649
                // -Djnlp.Prop1=Some "Value" 1
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   650
                // -Djnlp.Prop2=Some Value 2
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   651
                // jpackage will need:
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   652
                // "-Djnlp.Prop1=\"Some \\"Value\\" 1\" -Djnlp.Prop2=\"Some Value 2\""
57039
98d3963b0b7b 8214051: rename jpackager tool to jpackage
herrick
parents: 57031
diff changeset
   653
                // but since we using ProcessBuilder to run jpackage we will need to escape
57031
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   654
                // our escape symbols as well, so we will need to pass string below to ProcessBuilder:
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   655
                // "-Djnlp.Prop1=\\\"Some \\\\\\\"Value\\\\\\\" 1\\\" -Djnlp.Prop2=\\\"Some Value 2\\\""
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   656
                switch (code) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   657
                    case '"':
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   658
                        // " -> \" -> \\\"
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   659
                        if (i == 0 || in.codePointAt(i - 1) != '\\') {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   660
                            sb.appendCodePoint('\\');
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   661
                            sb.appendCodePoint(code);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   662
                        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   663
                        break;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   664
                    case '\\':
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   665
                        // We need to escape already escaped symbols as well
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   666
                        if ((i + 1) < codeLen) {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   667
                            int nextCode = in.codePointAt(i + 1);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   668
                            if (nextCode == '"') {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   669
                                // \" -> \\\"
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   670
                                sb.appendCodePoint('\\');
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   671
                                sb.appendCodePoint('\\');
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   672
                                sb.appendCodePoint('\\');
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   673
                                sb.appendCodePoint(nextCode);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   674
                            } else {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   675
                                sb.appendCodePoint('\\');
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   676
                                sb.appendCodePoint(code);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   677
                            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   678
                        } else {
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   679
                            sb.appendCodePoint(code);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   680
                        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   681
                        break;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   682
                    default:
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   683
                        sb.appendCodePoint(code);
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   684
                        break;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   685
                }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   686
            }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   687
            return sb.toString();
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   688
        }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   689
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   690
        return in;
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   691
    }
ea4755429ed8 8214021: Create additional automated tests for jpackager
herrick
parents:
diff changeset
   692
}