-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Bump cypress@8.4.1 #104301
Bump cypress@8.4.1 #104301
Conversation
…-7.6.0 # Conflicts: # package.json # packages/kbn-pm/dist/index.js # x-pack/plugins/security_solution/cypress/integration/overview/overview.spec.ts # x-pack/plugins/security_solution/cypress/integration/timelines/notes_tab.spec.ts # x-pack/plugins/security_solution/cypress/integration/timelines/open_timeline.spec.ts # x-pack/plugins/security_solution/cypress/integration/timelines/query_tab.spec.ts # x-pack/plugins/security_solution/cypress/objects/exception.ts # x-pack/plugins/security_solution/cypress/objects/rule.ts # x-pack/plugins/security_solution/cypress/objects/timeline.ts
@elasticmachine merge upstream |
…-7.6.0 # Conflicts: # package.json # packages/kbn-pm/dist/index.js # yarn.lock
…-7.6.0 # Conflicts: # x-pack/plugins/security_solution/cypress/objects/timeline.ts # x-pack/plugins/security_solution/package.json # yarn.lock
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@@ -8,6 +8,8 @@ | |||
"target/**/*" | |||
], | |||
"compilerOptions": { | |||
"target": "es5", | |||
"lib": ["es5", "dom"], |
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.
This is interesting changes, does this give us new abilities such as being able to use ??
where before we couldn't?
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.
From chat this does seem to be the case 👍
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.
LGTM,
- Looked it over, asked a few curiosity questions, nothing major
Thanks for this upgrade. This is going to be a big upgrade for our flakes and tests I'm sure. I bet this was harder than you made it all look, so thanks a million!
💚 Build SucceededMetrics [docs]Public APIs missing exports
History
To update your PR or re-run it, just comment with: |
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Add `?*` to the end of request aliases in instances table tests. #104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM.
Add `?*` to the end of request aliases in instances table tests. elastic#104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM.
Add `?*` to the end of request aliases in instances table tests. #104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM. Co-authored-by: Nathan L Smith <nathan.smith@elastic.co>
Summary
Bumps Cypress to the most recent version https://docs.cypress.io/guides/references/changelog
The biggest changes to notice:
cy.intercept
requires*
suffixstubSearchStrategyApi
was removed in favor of test data (it's tricky to stub search strategy properly)