Skip to content

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

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

Add functional tests for the new Symfony module assertions#65
TavoNiievez wants to merge 1 commit into
Codeception:5.4from
TavoNiievez:dominant-assertions-5.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() and
seeNumQueriesIsLessThan()/dontSeeDuplicateQueries().

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 Messenger transport assertions need symfony/messenger 6.3 and the
console ExecutionResult assertions need symfony/console 8.1, so on 5.4 the
transport assertions are covered by asserting that they report their version
requirement, the same way BrowserCest covers the browser-history assertions.
@TavoNiievez
TavoNiievez force-pushed the dominant-assertions-5.4 branch from 6a35129 to ab02857 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 71.

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