Removing constructor tests

Removing all tests that creates a new instance but doesn't call any
methods nor preform any assertions.

Author: JimTools

pkg/amqp-bunny/Tests/AmqpConsumerTest.php

@@ -27,14 +27,6 @@ public function testShouldImplementConsumerInterface()
         $this->assertClassImplements(Consumer::class, AmqpConsumer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithContextAndQueueAsArguments()
-    {
-        new AmqpConsumer($this->createContextMock(), new AmqpQueue('aName'));
-    }
-
     public function testShouldReturnQueue()
     {
         $queue = new AmqpQueue('aName');

pkg/amqp-bunny/Tests/AmqpProducerTest.php

@@ -24,15 +24,6 @@ class AmqpProducerTest extends TestCase
 {
     use ClassExtensionTrait;
 
-    /**
-     * @doesNotPerformAssertions
-     */
-
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new AmqpProducer($this->createBunnyChannelMock(), $this->createContextMock());
-    }
-
     public function testShouldImplementQueueInteropProducerInterface()
     {
         $this->assertClassImplements(Producer::class, AmqpProducer::class);

pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php

@@ -17,14 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface()
         $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithAmqpContextAsFirstArgument()
-    {
-        new AmqpSubscriptionConsumer($this->createAmqpContextMock());
-    }
-
     /**
      * @return AmqpContext|MockObject
      */

pkg/amqp-ext/Tests/AmqpConsumerTest.php

@@ -19,14 +19,6 @@ public function testShouldImplementConsumerInterface()
         $this->assertClassImplements(Consumer::class, AmqpConsumer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithContextAndQueueAsArguments()
-    {
-        new AmqpConsumer($this->createContext(), new AmqpQueue('aName'));
-    }
-
     /**
      * @return MockObject|AmqpContext
      */

pkg/amqp-ext/Tests/AmqpContextTest.php

@@ -28,24 +28,6 @@ public function testShouldImplementQueueInteropContextInterface()
         $this->assertClassImplements(Context::class, AmqpContext::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExtChannelAsFirstArgument()
-    {
-        new AmqpContext($this->createExtChannelMock());
-    }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument()
-    {
-        new AmqpContext(function () {
-            return $this->createExtChannelMock();
-        });
-    }
-
     public function testThrowIfNeitherCallbackNorExtChannelAsFirstArgument()
     {
         $this->expectException(\InvalidArgumentException::class);

pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php

@@ -17,14 +17,6 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface()
         $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithAmqpContextAsFirstArgument()
-    {
-        new AmqpSubscriptionConsumer($this->createAmqpContextMock());
-    }
-
     /**
      * @return AmqpContext|MockObject
      */

pkg/async-event-dispatcher/Tests/AsyncProcessorTest.php

@@ -24,14 +24,6 @@ public function testShouldImplementProcessorInterface()
         $this->assertClassImplements(Processor::class, AsyncProcessor::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRegistryAndProxyEventDispatcher()
-    {
-        new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock());
-    }
-
     public function testRejectIfMessageMissingEventNameProperty()
     {
         $processor = new AsyncProcessor($this->createRegistryMock(), $this->createProxyEventDispatcherMock());

pkg/async-event-dispatcher/Tests/PhpSerializerEventTransformerTest.php

@@ -21,14 +21,6 @@ public function testShouldImplementEventTransformerInterface()
         $this->assertClassImplements(EventTransformer::class, PhpSerializerEventTransformer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new PhpSerializerEventTransformer($this->createContextStub());
-    }
-
     public function testShouldReturnMessageWithPhpSerializedEventAsBodyOnToMessage()
     {
         $transformer = new PhpSerializerEventTransformer($this->createContextStub());

pkg/async-event-dispatcher/Tests/SimpleRegistryTest.php

@@ -18,14 +18,6 @@ public function testShouldImplementRegistryInterface()
         $this->assertClassImplements(Registry::class, SimpleRegistry::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithEventsMapAndTransformersMapAsArguments()
-    {
-        new SimpleRegistry([], []);
-    }
-
     public function testShouldAllowGetTransportNameByEventName()
     {
         $registry = new SimpleRegistry([

pkg/dbal/Tests/DbalConsumerTest.php

@@ -28,14 +28,6 @@ public function testShouldImplementConsumerInterface()
         $this->assertClassImplements(Consumer::class, DbalConsumer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DbalConsumer($this->createContextMock(), new DbalDestination('queue'));
-    }
-
     public function testShouldReturnInstanceOfDestination()
     {
         $destination = new DbalDestination('queue');

pkg/dbal/Tests/DbalContextTest.php

@@ -27,14 +27,6 @@ public function testShouldImplementContextInterface()
         $this->assertClassImplements(Context::class, DbalContext::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DbalContext($this->createConnectionMock());
-    }
-
     public function testCouldBeConstructedWithEmptyConfiguration()
     {
         $factory = new DbalContext($this->createConnectionMock(), []);

pkg/dbal/Tests/DbalProducerTest.php

@@ -21,14 +21,6 @@ public function testShouldImplementProducerInterface()
         $this->assertClassImplements(Producer::class, DbalProducer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DbalProducer($this->createContextMock());
-    }
-
     public function testShouldThrowIfDestinationOfInvalidType()
     {
         $this->expectException(InvalidDestinationException::class);

pkg/dbal/Tests/DbalSubscriptionConsumerTest.php

@@ -25,14 +25,6 @@ public function testShouldImplementSubscriptionConsumerInterface()
         $this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithDbalContextAsFirstArgument()
-    {
-        new DbalSubscriptionConsumer($this->createDbalContextMock());
-    }
-
     public function testShouldAddConsumerAndCallbackToSubscribersPropertyOnSubscribe()
     {
         $subscriptionConsumer = new DbalSubscriptionConsumer($this->createDbalContextMock());

pkg/dsn/Tests/DsnTest.php

@@ -8,14 +8,6 @@
 
 class DsnTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithDsnAsFirstArgument()
-    {
-        Dsn::parseFirst('foo://localhost:1234');
-    }
-
     public function testThrowsIfSchemePartIsMissing()
     {
         $this->expectException(\LogicException::class);

pkg/enqueue-bundle/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();

pkg/enqueue-bundle/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();

pkg/enqueue-bundle/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()
     {

pkg/enqueue-bundle/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);

pkg/enqueue-bundle/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);

pkg/enqueue-bundle/Tests/Unit/EnqueueBundleTest.php

@@ -15,12 +15,4 @@ public function testShouldExtendBundleClass()
     {
         $this->assertClassExtends(Bundle::class, EnqueueBundle::class);
     }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new EnqueueBundle();
-    }
 }

pkg/enqueue-bundle/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();

pkg/enqueue/Tests/ArrayProcessorRegistryTest.php

@@ -18,14 +18,6 @@ public function testShouldImplementProcessorRegistryInterface()
         $this->assertClassImplements(ProcessorRegistryInterface::class, ArrayProcessorRegistry::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArgument()
-    {
-        new ArrayProcessorRegistry();
-    }
-
     public function testShouldThrowExceptionIfProcessorIsNotSet()
     {
         $registry = new ArrayProcessorRegistry();

pkg/enqueue/Tests/Client/ChainExtensionTest.php

@@ -29,14 +29,6 @@ public function testShouldBeFinal()
         $this->assertClassFinal(ChainExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExtensionsArray()
-    {
-        new ChainExtension([$this->createExtension(), $this->createExtension()]);
-    }
-
     public function testThrowIfArrayContainsNotExtension()
     {
         $this->expectException(\LogicException::class);

pkg/enqueue/Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php

@@ -23,13 +23,6 @@
 
 class DelayRedeliveredMessageExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DelayRedeliveredMessageExtension($this->createDriverMock(), 12345);
-    }
 
     public function testShouldSendDelayedMessageAndRejectOriginalMessage()
     {

pkg/enqueue/Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php

@@ -34,14 +34,6 @@ public function testShouldBeFinal()
         $this->assertClassFinal(ExclusiveCommandExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithDriverAsFirstArgument()
-    {
-        new ExclusiveCommandExtension($this->createDriverStub());
-    }
-
     public function testShouldDoNothingIfMessageHasTopicPropertySetOnPreReceive()
     {
         $message = new NullMessage();

pkg/enqueue/Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php

@@ -30,14 +30,6 @@ public function testShouldImplementEndExtensionInterface()
         $this->assertClassImplements(EndExtensionInterface::class, FlushSpoolProducerExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithSpoolProducerAsFirstArgument()
-    {
-        new FlushSpoolProducerExtension($this->createSpoolProducerMock());
-    }
-
     public function testShouldFlushSpoolProducerOnEnd()
     {
         $producer = $this->createSpoolProducerMock();