Hook on finished instead of passed test event

ergebnis · Dec 3, 2023 · e910fd3 · e910fd3
1 parent c101046
commit e910fd3
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/composer-require-checker.json b/composer-require-checker.json
@@ -7,8 +7,8 @@
     "int",
     "null",
     "PHPUnit\\Event\\Code\\Test",
-    "PHPUnit\\Event\\Test\\Passed",
-    "PHPUnit\\Event\\Test\\PassedSubscriber",
+    "PHPUnit\\Event\\Test\\Finished",
+    "PHPUnit\\Event\\Test\\FinishedSubscriber",
     "PHPUnit\\Event\\Test\\Prepared",
     "PHPUnit\\Event\\Test\\PreparedSubscriber",
     "PHPUnit\\Event\\TestRunner\\ExecutionFinished",

diff --git a/psalm-baseline.xml b/psalm-baseline.xml
@@ -21,12 +21,12 @@
       <code>($durationInMilliseconds - $hoursInMilliseconds - $minutesInMilliseconds) / 1_000</code>
     </InvalidOperand>
   </file>
-  <file src="src/Subscriber/TestPassedSubscriber.php">
+  <file src="src/Subscriber/TestFinishedSubscriber.php">
     <UndefinedClass>
-      <code>Event\Test\PassedSubscriber</code>
+      <code>Event\Test\FinishedSubscriber</code>
     </UndefinedClass>
     <UnusedClass>
-      <code>TestPassedSubscriber</code>
+      <code>TestFinishedSubscriber</code>
     </UnusedClass>
     <UnusedParam>
       <code>$collector</code>

diff --git a/src/Extension.php b/src/Extension.php
@@ -35,7 +35,7 @@
      */
     final class Extension implements
         Runner\AfterLastTestHook,
-        Runner\AfterSuccessfulTestHook,
+        Runner\AfterTestHook,
         Runner\BeforeFirstTestHook
     {
         private int $suites = 0;
@@ -71,7 +71,7 @@ public function executeBeforeFirstTest(): void
             ++$this->suites;
         }
 
-        public function executeAfterSuccessfulTest(
+        public function executeAfterTest(
             string $test,
             float $time
         ): void {
@@ -210,7 +210,7 @@ public function bootstrap(
 
             $facade->registerSubscribers(
                 new Subscriber\TestPreparedSubscriber($timeKeeper),
-                new Subscriber\TestPassedSubscriber(
+                new Subscriber\TestFinishedSubscriber(
                     $maximumDuration,
                     $timeKeeper,
                     $collector,

diff --git a/src/Subscriber/TestPassedSubscriber.php → src/Subscriber/TestFinishedSubscriber.php b/src/Subscriber/TestPassedSubscriber.php → src/Subscriber/TestFinishedSubscriber.php
@@ -26,7 +26,7 @@
 /**
  * @internal
  */
-final class TestPassedSubscriber implements Event\Test\PassedSubscriber
+final class TestFinishedSubscriber implements Event\Test\FinishedSubscriber
 {
     private Collector\Collector $collector;
     private TimeKeeper $timeKeeper;
@@ -42,7 +42,7 @@ public function __construct(
         $this->collector = $collector;
     }
 
-    public function notify(Event\Test\Passed $event): void
+    public function notify(Event\Test\Finished $event): void
     {
         $testIdentifier = TestIdentifier::fromString($event->test()->id());