Skip to content

Add functional tests for the new Symfony module assertions - #68

Merged
TavoNiievez merged 1 commit into
Codeception:7.4from
TavoNiievez:dominant-assertions-7.4
Sep 3, 2026
Merged

Add functional tests for the new Symfony module assertions#68
TavoNiievez merged 1 commit into
Codeception:7.4from
TavoNiievez:dominant-assertions-7.4

Conversation

@TavoNiievez

@TavoNiievez TavoNiievez commented Sep 3, 2026

Copy link
Copy Markdown
Member

Functional coverage for the new assertions added in codeception/module-symfony (assertSessionHasFlashMessage, mockService/unmockService, seeNumQueriesIsLessThan/dontSeeDuplicateQueries, and the Messenger in-memory transport assertions).

Adds the matching app fixtures:

  • a flash-and-redirect controller (/flash) for assertSessionHasFlashMessage;
  • a Greeting service + /greeting page swapped via mockService;
  • a query-heavy /run-queries page for the Doctrine query-count assertions;
  • an in-memory async transport with SendWelcomeMessage routed to it for the Messenger transport assertions.

The console ExecutionResult assertions are Symfony 8.1+ only, so they are covered on the 8.1 branch instead of here.

This PR exercises code that is not released yet (module PR TavoNiievez/module-symfony#185), so it is expected to stay red until that change is merged and tagged.

Cover the assertions added in codeception/module-symfony:
assertSessionHasFlashMessage(), mockService(),
seeNumQueriesIsLessThan()/dontSeeDuplicateQueries() and the Messenger
in-memory transport assertions.

Add the matching app fixtures: a flash-and-redirect controller (/flash), a
Greeting service behind /greeting to swap with mockService(), a query-heavy
/run-queries page for the Doctrine query-count assertions, and an in-memory
async transport with SendWelcomeMessage routed to it.

The console ExecutionResult assertions need symfony/console 8.1, so they are
covered on the 8.1 branch only.
@TavoNiievez
TavoNiievez merged commit f528508 into Codeception:7.4 Sep 3, 2026
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant