Fixing risky tests

Makring tests that have no assertions with the annotation to suppress
phpunits warning around risky tests

closes 1370

Author: JimTools

Tests/ArrayProcessorRegistryTest.php

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

Tests/Client/ChainExtensionTest.php

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

Tests/Client/ConsumptionExtension/DelayRedeliveredMessageExtensionTest.php

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

Tests/Client/ConsumptionExtension/ExclusiveCommandExtensionTest.php

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

Tests/Client/ConsumptionExtension/FlushSpoolProducerExtensionTest.php

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

Tests/Client/ConsumptionExtension/LogExtensionTest.php

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

Tests/Client/ConsumptionExtension/SetRouterPropertiesExtensionTest.php

@@ -27,6 +27,9 @@ public function testShouldImplementMessageReceivedExtensionInterface()
         $this->assertClassImplements(MessageReceivedExtensionInterface::class, SetRouterPropertiesExtension::class);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithRequiredArguments()
     {
         new SetRouterPropertiesExtension($this->createDriverMock());

Tests/Client/ConsumptionExtension/SetupBrokerExtensionTest.php

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

Tests/Client/DelegateProcessorTest.php

@@ -13,6 +13,9 @@
 
 class DelegateProcessorTest extends TestCase
 {
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithRequiredArguments()
     {
         new DelegateProcessor($this->createProcessorRegistryMock());

Tests/Client/DriverFactoryTest.php

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