-
Notifications
You must be signed in to change notification settings - Fork 20
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
test(e2e): homepage screenshot tests #2033
test(e2e): homepage screenshot tests #2033
Conversation
To get the pipeline correctly running, #2032 should be merged first. Though, the PR code can be already reviewed. |
b5331d2
to
edb6b77
Compare
The pipeline is now green, so PR can be fully reviewed. |
8697de6
to
e7fa7e6
Compare
@peterpeterparker @AntonioVentilii-DFINITY thanks for the reviews! I applied all your comments, so the code is ready for another go. |
f5bd7c2
to
4c8c052
Compare
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.
LGTM, thanks
2e27948
to
58e790f
Compare
58e790f
to
af49eff
Compare
Motivation
The goal of this PR is to add basic snapshot tests for homepage (user logged in and logged out states).
Changes
Tests
Please see the committed screenshots.