Delete unused code

Author: sebastianbergmann

.psalm/baseline.xml

@@ -474,9 +474,6 @@
       <code><![CDATA[$this->status]]></code>
       <code><![CDATA[$this->time]]></code>
     </PossiblyNullArgument>
-    <PropertyTypeCoercion>
-      <code><![CDATA[$this->testDoubles]]></code>
-    </PropertyTypeCoercion>
     <RedundantCondition>
       <code>assert($test instanceof TestMethod)</code>
     </RedundantCondition>

src/Logging/TestDox/TestMethod/Subscriber/TestCreatedMockObjectForAbstractClassSubscriber.php

@@ -26,21 +26,12 @@ final class TestResult
     private readonly TestStatus $status;
     private readonly ?Throwable $throwable;
 
-    /**
-     * @psalm-var list<class-string|trait-string>
-     */
-    private readonly array $testDoubles;
-
-    /**
-     * @psalm-param  list<class-string|trait-string> $testDoubles
-     */
-    public function __construct(TestMethod $test, Duration $duration, TestStatus $status, ?Throwable $throwable, array $testDoubles)
+    public function __construct(TestMethod $test, Duration $duration, TestStatus $status, ?Throwable $throwable)
     {
-        $this->test        = $test;
-        $this->duration    = $duration;
-        $this->status      = $status;
-        $this->throwable   = $throwable;
-        $this->testDoubles = $testDoubles;
+        $this->test      = $test;
+        $this->duration  = $duration;
+        $this->status    = $status;
+        $this->throwable = $throwable;
     }
 
     public function test(): TestMethod
@@ -70,12 +61,4 @@ public function throwable(): ?Throwable
     {
         return $this->throwable;
     }
-
-    /**
-     * @psalm-return list<class-string|trait-string>
-     */
-    public function testDoubles(): array
-    {
-        return $this->testDoubles;
-    }
 }

src/Logging/TestDox/TestMethod/Subscriber/TestCreatedMockObjectForTraitSubscriber.php

@@ -27,21 +27,13 @@
 use PHPUnit\Event\Test\Failed;
 use PHPUnit\Event\Test\Finished;
 use PHPUnit\Event\Test\MarkedIncomplete;
-use PHPUnit\Event\Test\MockObjectCreated;
-use PHPUnit\Event\Test\MockObjectForAbstractClassCreated;
-use PHPUnit\Event\Test\MockObjectForTraitCreated;
-use PHPUnit\Event\Test\MockObjectFromWsdlCreated;
-use PHPUnit\Event\Test\PartialMockObjectCreated;
 use PHPUnit\Event\Test\Passed;
 use PHPUnit\Event\Test\Prepared;
 use PHPUnit\Event\Test\Skipped;
-use PHPUnit\Event\Test\TestProxyCreated;
-use PHPUnit\Event\Test\TestStubCreated;
 use PHPUnit\Event\UnknownSubscriberTypeException;
 use PHPUnit\Framework\TestStatus\TestStatus;
 use PHPUnit\Logging\TestDox\TestResult as TestDoxTestMethod;
 use ReflectionMethod;
-use SoapClient;
 
 /**
  * @internal This class is not covered by the backward compatibility promise for PHPUnit
@@ -56,11 +48,6 @@ final class TestResultCollector
     private ?TestStatus $status   = null;
     private ?Throwable $throwable = null;
 
-    /**
-     * @psalm-var list<class-string|trait-string>
-     */
-    private array $testDoubles = [];
-
     /**
      * @throws EventFacadeIsSealedException
      * @throws UnknownSubscriberTypeException
@@ -140,10 +127,9 @@ public function testPrepared(Prepared $event): void
             return;
         }
 
-        $this->time        = $event->telemetryInfo()->time();
-        $this->status      = TestStatus::unknown();
-        $this->throwable   = null;
-        $this->testDoubles = [];
+        $this->time      = $event->telemetryInfo()->time();
+        $this->status    = TestStatus::unknown();
+        $this->throwable = null;
     }
 
     public function testErrored(Errored $event): void
@@ -191,23 +177,6 @@ public function testConsideredRisky(ConsideredRisky $event): void
         $this->status = TestStatus::risky($event->message());
     }
 
-    public function testCreatedTestDouble(MockObjectCreated|MockObjectForAbstractClassCreated|MockObjectForTraitCreated|MockObjectFromWsdlCreated|PartialMockObjectCreated|TestProxyCreated|TestStubCreated $event): void
-    {
-        if ($event instanceof MockObjectForTraitCreated) {
-            $this->testDoubles[] = $event->traitName();
-
-            return;
-        }
-
-        if ($event instanceof MockObjectFromWsdlCreated) {
-            $this->testDoubles[] = SoapClient::class;
-
-            return;
-        }
-
-        $this->testDoubles[] = $event->className();
-    }
-
     /**
      * @throws InvalidArgumentException
      */
@@ -230,13 +199,11 @@ public function testFinished(Finished $event): void
             $event->telemetryInfo()->time()->duration($this->time),
             $this->status,
             $this->throwable,
-            $this->testDoubles,
         );
 
-        $this->time        = null;
-        $this->status      = null;
-        $this->throwable   = null;
-        $this->testDoubles = [];
+        $this->time      = null;
+        $this->status    = null;
+        $this->throwable = null;
     }
 
     /**
@@ -247,13 +214,6 @@ private function registerSubscribers(Facade $facade): void
     {
         $facade->registerSubscribers(
             new TestConsideredRiskySubscriber($this),
-            new TestCreatedMockObjectForAbstractClassSubscriber($this),
-            new TestCreatedMockObjectForTraitSubscriber($this),
-            new TestCreatedMockObjectFromWsdlSubscriber($this),
-            new TestCreatedMockObjectSubscriber($this),
-            new TestCreatedPartialMockObjectSubscriber($this),
-            new TestCreatedTestProxySubscriber($this),
-            new TestCreatedTestStubSubscriber($this),
             new TestErroredSubscriber($this),
             new TestFailedSubscriber($this),
             new TestFinishedSubscriber($this),