Skip to content

Commit f6c5b53

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 962401d commit f6c5b53

7 files changed

+21
-0
lines changed

Tests/Unit/Consumption/Extension/DoctrineClearIdentityMapExtensionTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@
1616

1717
class DoctrineClearIdentityMapExtensionTest extends TestCase
1818
{
19+
/**
20+
* @doesNotPerformAssertions
21+
*/
1922
public function testCouldBeConstructedWithRequiredArguments()
2023
{
2124
new DoctrineClearIdentityMapExtension($this->createRegistryMock());

Tests/Unit/Consumption/Extension/DoctrinePingConnectionExtensionTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@
1717

1818
class DoctrinePingConnectionExtensionTest extends TestCase
1919
{
20+
/**
21+
* @doesNotPerformAssertions
22+
*/
2023
public function testCouldBeConstructedWithRequiredAttributes()
2124
{
2225
new DoctrinePingConnectionExtension($this->createRegistryMock());

Tests/Unit/Consumption/Extension/ResetServicesExtensionTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@
1616

1717
class ResetServicesExtensionTest extends TestCase
1818
{
19+
/**
20+
* @doesNotPerformAssertions
21+
*/
1922
public function testCouldBeConstructedWithRequiredArguments()
2023
{
2124
new ResetServicesExtension($this->createResetterMock());

Tests/Unit/DependencyInjection/ConfigurationTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ public function testShouldBeFinal()
2525
$this->assertClassFinal(Configuration::class);
2626
}
2727

28+
/**
29+
* @doesNotPerformAssertions
30+
*/
2831
public function testCouldBeConstructedWithDebugAsArgument()
2932
{
3033
new Configuration(true);

Tests/Unit/DependencyInjection/EnqueueExtensionTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ public function testShouldBeFinal()
3030
$this->assertClassFinal(EnqueueExtension::class);
3131
}
3232

33+
/**
34+
* @doesNotPerformAssertions
35+
*/
3336
public function testCouldBeConstructedWithoutAnyArguments()
3437
{
3538
new EnqueueExtension();

Tests/Unit/EnqueueBundleTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@ public function testShouldExtendBundleClass()
1616
$this->assertClassExtends(Bundle::class, EnqueueBundle::class);
1717
}
1818

19+
/**
20+
* @doesNotPerformAssertions
21+
*/
1922
public function testCouldBeConstructedWithoutAnyArguments()
2023
{
2124
new EnqueueBundle();

Tests/Unit/Profiler/MessageQueueCollectorTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@ public function testShouldExtendDataCollectorClass()
2323
$this->assertClassExtends(DataCollector::class, MessageQueueCollector::class);
2424
}
2525

26+
/**
27+
* @doesNotPerformAssertions
28+
*/
2629
public function testCouldBeConstructedWithEmptyConstructor()
2730
{
2831
new MessageQueueCollector();

0 commit comments

Comments
 (0)