Display time and consumed memory even in case of insufficient MSI (#1562

) `Time: 3s. Memory: 15Mb` - this is printed on `ApplicationExecutionWasFinished` event, but if MSI check fails, exception is thrown and `ApplicationExecutionWasFinished` is not dispatched, so time and memory is not printed. This change fixes the issue. Now, `ApplicationExecutionWasFinished` event is dispatched regardless of thrown exception
infection · Aug 29, 2021 · c87fab9 · c87fab9
1 parent 8cc8140
commit c87fab9
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/src/Engine.php b/src/Engine.php
@@ -110,14 +110,16 @@ public function execute(): void
         $this->runInitialTestSuite();
         $this->runMutationAnalysis();
 
-        $this->minMsiChecker->checkMetrics(
-            $this->metricsCalculator->getTestedMutantsCount(),
-            $this->metricsCalculator->getMutationScoreIndicator(),
-            $this->metricsCalculator->getCoveredCodeMutationScoreIndicator(),
-            $this->consoleOutput
-        );
-
-        $this->eventDispatcher->dispatch(new ApplicationExecutionWasFinished());
+        try {
+            $this->minMsiChecker->checkMetrics(
+                $this->metricsCalculator->getTestedMutantsCount(),
+                $this->metricsCalculator->getMutationScoreIndicator(),
+                $this->metricsCalculator->getCoveredCodeMutationScoreIndicator(),
+                $this->consoleOutput
+            );
+        } finally {
+            $this->eventDispatcher->dispatch(new ApplicationExecutionWasFinished());
+        }
     }
 
     private function runInitialTestSuite(): void