Attempt 1: make sure to always flush buffered logs (#409)

scalameta · Aug 31, 2021 · 92f3ad9 · 92f3ad9
1 parent 9cfd8b1
commit 92f3ad9
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/junit-interface/src/main/java/munit/internal/junitinterface/EventDispatcher.java b/junit-interface/src/main/java/munit/internal/junitinterface/EventDispatcher.java
@@ -159,7 +159,6 @@ public void testRunFinished(Result result)
           ", " +
           result.getIgnoreCount()+" ignored" +
           ", "+result.getRunCount()+" total, "+(result.getRunTime()/1000.0)+"s") ;
-        logger.flush();
       }
     runStatistics.addTime(result.getRunTime());
   }
@@ -174,7 +173,6 @@ public void testRunStarted(Description desc)
   {
       if (settings.verbose) {
         logger.info(taskInfo + " started");
-        logger.flush();
       }
   }
 
@@ -183,7 +181,6 @@ void testExecutionFailed(String testName, Throwable err)
     post(new Event(Ansi.c(testName, Ansi.ERRMSG), settings.buildErrorMessage(err), Status.Error, 0L, err) {
       void logTo(RichLogger logger) {
         logger.error(ansiName+" failed: "+ansiMsg, error);
-        logger.flush();
       }
     });
   }

diff --git a/junit-interface/src/main/java/munit/internal/junitinterface/JUnitTask.java b/junit-interface/src/main/java/munit/internal/junitinterface/JUnitTask.java
@@ -79,6 +79,7 @@ public Task[] execute(EventHandler eventHandler, Logger[] loggers) {
         ed.testExecutionFailed(testClassName, ex);
       }
     } finally {
+      logger.flush();
       settings.restoreSystemProperties(oldprops);
     }
     return new Task[0]; // junit tests do not nest