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

Use stricter maxDiffPixelRatio in e2e tests #978

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

streamer45
Copy link
Collaborator

@streamer45 streamer45 commented Feb 24, 2025

Summary

Using a stricter maxDiffPixelRatio value for snapshot matching. To make it happy, we have to remove some of the randomness around usernames and regenerate a few snapshots.

@streamer45 streamer45 changed the title Strict maxDiffPixels Use strictier maxDiffPixelRatio in e2e tests Feb 25, 2025
@streamer45 streamer45 changed the title Use strictier maxDiffPixelRatio in e2e tests Use stricter maxDiffPixelRatio in e2e tests Feb 25, 2025
@streamer45 streamer45 requested a review from cpoile February 25, 2025 20:56
@streamer45 streamer45 self-assigned this Feb 25, 2025
@streamer45 streamer45 added the 2: Dev Review Requires review by a core committer label Feb 25, 2025
Copy link
Member

@cpoile cpoile left a comment

Choose a reason for hiding this comment

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

Hah, clever solution.
If you do that too much though... :)

@cpoile cpoile added 3: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants