Pass skipJVMTests argument only through command line

dnestoro · dnestoro · commit 05b4dbd41da5 · 2025-08-28T15:23:53.000+02:00
diff --git a/native-gradle-plugin/src/main/java/org/graalvm/buildtools/gradle/NativeImagePlugin.java b/native-gradle-plugin/src/main/java/org/graalvm/buildtools/gradle/NativeImagePlugin.java
@@ -725,7 +725,8 @@ public void registerTestBinary(Project project,
     }
 
     private boolean shouldSkipJVMTests() {
-        return Boolean.parseBoolean(System.getProperty(SharedConstants.SKIP_JVM_TESTS));
+        String option = System.getProperty(SharedConstants.SKIP_JVM_TESTS);
+        return (option != null && option.isEmpty()) || Boolean.parseBoolean(option);
     }
 
     /**
diff --git a/native-maven-plugin/src/main/java/org/graalvm/buildtools/maven/NativeExtension.java b/native-maven-plugin/src/main/java/org/graalvm/buildtools/maven/NativeExtension.java
@@ -197,7 +197,8 @@ public void afterProjectsRead(MavenSession session) {
     }
 
     private static boolean shouldSkipJVMTests(MavenSession session) {
-        return Boolean.parseBoolean(session.getSystemProperties().getProperty(SharedConstants.SKIP_JVM_TESTS));
+        String option = session.getSystemProperties().getProperty(SharedConstants.SKIP_JVM_TESTS);
+        return (option != null && option.isEmpty()) || Boolean.parseBoolean(option);
     }
 
     private static void setupMergeAgentFiles(PluginExecution exec, Xpp3Dom configuration, Context context) {

Original file line number	Diff line number	Diff line change
`@@ -725,7 +725,8 @@ public void registerTestBinary(Project project,`
`725`	`725`	`}`
`726`	`726`
`727`	`727`	`private boolean shouldSkipJVMTests() {`
`728`		`- return Boolean.parseBoolean(System.getProperty(SharedConstants.SKIP_JVM_TESTS));`
	`728`	`+ String option = System.getProperty(SharedConstants.SKIP_JVM_TESTS);`
	`729`	`+ return (option != null && option.isEmpty()) \|\| Boolean.parseBoolean(option);`
`729`	`730`	`}`
`730`	`731`
`731`	`732`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,8 @@ public void afterProjectsRead(MavenSession session) {`
`197`	`197`	`}`
`198`	`198`
`199`	`199`	`private static boolean shouldSkipJVMTests(MavenSession session) {`
`200`		`- return Boolean.parseBoolean(session.getSystemProperties().getProperty(SharedConstants.SKIP_JVM_TESTS));`
	`200`	`+ String option = session.getSystemProperties().getProperty(SharedConstants.SKIP_JVM_TESTS);`
	`201`	`+ return (option != null && option.isEmpty()) \|\| Boolean.parseBoolean(option);`
`201`	`202`	`}`
`202`	`203`
`203`	`204`	`private static void setupMergeAgentFiles(PluginExecution exec, Xpp3Dom configuration, Context context) {`