test(app): Add end-to-end test automation using Cypress #8081
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ress. Includes a total of 6 test suites and utilizes the Docker emulator.
Corey-ONeal
requested review from
y3rsh,
mcous,
SyntaxColoring,
b-cooper,
sanni-t,
DerekMaggio,
nusrat813 and
reachojhaneha
SyntaxColoring
changed the title
|
|
||
| // Confirm that initial page has loaded properly and all inital content is present | ||
| it('Successfully loads page', () => { | ||
| cy.get('.dZMtmt > .Svg-sc-1lpozsw-0 > path').should('exist') |
There was a problem hiding this comment.
Are these IDs deterministic, or will they change according to the whims of React?
Can/should we change the app source to provide more stable IDs for these tests to hook into?
I'm not familiar with common E2E practice—maybe that wouldn't be worth it, and it's okay to treat this as a snapshot?
| @@ -25,6 +22,8 @@ | |||
| "classnames": "2.2.5", | |||
| "connected-react-router": "6.8.0", | |||
| "core-js": "3.2.1", | |||
| "cypress-file-upload": "^5.0.8", | |||
There was a problem hiding this comment.
I can't figure out why, but a top-level make setup is giving me this yarn warning:
warning " > [email protected]" has incorrect peer dependency "cypress@^3.0.0".
We clearly ask for cypress-file-upload ^5.0.8, so I don't know why it's talking about 3.5.3. 🤷
mcous
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Added end-to-end test automation for the Opentrons run app using Cypress. Contains a total of 6 test suites and utilizes the ability to now open the app in a web browser. Was built around version 4.3.1 of the run app.
To run this manually, please ensure the Docker emulator is up and running, using
make docker-emulatorand already calibrated. You will also need to start the app in "external browser mode" prior to running the test automation usingmake external-browser-app. The app will then be accessible via http:https://localhost:8090.Once that is done, kick off the test automation with
make test-e2e. The test automation can take about 7-8 minutes to fully execute all 6 test suites due to the second test suite taking 6-7 mins to execute 4 different protocols on the emulator.Changelog
Changelog:
Review requests
Those with more experience using GitHub actions, please review the integration with GitHub actions (app-test-build-deploy.yaml) to ensure it is set up correctly.