-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
test: add "cypress-in-cypress" tests for navigating in and out of spec runner #19845
Conversation
Thanks for taking the time to open a PR!
|
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
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.
Great Job.
Do you think we could also check the expanding/collapsing of the left menu?
Definitely. We should be able to check the resizing (currently just component tested), and some settings stuff, now too. |
* 10.0-release: test: add "cypress-in-cypress" tests for navigating in and out of spec runner (#19845) chore: add test cases for migration (#19905) test: fix launchpad flake caused by unoptimized deps (#19903) feat: launchpad browser select (#19830) fix: increase timeout in cy.contains to avoid flake refactor: Remove connection on specs, increase pagination limits elsewhere (#19881) fix(unify): Fixing launchpad project setup tests (#19886) test(unify): Specs Page Workflows - E2E/Component (#19780) feat: migration rename files (#19807) chore: remove design system warnings (#19851)
The recently merged #19835 did not take advantage of Cypress-In-Cypress testing to prevent regressions, adding it now.
Additional details
To verify that these updates would catch the regression, comment out
initialized.value = false
on line 18 of unifiedRunner.tsRun the
cypress-in-cypress-e2e.cy.ts
andcypress-in-cypress-component.cy.ts
app e2e specs and verify the test fornavigation between specs and other parts of the app works
fails as the spec runner does not reopen when the test tries to visit it.Uncomment the
initialized.value = false
and rerun the tests, they should pass now.There is some duplication between the tests and I think for now that's fine. It seems like with Cypress In Cypress we will have several kinds of tests that we want to re-run in both testing modes so some pattern for sharing code between specs will be useful.
PR Tasks
cypress-documentation
?type definitions
?cypress.schema.json
?