Skip to content

adapt E2E tests to be compatible with the new visual design #6153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

LukasPaczos
Copy link
Contributor

Task/Issue URL: https://app.asana.com/1/137249556945/project/1208671518894266/task/1210387720528713?focus=true

Description

Adapts the Maestro tests to successfully run with or without the new visual design.

Steps to test this PR

QA optional - visual design currently isn't enabled for any build version or flavor.

If you do want to give it a try, you can hardcode https://www.jsonblob.com/api/1376886835833397248 locally and try any of the Maestro tests, especially the ones that interact with the bottom menu/tabs buttons or privacy shield.

Separately, I'll look into adding dedicated tests for the new visual design.

@LukasPaczos
Copy link
Contributor Author

I'm running the E2E test suite on this branch here, just to make sure nothing broke.

@LukasPaczos
Copy link
Contributor Author

Tests are all green, this is ready for review.

@LukasPaczos LukasPaczos requested a review from CDRussell May 29, 2025 07:43
Copy link
Member

@CDRussell CDRussell left a comment

Choose a reason for hiding this comment

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

Nice refactor! I like the conditional checks around hiding the keyboard in the shared flows.

@LukasPaczos LukasPaczos merged commit 2d8ad7b into develop May 29, 2025
8 checks passed
@LukasPaczos LukasPaczos deleted the fix/lukasz-p/make-e2e-tests-compatible-with-new-visual-design branch May 29, 2025 09:14
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.

2 participants