-
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
[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci #196591
Conversation
Pinging @elastic/security-threat-hunting-explore (Team:Threat Hunting:Explore) |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
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.
Thanks! Maybe that something we should make at a higher level? We've seen some issues on the @elastic/security-threat-hunting-investigations side as well for the more complex tests we have!
0b199da
to
cfc2c6c
Compare
…ests timing out on ci addresses elastic#196216
- Refactored test cases to use `data-test-subj` attributes for 10x faster element selection - Added missing `data-test-subj` attributes in components to facilitate the prev point
839fb35
to
9f63b9b
Compare
💚 Build Succeeded
Metrics [docs]Async chunks
History
cc @kapral18 |
Starting backport for target branches: 8.x |
…ests timing out on ci (elastic#196591) addresses elastic#196216 Removing accessibility selectors to ensure 10x speed of tests with data-test-subj locators. (cherry picked from commit b5a705e)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…tial tests timing out on ci (#196591) (#198717) # Backport This will backport the following commits from `main` to `8.x`: - [[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)](#196591) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Karen Grigoryan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-01T18:59:41Z","message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team:Threat Hunting:Explore","backport:prev-minor"],"title":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci","number":196591,"url":"https://github.com/elastic/kibana/pull/196591","mergeCommit":{"message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196591","number":196591,"mergeCommit":{"message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab"}}]}] BACKPORT--> Co-authored-by: Karen Grigoryan <[email protected]>
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ests timing out on ci (elastic#196591) addresses elastic#196216 Removing accessibility selectors to ensure 10x speed of tests with data-test-subj locators. (cherry picked from commit b5a705e)
…ntial tests timing out on ci (#196591) (#200549) # Backport This will backport the following commits from `main` to `8.16`: - [[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)](#196591) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Karen Grigoryan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-01T18:59:41Z","message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team:Threat Hunting:Explore","backport:prev-minor","v8.17.0"],"number":196591,"url":"https://github.com/elastic/kibana/pull/196591","mergeCommit":{"message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196591","number":196591,"mergeCommit":{"message":"[Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591)\n\naddresses #196216\r\n\r\nRemoving accessibility selectors to ensure 10x speed of tests with\r\ndata-test-subj locators.","sha":"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab"}},{"branch":"8.x","label":"v8.17.0","labelRegex":"^v8.17.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/198717","number":198717,"state":"MERGED","mergeCommit":{"sha":"273a1eef0fb2f3c717f9fff8be2097892b8b8b90","message":"[8.x] [Security Solution][Data Quality Dashboard] fix tests and potential tests timing out on ci (#196591) (#198717)\n\n# Backport\n\nThis will backport the following commits from `main` to `8.x`:\n- [[Security Solution][Data Quality Dashboard] fix tests and potential\ntests timing out on ci\n(#196591)](https://github.com/elastic/kibana/pull/196591)\n\n<!--- Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the [Backport tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT [{\"author\":{\"name\":\"Karen\nGrigoryan\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-11-01T18:59:41Z\",\"message\":\"[Security\nSolution][Data Quality Dashboard] fix tests and potential tests timing\nout on ci (#196591)\\n\\naddresses #196216\\r\\n\\r\\nRemoving accessibility\nselectors to ensure 10x speed of tests with\\r\\ndata-test-subj\nlocators.\",\"sha\":\"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"v9.0.0\",\"Team:Threat\nHunting\",\"Team:Threat\nHunting:Explore\",\"backport:prev-minor\"],\"title\":\"[Security\nSolution][Data Quality Dashboard] fix tests and potential tests timing\nout on\nci\",\"number\":196591,\"url\":\"https://github.com/elastic/kibana/pull/196591\",\"mergeCommit\":{\"message\":\"[Security\nSolution][Data Quality Dashboard] fix tests and potential tests timing\nout on ci (#196591)\\n\\naddresses #196216\\r\\n\\r\\nRemoving accessibility\nselectors to ensure 10x speed of tests with\\r\\ndata-test-subj\nlocators.\",\"sha\":\"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/196591\",\"number\":196591,\"mergeCommit\":{\"message\":\"[Security\nSolution][Data Quality Dashboard] fix tests and potential tests timing\nout on ci (#196591)\\n\\naddresses #196216\\r\\n\\r\\nRemoving accessibility\nselectors to ensure 10x speed of tests with\\r\\ndata-test-subj\nlocators.\",\"sha\":\"b5a705e54cd9c657064cc67f8cf1b72f4324d3ab\"}}]}]\nBACKPORT-->\n\nCo-authored-by: Karen Grigoryan <[email protected]>"}}]}] BACKPORT-->
addresses #196216
Removing accessibility selectors to ensure 10x speed of tests with data-test-subj locators.