-
Notifications
You must be signed in to change notification settings - Fork 8k
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
[Detection Engine][FTR] Audit DE exceptions tests #179706
[Detection Engine][FTR] Audit DE exceptions tests #179706
Conversation
@@ -90,7 +90,9 @@ FORMATTED_KB_URL="${KB_URL/https:\/\//}" | |||
# This is used in order to wait for the environment to be ready. | |||
sleep 150 | |||
|
|||
TEST_CLOUD=1 TEST_ES_URL="https://elastic:$PASSWORD@$FORMATTED_ES_URL:443" TEST_KIBANA_URL="https://elastic:$PASSWORD@$FORMATTED_KB_URL:443" yarn run $1 | |||
echo "uername: $USERNAME" |
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.
Was this intended?
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.
Hey, I'm sorry, I meant to open as draft and then I got pulled into something else. I'll clean up right now.
Pinging @elastic/security-detection-engine (Team:Detection Engine) |
|
||
// TODO: When available this tag should be @skipInServerless | ||
// This use case is not relevant to serverless. | ||
describe('@brokenInServerless legacy investigation_fields', () => { |
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.
Moved to create_rule_exceptions_ess
as it's not relevant to serverless.
@@ -1,984 +0,0 @@ | |||
/* | |||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one |
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.
Appears deleted, but just broke this file up into prebuilt_rules.ts
and rule_exceptions_execution.ts
and removed some tests that were redundant.
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: cc @yctercero |
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.
Looks great! 👍
Summary
Continues work on #169185 and #151877 . There wasn't a ton of folder restructure, just switching the tests to run on basic license/essentials tier and some test renaming for clarity sake.
Related to #151877 this PR:
Makes note of any issues in tickets that we will need to follow up on
Related to #169185 this PR:
Things to follow up on
Flakey test runner 🟢 - https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/5585