Skip to content

Commit 28f82bf

Browse files
committed
minor #1568 Update DefaultControllerTest.php (94noni)
This PR was merged into the main branch. Discussion ---------- Update DefaultControllerTest.php I thinks its worth mentioning it, a web test, kernel test etc Commits ------- d389df3 Update DefaultControllerTest.php
2 parents 850536c + d389df3 commit 28f82bf

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

tests/Controller/DefaultControllerTest.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,13 @@ public function testPublicUrls(string $url): void
4444
}
4545

4646
/**
47-
* A good practice for tests is to not use the service container, to make
47+
* A good practice for such tests is to not use the service container, to make
4848
* them more robust. However, in this example we must access to the container
4949
* to get the entity manager and make a database query. The reason is that
5050
* blog post fixtures are randomly generated and there's no guarantee that
5151
* a given blog post slug will be available.
52+
* You can also make some integration test to test services logic.
53+
* See https://symfony.com/doc/current/testing.html#integration-tests
5254
*/
5355
public function testPublicBlogPost(): void
5456
{

0 commit comments

Comments
 (0)