[SUREFIRE-2223] Surefire evaluates parameter jvm before skip

This closes #703
apache · Dec 22, 2023 · caf37e5 · caf37e5
1 parent 8676714
commit caf37e5
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/...-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java b/...-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
@@ -876,6 +876,11 @@ private String getDefaultExcludes() {
 
     @Override
     public void execute() throws MojoExecutionException, MojoFailureException {
+        if (isSkipExecution()) {
+            getConsoleLogger().info("Tests are skipped.");
+            return;
+        }
+
         cli = commandLineOptions();
         // Stuff that should have been final
         setupStuff();
@@ -1030,10 +1035,6 @@ DefaultScanResult scanDependencies() throws MojoFailureException {
 
     boolean verifyParameters() throws MojoFailureException, MojoExecutionException {
         setProperties(new SurefireProperties(getProperties()));
-        if (isSkipExecution()) {
-            getConsoleLogger().info("Tests are skipped.");
-            return false;
-        }
 
         String jvmToUse = getJvm();
         if (toolchain != null) {