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

Set up Cypress e2e testing in CI #1677

Merged
merged 4 commits into from
Dec 13, 2023
Merged

Conversation

williamjallen
Copy link
Collaborator

Cypress can now be used for e2e testing. We should begin converting our Selenium and Protractor tests to Cypress since Selenium is becoming outdated, and Protractor has been EOL for some time.

This PR only includes a single example "spec" file. Future PRs will add real test cases and eliminate the example spec file.

I also performed a light refactor of our CTest test structure. It would be a good idea to take a closer look at our CTest setup in the future.

tests/CMakeLists.txt Outdated Show resolved Hide resolved
@williamjallen
Copy link
Collaborator Author

This PR is finally ready for review...

cypress.config.js Outdated Show resolved Hide resolved
tests/CMakeLists.txt Outdated Show resolved Hide resolved
@williamjallen
Copy link
Collaborator Author

Temporarily blocked by regression fixed in #1852.

@williamjallen williamjallen added this pull request to the merge queue Dec 13, 2023
Merged via the queue into Kitware:master with commit 5871f3d Dec 13, 2023
2 checks passed
@williamjallen williamjallen deleted the cypress branch December 13, 2023 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants