Fix: Rename TestSuite\RunStarted to TestSuite\Started

Related to #25.

Co-authored-by: Andreas Möller <am@localheinz.com>
Co-authored-by: Arne Blankerts <Arne@Blankerts.de>

src/Event/DispatchingEmitter.php

@@ -300,11 +300,6 @@ public function testSuiteRunFinished(): void
         $this->dispatcher->dispatch(new TestSuite\RunFinished($this->telemetryInfo()));
     }
 
-    public function testSuiteRunStarted(): void
-    {
-        $this->dispatcher->dispatch(new TestSuite\RunStarted($this->telemetryInfo()));
-    }
-
     public function testSuiteSorted(int $executionOrder, int $executionOrderDefects, bool $resolveDependencies): void
     {
         $this->dispatcher->dispatch(new TestSuite\Sorted(
@@ -315,6 +310,11 @@ public function testSuiteSorted(int $executionOrder, int $executionOrderDefects,
         ));
     }
 
+    public function testSuiteStarted(): void
+    {
+        $this->dispatcher->dispatch(new TestSuite\Started($this->telemetryInfo()));
+    }
+
     private function telemetryInfo(): Telemetry\Info
     {
         $current = $this->system->snapshot();

src/Event/Emitter.php

@@ -131,7 +131,7 @@ public function testSuiteLoaded(TestSuite $testSuite): void;
 
     public function testSuiteRunFinished(): void;
 
-    public function testSuiteRunStarted(): void;
-
     public function testSuiteSorted(int $executionOrder, int $executionOrderDefects, bool $resolveDependencies): void;
+
+    public function testSuiteStarted(): void;
 }

src/Event/Registry.php

@@ -95,8 +95,8 @@ private static function registerDefaultTypes(TypeMap $typeMap): void
             TestSuite\AfterClassFinished::class,
             TestSuite\Loaded::class,
             TestSuite\RunFinished::class,
-            TestSuite\RunStarted::class,
             TestSuite\Sorted::class,
+            TestSuite\Started::class,
         ];
 
         foreach ($defaultEvents as $eventClass) {

src/Event/TestSuite/RunStarted.php → src/Event/TestSuite/Started.php

@@ -12,7 +12,7 @@
 use PHPUnit\Event\Event;
 use PHPUnit\Event\Telemetry;
 
-final class RunStarted implements Event
+final class Started implements Event
 {
     private Telemetry\Info $telemetryInfo;
 

src/Event/TestSuite/RunStartedSubscriber.php → src/Event/TestSuite/StartedSubscriber.php

@@ -11,7 +11,7 @@
 
 use PHPUnit\Event\Subscriber;
 
-interface RunStartedSubscriber extends Subscriber
+interface StartedSubscriber extends Subscriber
 {
-    public function notify(RunStarted $event): void;
+    public function notify(Started $event): void;
 }

src/TextUI/TestRunner.php

@@ -618,7 +618,7 @@ public function run(TestSuite $suite, array $arguments = [], array $warnings = [
             $this->write(PHP_EOL);
         }
 
-        Event\Registry::emitter()->testSuiteRunStarted();
+        Event\Registry::emitter()->testSuiteStarted();
 
         $suite->run($result);
 

tests/_files/NullEmitter.php

@@ -166,11 +166,11 @@ public function testSuiteRunFinished(): void
     {
     }
 
-    public function testSuiteRunStarted(): void
+    public function testSuiteSorted(int $executionOrder, int $executionOrderDefects, bool $resolveDependencies): void
     {
     }
 
-    public function testSuiteSorted(int $executionOrder, int $executionOrderDefects, bool $resolveDependencies): void
+    public function testSuiteStarted(): void
     {
     }
 }

tests/unit/Event/DispatchingEmitterTest.php

@@ -1299,34 +1299,6 @@ public function notify(TestSuite\RunFinished $event): void
         $this->assertInstanceOf(TestSuite\RunFinished::class, $subscriber->lastRecordedEvent());
     }
 
-    public function testTestSuiteRunStartedDispatchesTestSuiteRunStartedEvent(): void
-    {
-        $subscriber = new class extends RecordingSubscriber implements TestSuite\RunStartedSubscriber {
-            public function notify(TestSuite\RunStarted $event): void
-            {
-                $this->record($event);
-            }
-        };
-
-        $dispatcher = self::createDispatcherWithRegisteredSubscriber(
-            TestSuite\RunStartedSubscriber::class,
-            TestSuite\RunStarted::class,
-            $subscriber
-        );
-
-        $telemetrySystem = self::createTelemetrySystem();
-
-        $emitter = new DispatchingEmitter(
-            $dispatcher,
-            $telemetrySystem
-        );
-
-        $emitter->testSuiteRunStarted();
-
-        $this->assertSame(1, $subscriber->recordedEventCount());
-        $this->assertInstanceOf(TestSuite\RunStarted::class, $subscriber->lastRecordedEvent());
-    }
-
     public function testTestSuiteSortedDispatchesTestSuiteSortedEvent(): void
     {
         $executionOrder        = 9001;
@@ -1370,6 +1342,34 @@ public function notify(TestSuite\Sorted $event): void
         $this->assertSame($resolveDependencies, $event->resolveDependencies());
     }
 
+    public function testTestSuiteStartedDispatchesTestSuiteStartedEvent(): void
+    {
+        $subscriber = new class extends RecordingSubscriber implements TestSuite\StartedSubscriber {
+            public function notify(TestSuite\Started $event): void
+            {
+                $this->record($event);
+            }
+        };
+
+        $dispatcher = self::createDispatcherWithRegisteredSubscriber(
+            TestSuite\StartedSubscriber::class,
+            TestSuite\Started::class,
+            $subscriber
+        );
+
+        $telemetrySystem = self::createTelemetrySystem();
+
+        $emitter = new DispatchingEmitter(
+            $dispatcher,
+            $telemetrySystem
+        );
+
+        $emitter->testSuiteStarted();
+
+        $this->assertSame(1, $subscriber->recordedEventCount());
+        $this->assertInstanceOf(TestSuite\Started::class, $subscriber->lastRecordedEvent());
+    }
+
     private static function createDispatcherWithRegisteredSubscriber(string $subscriberInterface, string $eventClass, Subscriber $subscriber): Dispatcher
     {
         $typeMap = new TypeMap();

tests/unit/Event/TestSuite/RunStartedTest.php → tests/unit/Event/TestSuite/StartedTest.php

@@ -12,15 +12,15 @@
 use PHPUnit\Event\AbstractEventTestCase;
 
 /**
- * @covers \PHPUnit\Event\TestSuite\RunStarted
+ * @covers \PHPUnit\Event\TestSuite\Started
  */
-final class RunStartedTest extends AbstractEventTestCase
+final class StartedTest extends AbstractEventTestCase
 {
     public function testConstructorSetsValues(): void
     {
         $telemetryInfo = self::createTelemetryInfo();
 
-        $event = new RunStarted($telemetryInfo);
+        $event = new Started($telemetryInfo);
 
         $this->assertSame($telemetryInfo, $event->telemetryInfo());
     }