Fixing risky tests

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

closes 1370

Author: JimTools

pkg/amqp-bunny/Tests/AmqpConsumerTest.php

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

pkg/amqp-bunny/Tests/AmqpProducerTest.php

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

pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php

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

pkg/amqp-ext/Tests/AmqpConsumerTest.php

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

pkg/amqp-ext/Tests/AmqpContextTest.php

@@ -28,11 +28,17 @@ public function testShouldImplementQueueInteropContextInterface()
         $this->assertClassImplements(Context::class, AmqpContext::class);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithExtChannelAsFirstArgument()
     {
         new AmqpContext($this->createExtChannelMock());
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument()
     {
         new AmqpContext(function () {

pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php

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

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

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

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

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

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

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

pkg/dbal/Tests/DbalConsumerTest.php

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