Skip to content
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

Feat/suite core/e2e/convert unsupported browsers #16171

Merged

Conversation

Vere-Grey
Copy link
Contributor

Description

Converts browser test to PW
Unfortunately PW does not support firefox - mobile combination and it is not recommended to run it with outdated browsers -> I had to remove 3 tests completely.
Enables again visual regression checks for web. The problematic market tests are disabled anyway and I added instruction how fix their visual comparison once we want them to run again.

Related Issue

Resolve 15606

Screenshots:

@Vere-Grey Vere-Grey requested a review from HajekOndrej January 5, 2025 13:17
@Vere-Grey Vere-Grey self-assigned this Jan 5, 2025
@Vere-Grey Vere-Grey requested a review from vdovhanych as a code owner January 5, 2025 14:36
@Vere-Grey
Copy link
Contributor Author

I will rebase this on the PW upgrade once it is merge and change the failing visual regression check to Aria snapshot

@Vere-Grey Vere-Grey force-pushed the feat/suite-core/e2e/convert-unsupported-browsers branch 2 times, most recently from 8ee0901 to 38ddb48 Compare January 7, 2025 08:46
Unfortunatelly, PW does not support Firefox + Mobile combination
It is also not recommended to run automation on outdated browsers
adds also firefox desktop test
also mask larger element for asset visual regression verification
@Vere-Grey Vere-Grey force-pushed the feat/suite-core/e2e/convert-unsupported-browsers branch from 38ddb48 to 4bdc5ea Compare January 7, 2025 11:18
@HajekOndrej HajekOndrej merged commit 8db814d into develop Jan 7, 2025
31 checks passed
@HajekOndrej HajekOndrej deleted the feat/suite-core/e2e/convert-unsupported-browsers branch January 7, 2025 11:35
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