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

Mtbls 738 cypress first steps #154

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

calstevemart
Copy link
Contributor

@calstevemart calstevemart commented Nov 21, 2024

this was a 'spike' intended to understand the basic concepts of Cypress and its use.
Some thoughts:

  • Running cypress against the docker-compose localhost unreliable at this point in time. Auth requests hang and fail on the very first attempt (but not subsequently), sometimes network requests take too long and cause cypress timeout errors.
  • Running cypress in CI/CD we should use www-test. This would also work for local editor changes - configure the editor to point towards www-test, and have cypress pointed at your local editor
  • e2e tests intended to test features end to end rather than the whole application
  • never the less I think there would be some benefit to creating a large test spec for a 'happy path' study creation and population, as a kind of automated sanity test.

There are a lot of boilerplate cypress tests and fixtures included in this PR. These were very useful during this initial phase and I think we can get rid of them once we have a healthy base of test specs of our own.

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.

1 participant