Skip to content

[7.12] [Security Solutions] Fixes Cypress tests for indicator match by making the selectors more specific (#91947)#91971

Merged
kibanamachine merged 1 commit intoelastic:7.12from
kibanamachine:backport/7.12/pr-91947
Feb 19, 2021
Merged

[7.12] [Security Solutions] Fixes Cypress tests for indicator match by making the selectors more specific (#91947)#91971
kibanamachine merged 1 commit intoelastic:7.12from
kibanamachine:backport/7.12/pr-91947

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.12:

…g the selectors more specific (elastic#91947)

## Summary

Fixes the indicator match rules cypress e2e tests by making the selectors more specific. Previously other rules and forms code which live on the DOM beside the indicator match rules could interfere when moving around on the DOM. Now with more specific selectors this should be less likely to happen.

If it does happen again I will make the selectors even more specific. 

### Checklist

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
@kibanamachine
Copy link
Copy Markdown
Contributor Author

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @FrankHassanabad

@kibanamachine kibanamachine merged commit b22a59a into elastic:7.12 Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants