#750 -DskipTests Flag should be honoured in a maven run without addit…

…ional configuration
hcoles · Jan 30, 2022 · 7b19bf0 · 7b19bf0
1 parent 1c2d301
commit 7b19bf0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/pitest-maven-verification/src/test/java/org/pitest/PitMojoIT.java b/pitest-maven-verification/src/test/java/org/pitest/PitMojoIT.java
@@ -32,6 +32,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.io.File;
+import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.Properties;
@@ -353,17 +354,14 @@ public void shouldCorrectlyHandleOverrides() throws Exception {
         buildFilePath(testDir, "target", "site", "pit-reports").exists());
   }
 
-  @Test
+  @Test(expected = FileNotFoundException.class)
   public void shouldReadExclusionsFromSurefireConfig() throws Exception {
     File testDir = prepare("/pit-surefire-excludes");
     verifier.addCliOption("-DskipTests");
     verifier.executeGoal("test");
     verifier.executeGoal("org.pitest:pitest-maven:mutationCoverage");
 
-    String actual = readResults(testDir);
-    assertThat(actual)
-        .contains(
-            "<mutation detected='false' status='NO_COVERAGE' numberOfTestsRun='0'><sourceFile>NotCovered.java</sourceFile>");
+    readResults(testDir);
   }
 
   @Test

diff --git a/pitest-maven/src/main/java/org/pitest/maven/AbstractPitMojo.java b/pitest-maven/src/main/java/org/pitest/maven/AbstractPitMojo.java
@@ -312,7 +312,7 @@ public class AbstractPitMojo extends AbstractMojo {
   /**
    * honours common skipTests flag in a maven run
    */
-  @Parameter(defaultValue = "false")
+  @Parameter(property = "skipTests", defaultValue = "false")
   private boolean                     skipTests;
 
   /**