Skip to content

Conversation

damian-molinski
Copy link
Contributor

@damian-molinski damian-molinski commented Aug 19, 2025

Description

Native e2e integration tests maintenance

Related Issue(s)

Closes #3349
Closes #2902

Description of Changes

  • Made e2e tests pass again
  • Disabled all e2e tests as we're waiting for Playwright implementation

Related Pull Requests

emiride and others added 3 commits August 13, 2025 13:28
* test(cat-voices): fix integration tests

* building flutter driver and simplest test

* fix previous commit

* update docs

* export test utils

* prepare for test accounts

* test account

* optional unlocked flag

* app test

* remove warnings

* allow awaiting video and asset cache initialisation

* allow calling bootstrap in SetUp method

* allow awaiting ImagePrecacheService

* app tests

* account cubit safe checks emits

* account tests

* chore: remove debug prints

* rollback timeline change

* simplify tests and fix analyzer

* update UserService tests names

* RegistrationService createAccount masterKey.use

* improve ImagePrecacheService and VideoManager isInitialized completer logic

* chore: missing TODO comment

* chore: TODO about test accounts

* feat: awaiting db operations completion

* cleanup

* rollback timeline change

* remove SemanticsBinding

* chore: formatting

---------

Co-authored-by: Servercic <[email protected]>
Co-authored-by: Damian Moliński <[email protected]>
Co-authored-by: Damian Molinski <[email protected]>
* fund name text

* chore: bring back discovery tests

* Change DateTime with DateTimeExt

* feat: allow mocking CurrentCampaign and mocking DateTime so specific phase

* allow updating dummy

* chore: proposals tests

* onboarding tests

* recovery flow tests

* chore: more discovery tests
@damian-molinski damian-molinski added squad: face Frontend Catalyst Experience Team fyi: qa For information - Quality Assurance team no-track Used to skip tracking in Swarmia analytics, etc. labels Aug 19, 2025
@damian-molinski damian-molinski moved this from New to 🏗 In progress in Catalyst Aug 19, 2025
@damian-molinski damian-molinski moved this from 🏗 In progress to 🔖 Ready in Catalyst Aug 19, 2025
@damian-molinski damian-molinski moved this from 🔖 Ready to 👀 In review in Catalyst Sep 2, 2025
Copy link
Contributor

@kukkok3 kukkok3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we keep this tests until we have the playwright ones ready?

@damian-molinski
Copy link
Contributor Author

Should we keep this tests until we have the playwright ones ready?

@kukkok3 we're not removing them until we're confident with playwright, just added "skip" tag

Copy link
Contributor

github-actions bot commented Sep 2, 2025

📚 Docs Preview

The docs for this PR can be previewed at the following URL:

https://docs.dev.projectcatalyst.io/voices/fix/flutter_integration_tests

Copy link
Contributor

@emiride emiride left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@damian-molinski damian-molinski merged commit ff3d48b into main Sep 4, 2025
49 checks passed
@damian-molinski damian-molinski deleted the fix/flutter_integration_tests branch September 4, 2025 05:58
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Catalyst Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fyi: qa For information - Quality Assurance team no-track Used to skip tracking in Swarmia analytics, etc. squad: face Frontend Catalyst Experience Team
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Disable native flutter integration tests [Feature]: Flutter integration tests maintenance
4 participants