-
Notifications
You must be signed in to change notification settings - Fork 8.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
[Security Solution] Fix Cypress scenarios to use internal contex… #86609
[Security Solution] Fix Cypress scenarios to use internal contex… #86609
Conversation
…t instead of raw variables
@elasticmachine merge upstream |
…-context # Conflicts: # x-pack/plugins/security_solution/cypress/integration/alerts.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_detection_rules.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_detection_rules_custom.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_detection_rules_eql.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_detection_rules_export.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_detection_rules_prebuilt.spec.ts # x-pack/plugins/security_solution/cypress/integration/alerts_timeline.spec.ts # x-pack/plugins/security_solution/cypress/integration/cases.spec.ts # x-pack/plugins/security_solution/cypress/integration/timeline_attach_to_case.spec.ts # x-pack/plugins/security_solution/cypress/integration/timeline_creation.spec.ts # x-pack/plugins/security_solution/cypress/integration/timeline_local_storage.spec.ts # x-pack/plugins/security_solution/cypress/integration/timeline_templates_export.spec.ts # x-pack/plugins/security_solution/cypress/integration/timeline_toggle_column.spec.ts # x-pack/plugins/security_solution/cypress/integration/timelines_export.spec.ts # x-pack/plugins/security_solution/cypress/integration/value_lists.spec.ts # x-pack/plugins/security_solution/cypress/screens/timeline.ts # x-pack/plugins/security_solution/cypress/tasks/common.ts # x-pack/plugins/security_solution/cypress/tasks/hosts/events.ts # x-pack/plugins/security_solution/cypress/tasks/timeline.ts
…ibana into chore/cypress-context
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
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.
esArchiver changes LGTM
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededMetrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
* master: [APM] Define placement “Right” to offset tooltip (elastic#87729) Fix UI glitch on SOM delete confirmation modal (elastic#87623) Remove src/plugins/vis_default_editor -> src/plugins/visualizations cyclic dependencies (elastic#86988) [Timelion] Fix tests flakiness on suggestion click (elastic#87273) [Uptime] Fix/details page tabs (elastic#86296) [ML] Fix earliest and latest texts for date fields (elastic#87482) chore(NA): move grokdebugger plugin test fixtures out of __tests__ folder (elastic#87765) [Security Solution] Refactor Cypress scenarios to use internal contex… (elastic#86609) [Security Solution] Unskip cypress tests (elastic#86653)
…t instead of raw variables
Summary
I've addressed 4 issues:
es_archiver
sometimes was taking more than 60s to finish the process, so it timeout out and caused tests to be flakyhttps://github.com/elastic/kibana/pull/86609/files#diff-b5d2a46b3a0ee15acb1ab8a5b3a43f12c7357327d1091e39c9ef54896a036df4R68
Sharing context
instead of raw variableshttps://docs.cypress.io/guides/core-concepts/variables-and-aliases.html#Aliases
https://github.com/elastic/kibana/pull/86609/files#diff-f4573759efd270d01ba5b90d4884561abed6a0f8a0dc014b0ce24dfe44c5361aR123
should()
instead ofthen()
to support retry-ability of the assertionshttps://docs.cypress.io/guides/core-concepts/retry-ability.html#Use-should-with-a-callback
https://github.com/elastic/kibana/pull/86609/files#diff-f4573759efd270d01ba5b90d4884561abed6a0f8a0dc014b0ce24dfe44c5361aR209
Checklist