Fixing risky tests

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

closes 1370

Author: JimTools

Tests/FsConsumerTest.php

@@ -20,6 +20,9 @@ public function testShouldImplementConsumerInterface()
         $this->assertClassImplements(Consumer::class, FsConsumer::class);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithContextAndDestinationAndPreFetchCountAsArguments()
     {
         new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 1);
@@ -50,13 +53,19 @@ public function testShouldAllowGetPreviouslySetPreFetchCount()
         $this->assertSame(456, $consumer->getPreFetchCount());
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testShouldDoNothingOnAcknowledge()
     {
         $consumer = new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 123);
 
         $consumer->acknowledge(new FsMessage());
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testShouldDoNothingOnReject()
     {
         $consumer = new FsConsumer($this->createContextMock(), new FsDestination(TempFile::generate()), 123);

Tests/FsContextTest.php

@@ -24,6 +24,9 @@ public function testShouldImplementContextInterface()
         $this->assertClassImplements(Context::class, FsContext::class);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithExpectedArguments()
     {
         new FsContext(sys_get_temp_dir(), 1, 0666, 100);
@@ -122,6 +125,9 @@ public function testShouldThrowIfNotFsDestinationGivenOnCreateConsumer()
         $this->assertInstanceOf(FsConsumer::class, $consumer);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testShouldCreateConsumer()
     {
         $tmpFile = new TempFile(sys_get_temp_dir().'/foo');

Tests/FsProducerTest.php

@@ -23,6 +23,9 @@ public function testShouldImplementProducerInterface()
         $this->assertClassImplements(Producer::class, FsProducer::class);
     }
 
+    /**
+     * @doesNotPerformAssertions
+     */
     public function testCouldBeConstructedWithContextAsFirstArgument()
     {
         new FsProducer($this->createContextMock());