Skip to content

[9.0] [Security Solution] Implement prebuilt rule upgrade notifications test plan (#225912)#229585

Merged
maximpn merged 1 commit intoelastic:9.0from
maximpn:backport/9.0/pr-225912
Jul 28, 2025
Merged

[9.0] [Security Solution] Implement prebuilt rule upgrade notifications test plan (#225912)#229585
maximpn merged 1 commit intoelastic:9.0from
maximpn:backport/9.0/pr-225912

Conversation

@maximpn
Copy link
Copy Markdown
Contributor

@maximpn maximpn commented Jul 28, 2025

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…t plan (elastic#225912)

**Addresses:** elastic#202078

## Summary

This PR implements the [Prebuilt Rules Upgrade Notifications test plan](https://github.com/elastic/kibana/blob/main/x-pack/solutions/security/plugins/security_solution/docs/testing/test_plans/detection_response/prebuilt_rules/prebuilt_rule_upgrade_notifications.md).

## Details

This PR implements test scenarios listed in the [Prebuilt Rules Upgrade Notifications test plan](https://github.com/elastic/kibana/blob/main/x-pack/solutions/security/plugins/security_solution/docs/testing/test_plans/detection_response/prebuilt_rules/prebuilt_rule_upgrade_notifications.md). Unit tests listed in the test plan was skipped due to low testability of the UI code, high maintenance effort and low total value since the same scenarios are covered by the e2e tests.

## Flaky test runner

- ✅ [added integration tests (200 runs)](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/8656)
- ✅ [added e2e tests (75 runs)](https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/8664) Since the whole Rule Management Cypress tests group was ran you may observe some failures in unrelated to this PR tests.

(cherry picked from commit 92f07aa)

# Conflicts:
#	x-pack/solutions/security/plugins/security_solution/public/detection_engine/rule_management_ui/components/mini_callout/mini_callout.tsx
@maximpn maximpn requested a review from kibanamachine as a code owner July 28, 2025 09:55
@maximpn maximpn added the backport This PR is a backport of another PR label Jul 28, 2025
@maximpn maximpn enabled auto-merge (squash) July 28, 2025 09:56
@maximpn maximpn merged commit 74984f4 into elastic:9.0 Jul 28, 2025
12 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #16 / CustomFields renders correctly

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 8.9MB 8.9MB +6.0B

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.

3 participants