Skip to content

[9.0] initial @kbn/scout-security plugin (#210433)#215528

Merged
MadameSheema merged 4 commits intoelastic:9.0from
MadameSheema:backport/9.0/pr-210433
Mar 21, 2025
Merged

[9.0] initial @kbn/scout-security plugin (#210433)#215528
MadameSheema merged 4 commits intoelastic:9.0from
MadameSheema:backport/9.0/pr-210433

Conversation

@MadameSheema
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

## Summary

Closes elastic#210298

In this PR we are adding the initial structure for the
@kibana/scout-security package, note that this is not ready to be used
and any new test using this package, is not going to be executed as part
of the regular pipelines, meaning, you are not going to add coverage to
the application.


@kibana/scout-security package is a test package that extends @kbn/scout
with test helpers specifically designed to test Security Solution
functionalities in Kibana. All tests under Security plugins should only
import from @kbn/scout-security, not from @kbn/scout.



This PR is a POC to start testing development by providing custom
Playwright fixtures, page objects, and utilities tailored for
Security-related testing scenarios.

Things to follow-up:



- CustomQueryRule interface is already declared in
`x-pack/solutions/security/plugins/security_solution/common/api/detection_engine/model/rule_schema/rule_schemas.gen.ts`
as `QueryRuleCreateProps`

- DETECTION_ENGINE_RULES_URL and DETECTION_ENGINE_RULES_BULK_ACTION are
already declared in `@kbn/security-solution-plugin/common/constants`

It would be great if all of that is extracted from the plugin to a
package so it can be reused instead of having to duplicate the code.

Until the package is not ready to be used and has not been introduced to
the different teams, appex-qa and myself will be the owners of it to
make sure that best practices are followed

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Dzmitry Lemechko <dzmitry.lemechko@elastic.co>
(cherry picked from commit d5a0501)

# Conflicts:
#	.buildkite/scout_ci_config.yml
@MadameSheema MadameSheema added the backport This PR is a backport of another PR label Mar 21, 2025
@MadameSheema MadameSheema enabled auto-merge (squash) March 21, 2025 15:44
@MadameSheema MadameSheema requested review from csr and dmlemeshko March 21, 2025 19:10
@MadameSheema MadameSheema disabled auto-merge March 21, 2025 19:39
@MadameSheema MadameSheema enabled auto-merge (squash) March 21, 2025 20:17
@MadameSheema MadameSheema merged commit 4695a78 into elastic:9.0 Mar 21, 2025
9 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/scout 113 119 +6
@kbn/scout-security - 66 +66
total +72

Any counts in public APIs

Total count of every any typed public API. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats any for more detailed information.

id before after diff
@kbn/scout-security - 2 +2

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
@kbn/scout 17 15 -2
@kbn/scout-security - 6 +6
total +4
Unknown metric groups

API count

id before after diff
@kbn/scout 424 441 +17
@kbn/scout-security - 356 +356
total +373

ESLint disabled line counts

id before after diff
securitySolution 576 577 +1

Total ESLint disabled count

id before after diff
securitySolution 659 660 +1

Unreferenced deprecated APIs

id before after diff
@kbn/scout-security - 18 +18

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