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

Add cypress e2e test to confirm PRs don't break app #39

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

patcon
Copy link

@patcon patcon commented Jul 3, 2022

Includes commits from #38

Cypress is an end-to-end testing tool. It can run in CI environments like GitHub Actions.

This PR adds:

  • cypress boilerplate to minimally allow it to work
  • a single e2e test for running through the form, including confirming validation works (it does not test file upload at the moment)
  • a github action workflow to build the app, serve it from a one-off docker container, and run e2e test against it
  • the workflow also saves the video captures that cypress makes for each test run, and preserves them as build artifacts on github (example at bottom of build page)

This PR also includes fixes from #37, so that the app works to test.

Hopefully this helps you manage PRs more easily :) Thanks again for this resource!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant