Skip to content

Commit ff7f070

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 e627cd4 commit ff7f070

File tree

3 files changed

+12
-0
lines changed

3 files changed

+12
-0
lines changed

Tests/AmqpConsumerTest.php

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

22+
/**
23+
* @doesNotPerformAssertions
24+
*/
2225
public function testCouldBeConstructedWithContextAndQueueAsArguments()
2326
{
2427
new AmqpConsumer($this->createContext(), new AmqpQueue('aName'));

Tests/AmqpContextTest.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,17 @@ public function testShouldImplementQueueInteropContextInterface()
2828
$this->assertClassImplements(Context::class, AmqpContext::class);
2929
}
3030

31+
/**
32+
* @doesNotPerformAssertions
33+
*/
3134
public function testCouldBeConstructedWithExtChannelAsFirstArgument()
3235
{
3336
new AmqpContext($this->createExtChannelMock());
3437
}
3538

39+
/**
40+
* @doesNotPerformAssertions
41+
*/
3642
public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument()
3743
{
3844
new AmqpContext(function () {

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)