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
+ */
 publicfunctiontestCouldBeConstructedWithContextAndQueueAsArguments()
 {
 newAmqpConsumer($this->createContextMock(), newAmqpQueue('aName'));

pkg/amqp-bunny/Tests/AmqpProducerTest.php

@@ -24,6 +24,10 @@ class AmqpProducerTest extends TestCase
 {
 use ClassExtensionTrait;
 
+/**
+ * @doesNotPerformAssertions
+ */
+
 publicfunctiontestCouldBeConstructedWithRequiredArguments()
 {
 newAmqpProducer($this->createBunnyChannelMock(), $this->createContextMock());

pkg/amqp-bunny/Tests/AmqpSubscriptionConsumerTest.php

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

pkg/amqp-ext/Tests/AmqpConsumerTest.php

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

pkg/amqp-ext/Tests/AmqpContextTest.php

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

pkg/amqp-ext/Tests/AmqpSubscriptionConsumerTest.php

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

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

@@ -24,6 +24,9 @@ public function testShouldImplementProcessorInterface()
 $this->assertClassImplements(Processor::class, AsyncProcessor::class);
  }
 
+/**
+ * @doesNotPerformAssertions
+ */
 publicfunctiontestCouldBeConstructedWithRegistryAndProxyEventDispatcher()
 {
 newAsyncProcessor($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
+ */
 publicfunctiontestCouldBeConstructedWithoutAnyArguments()
 {
 newPhpSerializerEventTransformer($this->createContextStub());

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

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

pkg/dbal/Tests/DbalConsumerTest.php

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