Merge pull request #1569 from stalep/quarkus-log-file2

When starting native image, use argument for log file location

Author: stalep

test-framework/common/src/main/java/io/quarkus/test/common/NativeImageLauncher.java

@@ -59,8 +59,7 @@ public void start() throws Exception {
         args.add(path);
         args.add("-Dquarkus.http.port=" + port);
         args.add("-Dtest.url=" + TestHTTPResourceManager.getUri());
-        //args.add("-Dquarkus.log.file.path=target/quarkus.log");
-        PropertyTestUtil.setLogFileProperty();
+        args.add("-Dquarkus.log.file.path=" + PropertyTestUtil.getLogFileLocation());
 
         System.out.println("Executing " + args);
 

test-framework/common/src/main/java/io/quarkus/test/common/PropertyTestUtil.java

@@ -22,9 +22,13 @@
 public class PropertyTestUtil {
 
     public static void setLogFileProperty() {
+        System.setProperty("quarkus.log.file.path", getLogFileLocation());
+    }
+
+    public static String getLogFileLocation() {
         if (Files.isDirectory(Paths.get("build"))) {
-            System.setProperty("quarkus.log.file.path", "build" + File.separator + "quarkus.log");
-        } else
-            System.setProperty("quarkus.log.file.path", "target" + File.separator + "quarkus.log");
+            return "build" + File.separator + "quarkus.log";
+        }
+        return "target" + File.separator + "quarkus.log";
     }
 }