Skip to content

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

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

Add functional tests for the new Symfony module assertions#64
TavoNiievez wants to merge 1 commit into
Codeception:6.4from
TavoNiievez:dominant-assertions-6.4

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, 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#163), 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 force-pushed the dominant-assertions-6.4 branch from b7ea00a to 5fa5a1e Compare September 3, 2026 18:13
@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 70.

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