-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add cypress and basic commands #449
Conversation
✅ Deploy Preview for cheery-moxie-4f1121 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Image snapshotting was unreliable and broke between different browsers and machines
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.
tACK 64b8c91
nice tests
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.
tACK 15b8e4d aside from some local env timeout issues - but as long as GitHub Actions runs well, the tests should be consistent and thus I'm fine with merging it.
Abstract
Add e2e tests with cypress.
As of right now, I've added a few basic commands like
createWallet
. This will be used to compose more complicated tests.The type of tests that will be added is up to debate, but will most likely include recurring bugs that are too complex to test with unit or integration tests