Make tests work on all operating systems

Author: jjohannes

src/test/java/org/gradlex/javamodule/packaging/test/JavaModulePackagingOptionsTest.java

@@ -96,7 +96,7 @@ void can_configure_java_options() {
 
         build.build(":app:jpackage");
 
-        assertThat(build.file("app/build/packages/macos/app.app/Contents/app/app.cfg").getAsPath()).hasContent("""
+        assertThat(build.appContentsFolder().file("app.cfg").getAsPath()).hasContent("""
             [Application]
             app.mainmodule=org.example.app/org.example.app.Main
             
@@ -133,7 +133,7 @@ void can_set_verbose_option() {
 
         var result = build.build(":app:jpackage");
 
-        assertThat(result.getOutput()).contains("Creating app package: app.app in");
+        assertThat(result.getOutput()).contains("Creating app package: ");
     }
 
     @Test
@@ -199,9 +199,8 @@ void can_build_package_in_one_step() {
 
         build.build(":app:jpackage");
 
-        assertThat(build.file("app/build/packages/macos/app-1.0.dmg").getAsPath()).exists();
-        assertThat(build.file("app/build/packages/macos/app-1.0.dmg.sha256").getAsPath()).exists();
-        assertThat(build.projectDir.dir("app/build/packages/macos").getAsPath()).isDirectoryNotContaining(
-                f -> f.toFile().isDirectory());
+        assertThat(build.appImageFolder().file("app-1.0.dmg").getAsPath()).exists();
+        assertThat(build.appImageFolder().file("app-1.0.dmg.sha256").getAsPath()).exists();
+        assertThat(build.appImageFolder().getAsPath()).isDirectoryNotContaining(f -> f.toFile().isDirectory());
     }
 }

src/test/java/org/gradlex/javamodule/packaging/test/JavaModulePackagingResourcesTest.java

@@ -21,6 +21,7 @@
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.gradlex.javamodule.packaging.test.fixture.GradleBuild.currentTarget;
 
 /**
  * Tests for adding custom resources to the image/package.
@@ -109,11 +110,13 @@ void can_add_resources_for_jpackage() {
         build.build(":app:jpackage");
 
         // Intermediate location to collect files
-        assertThat(build.file("app/build/tmp/jpackage/macos/jpackage-resources/dummy.txt").getAsPath()).exists();
-        assertThat(build.file("app/build/tmp/jpackage/macos/jpackage-resources/app.icns").getAsPath()).exists();
+        assertThat(build.file("app/build/tmp/jpackage/%s/jpackage-resources/dummy.txt".formatted(currentTarget()))
+                .getAsPath()).exists();
+        assertThat(build.file("app/build/tmp/jpackage/%s/jpackage-resources/app.icns".formatted(currentTarget()))
+                .getAsPath()).exists();
 
         // Icons end up in Resources
-        assertThat(build.file("app/build/packages/macos/app.app/Contents/Resources/app.icns").getAsPath()).hasSize(0);
+        assertThat(build.appContentsFolder().file("Resources/app.icns").getAsPath()).hasSize(0);
     }
 
     @Test
@@ -130,8 +133,7 @@ void can_add_resources_for_app_folder() {
 
         build.build(":app:jpackage");
 
-        // Icons end up in Resources
-        assertThat(build.file("app/build/packages/macos/app.app/Contents/app/dummy.txt").getAsPath()).exists();
+        assertThat(build.appContentsFolder().file("app/dummy.txt").getAsPath()).exists();
     }
 
     @Test
@@ -153,6 +155,6 @@ void can_add_resources_to_image_root() {
         build.build(":app:jpackage");
 
         // Icons end up in Resources
-        assertThat(build.file("app/build/packages/macos/app.app/Contents/customFolder/dummy.txt").getAsPath()).exists();
+        assertThat(build.appContentsFolder().file("customFolder/dummy.txt").getAsPath()).exists();
     }
 }

src/test/java/org/gradlex/javamodule/packaging/test/fixture/GradleBuild.java

@@ -108,19 +108,22 @@ public WritableFile file(String path) {
         return new WritableFile(projectDir, path);
     }
 
-    public BuildResult build(String task) {
-        return runner(task).build();
+    public Directory appImageFolder() {
+        if (runsOnMacos()) return projectDir.dir("app/build/packages/macos");
+        if (runsOnLinux()) return projectDir.dir("app/build/packages/linux");
+        if (runsOnWindows()) return projectDir.dir("app/build/packages/windows");
+        throw new IllegalStateException("unknown os");
     }
 
-    public BuildResult run() {
-        return runner("run").build();
+    public Directory appContentsFolder() {
+        if (runsOnMacos()) return projectDir.dir("app/build/packages/macos/app.app/Contents");
+        if (runsOnLinux()) return projectDir.dir("app/build/packages/linux/app/lib");
+        if (runsOnWindows()) return projectDir.dir("app/build/packages/windows/app");
+        throw new IllegalStateException("unknown os");
     }
 
-    public BuildResult printRuntimeJars() {
-        return runner(":app:printRuntimeJars", "-q").build();
-    }
-    public BuildResult printCompileJars() {
-        return runner(":app:printCompileJars", "-q").build();
+    public BuildResult build(String task) {
+        return runner(task).build();
     }
 
     public BuildResult fail(String task) {
@@ -162,6 +165,13 @@ private static Path createBuildTmpDir() {
         }
     }
 
+    public static String currentTarget() {
+        if (runsOnMacos()) return "macos";
+        if (runsOnLinux()) return "linux";
+        if (runsOnWindows()) return "windows";
+        throw new IllegalStateException("unknown os");
+    }
+
     public static boolean runsOnWindows() {
         return hostOs().contains("win");
     }