Skip to content
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

[8.x] [Security Solution][Investigations] - update list of dependencies to run cypress tests (#198586) #198706

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…run cypress tests (elastic#198586)

## Summary

This PR updates the list of dependencies that need to be watched to
trigger the Cypress tests for the Threat Hunting Investigations team.
The list is unfortunately not exhaustive. I went through all the imports
in the following folders:
- `/packages/kbn-expandable-flyout`
- `/x-pack/plugins/security_solution/public/timelines`
- `/x-pack/plugins/security_solution/public/flyout`
- `/x-pack/plugins/security_solution/public/notes`

I did not go through the
`/x-pack/plugins/security_solution/public/common` as they were way too
many files... Also I did not go through the dependencies of all of our
listed dependencies as this would take days...

It is an improvement on what we've had until now though, and should
protect us from outside code being changed!

elastic/security-team#10579
(cherry picked from commit 79edde9)
@kibanamachine kibanamachine merged commit d0a6118 into elastic:8.x Nov 1, 2024
25 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #4 / DeleteAttachmentConfirmationModal calls onConfirm

Metrics [docs]

✅ unchanged

cc @PhilippeOberti

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants