Exit immediately if no failed tests are found.

Author: tpetrov-lp

pom.xml

@@ -7,7 +7,7 @@
     <groupId>com.clevertap</groupId>
     <artifactId>supertest-maven-plugin</artifactId>
     <packaging>maven-plugin</packaging>
-    <version>1.10</version>
+    <version>1.11-SNAPSHOT</version>
     <description>A wrapper for Maven's Surefire Plugin, with advanced re-run capabilities.
     </description>
     <name>supertest-maven-plugin</name>

src/main/java/com/clevertap/maven/plugins/supertest/SuperTestMavenPlugin.java

@@ -101,6 +101,12 @@ public void execute() throws MojoExecutionException {
             }
 
             final String runCommand = createRerunCommand(classnameToTestcaseList);
+
+            // previous run exited with code > 0, but all tests were actually run successfully
+            if (runCommand == null) {
+                return;
+            }
+
             final StringBuilder rerunCommand = new StringBuilder(runCommand);
             rerunCommand.append(buildProcessedMvnTestOpts(artifactId, groupId));
             if (rerunProfile != null) {
@@ -245,13 +251,15 @@ public File[] getXmlFileList(File baseDir) {
      * @return rerunCommand
      */
     public String createRerunCommand(Map<String, List<String>> classnameToTestcaseList) {
+        boolean hasTestsAppended = false;
         final StringBuilder retryRun = new StringBuilder("mvn test");
         retryRun.append(" -Dtest=");
         // TODO: 04/02/2022 replace with Java 8 streams
         for (String className : classnameToTestcaseList.keySet()) {
             List<String> failedTestCaseList = classnameToTestcaseList.get(className);
             if (!failedTestCaseList.isEmpty()) {
                 retryRun.append(className);
+                hasTestsAppended = true;
                 if(failedTestCaseList.contains("")) {
                     retryRun.append(",");
                     continue;
@@ -267,6 +275,6 @@ public String createRerunCommand(Map<String, List<String>> classnameToTestcaseLi
                 }
             }
         }
-        return retryRun.toString();
+        return hasTestsAppended ? retryRun.toString() : null;
     }
 }