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
fix feature tests #1263
fix feature tests #1263
Conversation
95a73f7
to
b377b69
Compare
@@ -51,6 +51,8 @@ CYPRESS_baseUrl=http://localhost:8000 npm run cypress:open | |||
|
|||
See `tests/cypress/package.json` for more cypress scripts. | |||
|
|||
As of Cypress 12.5.1 with Firefox 109, there is a CSRF issue that prevents the tests from passing; unclear if this is a bug in Cypress or what. Use one of the other browser options. |
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.
Tricksy.
@@ -201,14 +202,19 @@ def load_sample_data(app, *args, **kwargs): | |||
|
|||
TransitAgency = app.get_model("core", "TransitAgency") | |||
|
|||
# load the sample data from a JSON file so that it can be accessed by Cypress as well |
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.
See this note. Essentially going back to fixtures, just for the sake of the sample data. Doesn't change anything in Azure.
b377b69
to
96b16da
Compare
96b16da
to
4eba3f9
Compare
Matches the local dev setup more closely, and simplifies the CI setup.
Progress! Three Cypress tests are passing 🎉 |
-p 5000:5000 \ | ||
ghcr.io/cal-itp/eligibility-server:dev | ||
touch .env | ||
docker compose up --detach client server |
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.
Was able to simplify, and more closely match our local setup.
This reverts commit 35f90a2.
Fixes #340.
Gets the feature tests working again, and sets them to run on every
push
in CI.