Skip to content

Commit 05b4dbd

Browse files
committed
Pass skipJVMTests argument only through command line
1 parent 11a0625 commit 05b4dbd

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

native-gradle-plugin/src/main/java/org/graalvm/buildtools/gradle/NativeImagePlugin.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -725,7 +725,8 @@ public void registerTestBinary(Project project,
725725
}
726726

727727
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);
729730
}
730731

731732
/**

native-maven-plugin/src/main/java/org/graalvm/buildtools/maven/NativeExtension.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,8 @@ public void afterProjectsRead(MavenSession session) {
197197
}
198198

199199
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);
201202
}
202203

203204
private static void setupMergeAgentFiles(PluginExecution exec, Xpp3Dom configuration, Context context) {

0 commit comments

Comments
 (0)