Skip to content

Commit a42a793

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 e53f09c commit a42a793

3 files changed

Lines changed: 12 additions & 0 deletions

File tree

Tests/BufferedStompClientTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ public function testShouldExtendLibStompClient()
2121
$this->assertClassExtends(Client::class, BufferedStompClient::class);
2222
}
2323

24+
/**
25+
* @doesNotPerformAssertions
26+
*/
2427
public function testCouldBeConstructedWithRequiredArguments()
2528
{
2629
new BufferedStompClient('tcp://localhost:12345');

Tests/StompConsumerTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ public function testShouldImplementMessageConsumerInterface()
2525
$this->assertClassImplements(Consumer::class, StompConsumer::class);
2626
}
2727

28+
/**
29+
* @doesNotPerformAssertions
30+
*/
2831
public function testCouldBeConstructedWithRequiredAttributes()
2932
{
3033
new StompConsumer($this->createStompClientMock(), $this->createDummyDestination());

Tests/StompContextTest.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,17 @@ public function testShouldImplementSessionInterface()
2323
$this->assertClassImplements(Context::class, StompContext::class);
2424
}
2525

26+
/**
27+
* @doesNotPerformAssertions
28+
*/
2629
public function testCouldBeCreatedWithRequiredArguments()
2730
{
2831
new StompContext($this->createStompClientMock(), ExtensionType::RABBITMQ);
2932
}
3033

34+
/**
35+
* @doesNotPerformAssertions
36+
*/
3137
public function testCouldBeConstructedWithExtChannelCallbackFactoryAsFirstArgument()
3238
{
3339
new StompContext(function () {

0 commit comments

Comments
 (0)