-
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] Integration tests for multi line string field diff algorithm #188688
[Security Solution] Integration tests for multi line string field diff algorithm #188688
Conversation
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
⏳ Build in-progress, with failures
Failed CI StepsTest Failures
History
cc @dplumlee |
Flaky Test Runner Stats🎉 All tests passed! - kibana-flaky-test-suite-runner#6582[✅] x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/management/trial_license_complete_tier/configs/ess.config.ts: 100/100 tests passed. |
5360f47
to
daa4110
Compare
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]
To update your PR or re-run it, just comment with: cc @dplumlee |
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.
LGTM! ✅
Summary
Completes #180159
Switches fields to use the multi line string diff algorithm assigned to them in the overarching ticket
Adds integration tests in accordance to #188323 for the
upgrade/_review
API endpoint for the multi line string field diff algorithm.Checklist
Delete any items that are not applicable to this PR.
For maintainers