Skip to content

Comments

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

Merged
maximpn merged 3 commits intoelastic:8.18from
maximpn:backport/8.18/pr-226759
Jul 12, 2025
Merged

[8.18] [Security Solution] Enable accidentally skipped prebuilt rules package integration tests (#226759)#227566
maximpn merged 3 commits intoelastic:8.18from
maximpn:backport/8.18/pr-226759

Conversation

@maximpn
Copy link
Contributor

@maximpn maximpn commented Jul 11, 2025

Backport

This will backport the following commits from main to 8.18:

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:03
@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:03
@maximpn maximpn merged commit 3525f25 into elastic:8.18 Jul 12, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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