Skip to content

Commit 9a5e8f9

Browse files
minor symfony#58424 [DoctrineBridge] Fix risky test warnings (alexandre-daubois)
This PR was merged into the 5.4 branch. Discussion ---------- [DoctrineBridge] Fix risky test warnings | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | - | License | MIT PHPUnit raises warning for these tests and consider them risky because they don't perform assertions (which is normal). Commits ------- 41327bd [DoctrineBridge] Fix risky test warnings
2 parents 18ad416 + 41327bd commit 9a5e8f9

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/Symfony/Bridge/Doctrine/Tests/Form/Type/EntityTypePerformanceTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
*/
2525
class EntityTypePerformanceTest extends FormPerformanceTestCase
2626
{
27-
private const ENTITY_CLASS = 'Symfony\Bridge\Doctrine\Tests\Fixtures\SingleIntIdEntity';
27+
private const ENTITY_CLASS = SingleIntIdEntity::class;
2828

2929
/**
3030
* @var \Doctrine\ORM\EntityManager

src/Symfony/Component/Form/Test/FormPerformanceTestCase.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ private function doRunTest()
4545
$this->fail(sprintf('expected running time: <= %s but was: %s', $this->maxRunningTime, $time));
4646
}
4747

48+
$this->expectNotToPerformAssertions();
49+
4850
return $result;
4951
}
5052

0 commit comments

Comments
 (0)