Skip to content
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

[8.x] initial @kbn/scout-security plugin (#210433) #215527

Merged
merged 8 commits into from
Mar 23, 2025

Conversation

MadameSheema
Copy link
Member

Backport

This will backport the following commits from main to 8.x:

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
#	.github/CODEOWNERS
#	src/platform/packages/shared/kbn-es/src/stateful_resources/roles.yml
#	x-pack/solutions/security/plugins/security_solution/tsconfig.json
@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 23, 2025 17:18
@MadameSheema MadameSheema merged commit 4d53520 into elastic:8.x Mar 23, 2025
8 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 579 580 +1

Total ESLint disabled count

id before after diff
securitySolution 662 663 +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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants