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

[Security Solution] remove old and skipped Threat Intelligence Cypress tests #191166

Conversation

PhilippeOberti
Copy link
Contributor

Summary

This PR is a follow up of this one that migrated all the Threat Intelligence Cypress tests to the security_solution_cypress folder.
It focuses on:

  • removing the old tests, the entire cypress folder under the x-pack/threat_intelligence location
  • updating the CONTRIBUTING and README files

@PhilippeOberti PhilippeOberti changed the title [Security Solution] remove old and skilled Threat Intelligence Cypress tests [Security Solution] remove old and skipped Threat Intelligence Cypress tests Aug 23, 2024
@PhilippeOberti PhilippeOberti marked this pull request as ready for review September 11, 2024 04:13
@PhilippeOberti PhilippeOberti requested review from a team as code owners September 11, 2024 04:13
@PhilippeOberti PhilippeOberti added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.16.0 labels Sep 11, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@PhilippeOberti PhilippeOberti force-pushed the ti-cypess-in-security_solution_cypress-part-2 branch from 45325fd to 858cac1 Compare September 11, 2024 04:14
@PhilippeOberti PhilippeOberti requested a review from a team as a code owner September 11, 2024 04:29
@PhilippeOberti PhilippeOberti force-pushed the ti-cypess-in-security_solution_cypress-part-2 branch from 7c8a5c4 to dd15928 Compare September 11, 2024 14:42
@PhilippeOberti PhilippeOberti force-pushed the ti-cypess-in-security_solution_cypress-part-2 branch from dd15928 to 1805fc7 Compare September 11, 2024 15:12
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack 721 720 -1
threatIntelligence 7 5 -2
total -3

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 745 744 -1
threatIntelligence 7 5 -2
total -3

History

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

@PhilippeOberti PhilippeOberti merged commit 8377275 into elastic:main Sep 12, 2024
41 checks passed
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Sep 12, 2024
@PhilippeOberti PhilippeOberti deleted the ti-cypess-in-security_solution_cypress-part-2 branch September 12, 2024 14:00
gergoabraham pushed a commit to gergoabraham/kibana that referenced this pull request Sep 13, 2024
markov00 pushed a commit to markov00/kibana that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants