Skip to content

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

Closed
TavoNiievez wants to merge 1 commit into
Codeception:8.1from
TavoNiievez:dominant-assertions-8.1
Closed

TavoNiievez wants to merge 1 commit into
Codeception:8.1from
TavoNiievez:dominant-assertions-8.1

Conversation

@TavoNiievez

@TavoNiievez TavoNiievez commented Jun 27, 2026 •

Copy link
Copy Markdown
Member

Functional coverage for the new assertions added in codeception/module-symfony (assertSessionHasFlashMessage, the console ExecutionResult assertions, 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 ResultCommand exercising success / failure / invalid / stderr for runCommand() + assertCommand* (Symfony 8.1+ only, hence on this branch);
  • 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.

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

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

Add the matching app fixtures: a flash-and-redirect controller (/flash), a
ResultCommand exercising success / failure / invalid / stderr for
runCommand(), 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.
@TavoNiievez
TavoNiievez force-pushed the dominant-assertions-8.1 branch from d49cae0 to 9dcf04b Compare September 3, 2026 18:12
@TavoNiievez TavoNiievez closed this Sep 3, 2026
@TavoNiievez

Copy link
Copy Markdown
Member Author

Closed and recreated from the same branch (same commit) to clear stale force-push entries in this timeline. The replacement is pull request 69.

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