-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Explore using Cypress #17793
Comments
This investigation can be done as a part of what @esthkim will be working on. |
How would this fit in with the interactive visual diff testing that @danielrozenberg added in #19114? |
I looked briefly into Cypress and saw how it could be really helpful for e2e tests (faster runs, easier to write tests, interactive debugging, etc). The only downside that I could find is that it only works with Chrome, with cross browser support coming later - cypress-io/cypress#310 I'll try to add a small Cypress test in in our Travis builds and see what happens! |
Hey Jon, here's what I found when trying out Cypress:
If it helps, we'll be implementing e2e testing with Selenium soon! Stay tuned. |
Selenium testing sounds like it can work too! We just want some way to write e2e tests for stories 😄 |
Cypress seems like it would provide a lot of awesome integration testing support that we could benefit from in amp-story (and I'm sure others may as well). Can we explore this and see whether it's feasible and/or desirable to use for AMP?
/cc @danielrozenberg @gmajoulet
The text was updated successfully, but these errors were encountered: