[SUREFIRE-2124] Avoid creating unnecessary target files for pom projects

apache · Jun 27, 2024 · 4eaf1f4 · 4eaf1f4
1 parent d626ced
commit 4eaf1f4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/...n-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java b/...n-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java
@@ -553,15 +553,11 @@ protected int getRerunFailingTestsCount() {
     @SuppressWarnings("unchecked")
     protected void handleSummary(RunResult summary, Exception firstForkException)
             throws MojoExecutionException, MojoFailureException {
-        File summaryFile = getSummaryFile();
-        if (!summaryFile.getParentFile().isDirectory()) {
-            //noinspection ResultOfMethodCallIgnored
-            summaryFile.getParentFile().mkdirs();
-        }
-
         try {
             Object token = getPluginContext().get(FAILSAFE_IN_PROGRESS_CONTEXT_KEY);
-            writeSummary(summary, summaryFile, token != null);
+            if (!summary.isNoTestsRun()) {
+                writeSummary(summary, getSummaryFile(), token != null);
+            }
         } catch (Exception e) {
             throw new MojoExecutionException(e.getMessage(), e);
         }

diff --git a/surefire-api/src/main/java/org/apache/maven/surefire/api/suite/RunResult.java b/surefire-api/src/main/java/org/apache/maven/surefire/api/suite/RunResult.java
@@ -86,9 +86,9 @@ public RunResult(
         this.errors = errors;
         this.failures = failures;
         this.skipped = skipped;
+        this.flakes = flakes;
         this.failure = failure;
         this.timeout = timeout;
-        this.flakes = flakes;
     }
 
     private static String getStackTrace(Exception e) {
@@ -174,6 +174,10 @@ public static RunResult noTestsRun() {
         return new RunResult(0, 0, 0, 0);
     }
 
+    public boolean isNoTestsRun() {
+        return completedCount == 0 && errors == 0 && skipped == 0 && flakes == 0 && failure == null && !timeout;
+    }
+
     @Override
     @SuppressWarnings("RedundantIfStatement")
     public boolean equals(Object o) {