-
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
Feature/setup cypress for e2e testing #106
Conversation
@iamdharmesh note this is waiting your review |
@10up/open-source-practice Cypress e2e PR up for review |
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.
Hi @barneyjeffries,
Thanks a lot for the Great work here, the code looks good to me.
I noticed that 1 E2E test is failing. Could you please help to check and fix it?
We are good to merge this once we fix the failing e2e test.
Thanks
cc: @jeffpaul
Description of the Change
This PR adds Cypress for e2e testing, configured and with 2 tests set-up:
Closes #65
How to test the Change
npm install
npm run build
npm run env:start
npm run cypress:open
Changelog Entry
Checklist: