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

Initial integration testing configuration #122

Merged
merged 13 commits into from
Oct 4, 2021
Merged

Initial integration testing configuration #122

merged 13 commits into from
Oct 4, 2021

Conversation

thekaveman
Copy link
Member

@thekaveman thekaveman commented Sep 30, 2021

Initial configuration for automated end-to-end integration testing using cypress.

Running this PR

Using Docker Compose

Run the tests with Docker Compose against the client service:

First ensure your .env file has an updated CYPRESS_baseUrl variable:

# using the Docker Compose service address
CYPRESS_baseURL=http://client:8000

Then from within the localhost directory:

docker compose run tests-e2e

From the Dev Container

cypress is installed and available to run directly in the devcontainer.

First ensure your .env file has an updated CYPRESS_baseUrl variable:

# using localhost since we're inside the container
CYPRESS_baseURL=http://localhost:8000

Then Rebuild and Reopen, and start the benefits app with F5. Then from within the tests/e2e directory:

npx cypress run

@thekaveman thekaveman force-pushed the tests-e2e branch 2 times, most recently from 5a50e0f to 49066dc Compare September 30, 2021 19:48
@thekaveman
Copy link
Member Author

Rebased on dev for a cleaner PR.

@thekaveman thekaveman added this to the October 2021 milestone Sep 30, 2021
@thekaveman thekaveman marked this pull request as ready for review October 1, 2021 15:14
@thekaveman
Copy link
Member Author

Rebased on latest dev

@machikoyasuda
Copy link
Member

Woot!!!!!

I got it working on docker compose. This is what that test run output looks like:
image

And then I got it running in the dev container:
image

@thekaveman thekaveman merged commit eb1fa17 into dev Oct 4, 2021
@thekaveman thekaveman deleted the tests-e2e branch October 4, 2021 19:36
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

2 participants