Removing constructor tests

Removing all tests that creates a new instance but doesn't call any methods nor preform any assertions.
php-enqueue · Jan 15, 2025 · 0f35503 · 0f35503
1 parent f6c5b53
commit 0f35503
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 55 deletions.
diff --git a/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php b/Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php
@@ -16,14 +16,6 @@
 
 class DoctrineClearIdentityMapExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DoctrineClearIdentityMapExtension($this->createRegistryMock());
-    }
-
     public function testShouldClearIdentityMap()
     {
         $manager = $this->createManagerMock();

diff --git a/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php b/Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php
@@ -17,14 +17,6 @@
 
 class DoctrinePingConnectionExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredAttributes()
-    {
-        new DoctrinePingConnectionExtension($this->createRegistryMock());
-    }
-
     public function testShouldNotReconnectIfConnectionIsOK()
     {
         $connection = $this->createConnectionMock();

diff --git a/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php b/Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php
@@ -16,13 +16,6 @@
 
 class ResetServicesExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new ResetServicesExtension($this->createResetterMock());
-    }
 
     public function testItShouldResetServices()
     {

diff --git a/Tests/Unit/DependencyInjection/ConfigurationTest.php b/Tests/Unit/DependencyInjection/ConfigurationTest.php
@@ -25,14 +25,6 @@ public function testShouldBeFinal()
         $this->assertClassFinal(Configuration::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithDebugAsArgument()
-    {
-        new Configuration(true);
-    }
-
     public function testShouldProcessSeveralTransports()
     {
         $configuration = new Configuration(true);

diff --git a/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php b/Tests/Unit/DependencyInjection/EnqueueExtensionTest.php
@@ -30,14 +30,6 @@ public function testShouldBeFinal()
         $this->assertClassFinal(EnqueueExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new EnqueueExtension();
-    }
-
     public function testShouldRegisterConnectionFactory()
     {
         $container = $this->getContainerBuilder(true);

diff --git a/Tests/Unit/EnqueueBundleTest.php b/Tests/Unit/EnqueueBundleTest.php
@@ -15,12 +15,4 @@ public function testShouldExtendBundleClass()
     {
         $this->assertClassExtends(Bundle::class, EnqueueBundle::class);
     }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new EnqueueBundle();
-    }
 }
diff --git a/Tests/Unit/Profiler/MessageQueueCollectorTest.php b/Tests/Unit/Profiler/MessageQueueCollectorTest.php
@@ -23,14 +23,6 @@ public function testShouldExtendDataCollectorClass()
         $this->assertClassExtends(DataCollector::class, MessageQueueCollector::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithEmptyConstructor()
-    {
-        new MessageQueueCollector();
-    }
-
     public function testShouldReturnExpectedName()
     {
         $collector = new MessageQueueCollector();