-
Notifications
You must be signed in to change notification settings - Fork 9
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
Re-add option for running Cypress in a container #681
Comments
The closest I've gotten on this is to run the following (note that It took me a few tries to get the volume mapping correct since we're using fixtures that are a few levels above our Cypress tests. The next thing to figure out is how to get the app running -- with the command I just mentioned, the tests are failing because of 404s. I was referencing this article: https://www.cypress.io/blog/2019/05/02/run-cypress-with-a-single-docker-command/ |
I created a Docker Compose configuration that's more or less equivalent to the command above. With the app running locally, you can run the Cypress tests and see some passing. I created a WIP branch here: https://github.com/cal-itp/benefits/compare/chore/cypress-container You can try it out by checking out the branch and then, outside the devcontainer, running:
|
I forgot I looked into this a long time ago. Using Cypress in interactive mode in Docker requires X11, which I can almost guarantee is more trouble than the current setup, especially across operating systems. I suggest closing this ticket. |
I think we still want to look into this - not worried so much about getting a complete Cypress graphical interface (requiring X11) - but just being able to run the tests without local NPM madness. Sounds like @angela-tran had a good process working, we'll keep going from there. |
This feels shaped enough, just not sure if we need to do it soon. This opens up a larger question of: what should our Cypress tests be covering? |
We decided this effort isn't worth it right now. We're going to revisit the idea of these types of UI/UX tests in the near future. |
See discussion in #671.
Acceptance Criteria
Additional context
The text was updated successfully, but these errors were encountered: