Skip to content

[9.0] [Security Solution] Enable accidentally skipped prebuilt rules package integration tests (#226759)#227565

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

[9.0] [Security Solution] Enable accidentally skipped prebuilt rules package integration tests (#226759)#227565
maximpn merged 1 commit intoelastic:9.0from
maximpn:backport/9.0/pr-226759

Conversation

@maximpn
Copy link
Copy Markdown
Contributor

@maximpn maximpn commented Jul 11, 2025

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…e integration tests (elastic#226759)

## Summary

This PR enables accidentally skipped prebuilt rules package integration tests.

## Details

The integration tests were accidentally skipped by missing `loadTestFile(require.resolve('./prebuilt_rules_package'));` line in elastic#225495 as the result of merge conflicts.

Enabling those integration tests revealed an issue with the installing all prebuilt rules integration tests. Since the prebuilt rules package contains ML rules all rules installation fails under the low-tier license as ML rules require high-tier license. So this single test was changed to run under high-tier license.

Just for the record, ML related UI issues has been addressed in elastic#224676. It means that users may observe this error under low-tier license only by using the API directly.

(cherry picked from commit 189cfb5)

# Conflicts:
#	x-pack/test/security_solution_api_integration/test_suites/detections_response/rules_management/prebuilt_rules/common/index.ts
@maximpn maximpn requested a review from kibanamachine as a code owner July 11, 2025 07:02
@maximpn maximpn added the backport This PR is a backport of another PR label Jul 11, 2025
@maximpn maximpn enabled auto-merge (squash) July 11, 2025 07:02
@maximpn maximpn merged commit 9198056 into elastic:9.0 Jul 11, 2025
12 checks passed
@maximpn maximpn deleted the backport/9.0/pr-226759 branch July 11, 2025 09:45
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.

2 participants