Removing constructor tests

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

Author: JimTools

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();

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);

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()
     {

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();

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();

Tests/Client/ConsumptionExtension/LogExtensionTest.php

@@ -55,14 +55,6 @@ public function testShouldSubClassOfLogExtension()
         $this->assertClassExtends(\Enqueue\Consumption\Extension\LogExtension::class, LogExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new LogExtension();
-    }
-
     public function testShouldLogStartOnStart()
     {
         $logger = $this->createLogger();

Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php

@@ -27,14 +27,6 @@ public function testShouldImplementMessageReceivedExtensionInterface()
         $this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new SetRouterPropertiesExtension($this->createDriverMock());
-    }
-
     public function testShouldSetRouterProcessorPropertyIfNotSetAndOnRouterQueue()
     {
         $config = Config::create('test', '.', '', '', 'router-queue', '', 'router-processor-name');

Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php

@@ -21,14 +21,6 @@ public function testShouldImplementStartExtensionInterface()
         $this->assertClassImplements(StartExtensionInterface::class, SetupBrokerExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new SetupBrokerExtension($this->createDriverMock());
-    }
-
     public function testShouldSetupBroker()
     {
         $logger = new NullLogger();

Tests/Client/DelegateProcessorTest.php

@@ -13,14 +13,6 @@
 
 class DelegateProcessorTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new DelegateProcessor($this->createProcessorRegistryMock());
-    }
-
     public function testShouldThrowExceptionIfProcessorNameIsNotSet()
     {
         $this->expectException(\LogicException::class);

Tests/Client/DriverFactoryTest.php

@@ -61,14 +61,6 @@ public function testShouldBeFinal()
         $this->assertTrue($rc->isFinal());
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new DriverFactory();
-    }
-
     public function testThrowIfPackageThatSupportSchemeNotInstalled()
     {
         $scheme = 'scheme5b7aa7d7cd213';

Tests/Client/DriverPreSendTest.php

@@ -19,18 +19,6 @@ public function testShouldBeFinal()
         self::assertClassFinal(DriverPreSend::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExpectedArguments()
-    {
-        new DriverPreSend(
-            new Message(),
-            $this->createProducerMock(),
-            $this->createDriverMock()
-        );
-    }
-
     public function testShouldAllowGetArgumentSetInConstructor()
     {
         $expectedMessage = new Message();

Tests/Client/Extension/PrepareBodyExtensionTest.php

@@ -22,14 +22,6 @@ public function testShouldImplementExtensionInterface()
         $this->assertTrue($rc->implementsInterface(PreSendCommandExtensionInterface::class));
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldConstructedWithoutAnyArguments()
-    {
-        new PrepareBodyExtension();
-    }
-
     /**
      * @dataProvider provideMessages
      *

Tests/Client/PostSendTest.php

@@ -21,20 +21,6 @@ public function testShouldBeFinal()
         self::assertClassFinal(PostSend::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExpectedArguments()
-    {
-        new PostSend(
-            new Message(),
-            $this->createProducerMock(),
-            $this->createDriverMock(),
-            $this->createDestinationMock(),
-            $this->createTransportMessageMock()
-        );
-    }
-
     public function testShouldAllowGetArgumentSetInConstructor()
     {
         $expectedMessage = new Message();

Tests/Client/PreSendTest.php

@@ -18,19 +18,6 @@ public function testShouldBeFinal()
         self::assertClassFinal(PreSend::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExpectedArguments()
-    {
-        new PreSend(
-            'aCommandOrTopic',
-            new Message(),
-            $this->createProducerMock(),
-            $this->createDriverMock()
-        );
-    }
-
     public function testShouldAllowGetArgumentSetInConstructor()
     {
         $expectedCommandOrTopic = 'theCommandOrTopic';

Tests/Client/ProducerTest.php

@@ -24,26 +24,6 @@ public function testShouldBeFinal()
         self::assertClassFinal(Producer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new Producer($this->createDriverMock(), $this->createRpcFactoryMock());
-    }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithOptionalArguments()
-    {
-        new Producer(
-            $this->createDriverMock(),
-            $this->createRpcFactoryMock(),
-            $this->createMock(ExtensionInterface::class)
-        );
-    }
-
     /**
      * @return \PHPUnit\Framework\MockObject\MockObject
      */

Tests/Client/SpoolProducerTest.php

@@ -18,14 +18,6 @@ public function testShouldImplementProducerInterface()
         self::assertClassImplements(ProducerInterface::class, SpoolProducer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRealProducer()
-    {
-        new SpoolProducer($this->createProducerMock());
-    }
-
     public function testShouldQueueEventMessageOnSend()
     {
         $message = new Message();

Tests/Client/TraceableProducerTest.php

@@ -18,14 +18,6 @@ public function testShouldImplementProducerInterface()
         $this->assertClassImplements(ProducerInterface::class, TraceableProducer::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithInternalMessageProducer()
-    {
-        new TraceableProducer($this->createProducerMock());
-    }
-
     public function testShouldPassAllArgumentsToInternalEventMessageProducerSendMethod()
     {
         $topic = 'theTopic';

Tests/ConnectionFactoryFactoryTest.php

@@ -37,14 +37,6 @@ public function testShouldBeFinal()
         $this->assertTrue($rc->isFinal());
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new ConnectionFactoryFactory();
-    }
-
     /**
      * @doesNotPerformAssertions
      */

Tests/Consumption/CallbackProcessorTest.php

@@ -18,15 +18,6 @@ public function testShouldImplementProcessorInterface()
         $this->assertClassImplements(Processor::class, CallbackProcessor::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithCallableAsArgument()
-    {
-        new CallbackProcessor(function () {
-        });
-    }
-
     public function testShouldCallCallbackAndProxyItsReturnedValue()
     {
         $expectedMessage = new NullMessage();

Tests/Consumption/ChainExtensionTest.php

@@ -33,14 +33,6 @@ public function testShouldImplementExtensionInterface()
         $this->assertClassImplements(ExtensionInterface::class, ChainExtension::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithExtensionsArray()
-    {
-        new ChainExtension([$this->createExtension(), $this->createExtension()]);
-    }
-
     public function testShouldProxyOnInitLoggerToAllInternalExtensions()
     {
         $context = new InitLogger(new NullLogger());

Tests/Consumption/Exception/IllegalContextModificationExceptionTest.php

@@ -20,12 +20,4 @@ public function testShouldExtendLogicException()
     {
         $this->assertClassExtends(\LogicException::class, IllegalContextModificationException::class);
     }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new IllegalContextModificationException();
-    }
 }

Tests/Consumption/Exception/InvalidArgumentExceptionTest.php

@@ -21,14 +21,6 @@ public function testShouldExtendLogicException()
         $this->assertClassExtends(\LogicException::class, InvalidArgumentException::class);
     }
 
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new InvalidArgumentException();
-    }
-
     public function testThrowIfAssertInstanceOfNotSameAsExpected()
     {
         $this->expectException(InvalidArgumentException::class);

Tests/Consumption/Exception/LogicExceptionTest.php

@@ -20,12 +20,4 @@ public function testShouldExtendLogicException()
     {
         $this->assertClassExtends(\LogicException::class, LogicException::class);
     }
-
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithoutAnyArguments()
-    {
-        new LogicException();
-    }
 }

Tests/Consumption/Extension/LimitConsumedMessagesExtensionTest.php

@@ -16,14 +16,6 @@
 
 class LimitConsumedMessagesExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new LimitConsumedMessagesExtension(12345);
-    }
-
     public function testOnPreConsumeShouldInterruptWhenLimitIsReached()
     {
         $logger = $this->createLoggerMock();

Tests/Consumption/Extension/LimitConsumerMemoryExtensionTest.php

@@ -17,14 +17,6 @@
 
 class LimitConsumerMemoryExtensionTest extends TestCase
 {
-    /**
-     * @doesNotPerformAssertions
-     */
-    public function testCouldBeConstructedWithRequiredArguments()
-    {
-        new LimitConsumerMemoryExtension(12345);
-    }
-
     public function testShouldThrowExceptionIfMemoryLimitIsNotInt()
     {
         $this->expectException(\InvalidArgumentException::class);