Merge pull request #5 from robiningelbrecht/fix-final-status-bug

Fix final status bug

Author: robiningelbrecht

src/MinCoverage/ResultStatus.php

@@ -8,6 +8,16 @@ enum ResultStatus: string
     case WARNING = 'warning';
     case FAILED = 'failed';
 
+    public static function fromWeight(int $weight): self
+    {
+        return match ($weight) {
+            1 => self::SUCCESS ,
+            2 => self::WARNING,
+            3 => self::FAILED,
+            default => throw new \InvalidArgumentException('Invalid weight '.$weight),
+        };
+    }
+
     public function getWeight(): int
     {
         return match ($this) {

src/Subscriber/Application/ApplicationFinishedSubscriber.php

@@ -74,7 +74,10 @@ public function notify(Finished $event): void
             metrics: $metrics,
             metricTotal: $metricTotal,
         );
-        $finalStatus = array_values($results)[0]->getStatus();
+        $statusWeights = array_map(fn (MinCoverageResult $result) => $result->getStatus()->getWeight(), $results);
+        rsort($statusWeights, SORT_NUMERIC);
+
+        $finalStatus = ResultStatus::fromWeight($statusWeights[0]);
 
         $this->consoleOutput->print($results, $finalStatus);
 

tests/Subscriber/Application/ApplicationFinishedSubscriberTest.php

@@ -123,7 +123,7 @@ public function testNotifyWhenCoverageIsOk(): void
         $this->assertMatchesTextSnapshot($spyOutput);
     }
 
-    public function testNotifyWhitOnlyTotal(): void
+    public function testNotifyWithOnlyTotal(): void
     {
         $spyOutput = new SpyOutput();
         $subscriber = new ApplicationFinishedSubscriber(
@@ -154,7 +154,7 @@ public function testNotifyWhitOnlyTotal(): void
         $this->assertMatchesTextSnapshot($spyOutput);
     }
 
-    public function testNotifyWhitInvalidRules(): void
+    public function testNotifyWithInvalidRules(): void
     {
         $spyOutput = new SpyOutput();