-
Notifications
You must be signed in to change notification settings - Fork 534
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
VRT: Mock avatars to return stable images #4547
Conversation
|
size-limit report 📦
|
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😂
* just trying something out * should come sooner * test(vrt): update snapshots --------- Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com>
Context:
To make our stories realistic and useful, we use examples of real collaborations in our stories (instead of fake or generated data). This is very nice!
This also means when one of the collaborators updates their avatar, some of stories also update. This is also very nice!
But, this also means that our visual tests can fail because they have the old avatar in snapshots to compare. This is not nice at all! 😓 Most recently reported in #4532 (comment)
Solution:
Added a route mock for our e2e tests (visual tests) that always returns the same image that is safe for visual comparisons. Chose the primer avatar for this. (could have also used a static image, but this feels as good)
Alternatives considered:
Rollout strategy
Merge checklist