Skip to content

Commit 9a9a607

Browse files
committed
Fixing risky tests
Makring tests that have no assertions with the annotation to suppress phpunits warning around risky tests closes 1370
1 parent f412164 commit 9a9a607

File tree

3 files changed

+10
-0
lines changed

3 files changed

+10
-0
lines changed

Tests/AmqpConsumerTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,9 @@ public function testShouldImplementConsumerInterface()
2727
$this->assertClassImplements(Consumer::class, AmqpConsumer::class);
2828
}
2929

30+
/**
31+
* @doesNotPerformAssertions
32+
*/
3033
public function testCouldBeConstructedWithContextAndQueueAsArguments()
3134
{
3235
new AmqpConsumer($this->createContextMock(), new AmqpQueue('aName'));

Tests/AmqpProducerTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,10 @@ class AmqpProducerTest extends TestCase
2424
{
2525
use ClassExtensionTrait;
2626

27+
/**
28+
* @doesNotPerformAssertions
29+
*/
30+
2731
public function testCouldBeConstructedWithRequiredArguments()
2832
{
2933
new AmqpProducer($this->createBunnyChannelMock(), $this->createContextMock());

Tests/AmqpSubscriptionConsumerTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ public function testShouldImplementQueueInteropSubscriptionConsumerInterface()
1717
$this->assertTrue($rc->implementsInterface(SubscriptionConsumer::class));
1818
}
1919

20+
/**
21+
* @doesNotPerformAssertions
22+
*/
2023
public function testCouldBeConstructedWithAmqpContextAsFirstArgument()
2124
{
2225
new AmqpSubscriptionConsumer($this->createAmqpContextMock());

0 commit comments

Comments
 (0)