Fix: Rename TestSuite\RunStarted to TestSuite\Started

Related to #25.

.psalm/baseline.xml

@@ -447,11 +447,6 @@
       <code>RunFinishedSubscriber</code>
     </UnusedClass>
   </file>
-  <file src="src/Event/TestSuite/RunStartedSubscriber.php">
-    <UnusedClass occurrences="1">
-      <code>RunStartedSubscriber</code>
-    </UnusedClass>
-  </file>
   <file src="src/Event/TestSuite/Sorted.php">
     <PossiblyUnusedMethod occurrences="3">
       <code>executionOrder</code>
@@ -464,6 +459,11 @@
       <code>SortedSubscriber</code>
     </UnusedClass>
   </file>
+  <file src="src/Event/TestSuite/StartedSubscriber.php">
+    <UnusedClass occurrences="1">
+      <code>StartedSubscriber</code>
+    </UnusedClass>
+  </file>
   <file src="src/Framework/Assert.php">
     <ArgumentTypeCoercion occurrences="2">
       <code>$actualElement-&gt;childNodes-&gt;item($i)</code>

src/Event/DispatchingEmitter.php

@@ -309,11 +309,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(
@@ -324,6 +319,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

@@ -91,15 +91,15 @@ private static function registerDefaultTypes(TypeMap $typeMap): void
             Test\SkippedIncomplete::class,
             Test\SkippedWithMessage::class,
             TestDouble\MockCreated::class,
-            TestDouble\MockObjectCreatedForTrait::class,
             TestDouble\MockObjectCreatedForAbstractClass::class,
+            TestDouble\MockObjectCreatedForTrait::class,
             TestDouble\PartialMockObjectCreated::class,
             TestDouble\TestProxyCreated::class,
             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
 {
     /**
      * @var Telemetry\Info

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

@@ -672,7 +672,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
         self::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();
-
-        self::assertSame(1, $subscriber->recordedEventCount());
-        self::assertInstanceOf(TestSuite\RunStarted::class, $subscriber->lastRecordedEvent());
-    }
-
     public function testTestSuiteSortedDispatchesTestSuiteSortedEvent(): void
     {
         $executionOrder        = 9001;
@@ -1370,6 +1342,34 @@ public function notify(TestSuite\Sorted $event): void
         self::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();
+
+        self::assertSame(1, $subscriber->recordedEventCount());
+        self::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);
 
         self::assertSame($telemetryInfo, $event->telemetryInfo());
     }