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

[Security Solution] Test plan for prebuilt rule flyout #167727

Merged
merged 5 commits into from
Oct 17, 2023

Conversation

nikitaindik
Copy link
Contributor

@nikitaindik nikitaindik commented Oct 2, 2023

Resolves: #166160

Summary

Adds a test plan for the prebuilt rules flyout. Covers installation and upgrade actions and also the content shown in Overview and Investigation Guide tabs.

Scherm­afbeelding 2023-10-03 om 22 29 39

@nikitaindik nikitaindik force-pushed the rule-preview-test-scenarios branch from 4ec19fd to 2aaeca0 Compare October 3, 2023 17:02
@nikitaindik nikitaindik added release_note:skip Skip the PR/issue when compiling release notes test-plan Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detection Rule Management Security Detection Rule Management Team Feature:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area v8.11.0 labels Oct 3, 2023
@nikitaindik nikitaindik marked this pull request as ready for review October 3, 2023 20:30
@nikitaindik nikitaindik requested a review from a team as a code owner October 3, 2023 20:30
@nikitaindik nikitaindik requested a review from xcrzx October 3, 2023 20:30
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@nikitaindik nikitaindik requested review from banderror and removed request for xcrzx October 3, 2023 20:30
@banderror banderror force-pushed the rule-preview-test-scenarios branch from c40f3ed to 9aaae8a Compare October 5, 2023 10:09
Copy link
Contributor

@banderror banderror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the plan and made some small adjustments to the scenarios w/o changing what they actually do and check.

One outstanding comment is that we should probably add some scenarios for hiding tabs and sections when there are no properties to show in them.

Other than that LGTM 👍

Copy link
Contributor

@banderror banderror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed together with @nikitaindik, LGTM

@banderror banderror merged commit 930b440 into elastic:main Oct 17, 2023
2 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 17, 2023
**Resolves: elastic#166160

## Summary

Adds a test plan for the prebuilt rules flyout. Covers installation and
upgrade actions and also the content shown in Overview and Investigation
Guide tabs.

<img width="1301" alt="Scherm­afbeelding 2023-10-03 om 22 29 39"
src="https://github.com/elastic/kibana/assets/15949146/1aa7aa8b-3746-4767-82ad-f4910034d5eb">

---------

Co-authored-by: Georgii Gorbachev <[email protected]>
(cherry picked from commit 930b440)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.11

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 17, 2023
… (#169155)

# Backport

This will backport the following commits from `main` to `8.11`:
- [[Security Solution] Test plan for prebuilt rule flyout
(#167727)](#167727)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-10-17T17:44:13Z","message":"[Security
Solution] Test plan for prebuilt rule flyout (#167727)\n\n**Resolves:
https://github.com/elastic/kibana/issues/166160**\r\n\r\n##
Summary\r\n\r\nAdds a test plan for the prebuilt rules flyout. Covers
installation and\r\nupgrade actions and also the content shown in
Overview and Investigation\r\nGuide tabs.\r\n\r\n<img width=\"1301\"
alt=\"Scherm­afbeelding 2023-10-03 om 22 29
39\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/1aa7aa8b-3746-4767-82ad-f4910034d5eb\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"930b440c9b91a99a3b3398d5d4941d67c88d9128","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","test-plan","Team:Detections
and Resp","Team: SecuritySolution","Team:Detection Rule
Management","Feature:Prebuilt Detection
Rules","v8.11.0","v8.12.0"],"number":167727,"url":"https://github.com/elastic/kibana/pull/167727","mergeCommit":{"message":"[Security
Solution] Test plan for prebuilt rule flyout (#167727)\n\n**Resolves:
https://github.com/elastic/kibana/issues/166160**\r\n\r\n##
Summary\r\n\r\nAdds a test plan for the prebuilt rules flyout. Covers
installation and\r\nupgrade actions and also the content shown in
Overview and Investigation\r\nGuide tabs.\r\n\r\n<img width=\"1301\"
alt=\"Scherm­afbeelding 2023-10-03 om 22 29
39\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/1aa7aa8b-3746-4767-82ad-f4910034d5eb\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"930b440c9b91a99a3b3398d5d4941d67c88d9128"}},"sourceBranch":"main","suggestedTargetBranches":["8.11"],"targetPullRequestStates":[{"branch":"8.11","label":"v8.11.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/167727","number":167727,"mergeCommit":{"message":"[Security
Solution] Test plan for prebuilt rule flyout (#167727)\n\n**Resolves:
https://github.com/elastic/kibana/issues/166160**\r\n\r\n##
Summary\r\n\r\nAdds a test plan for the prebuilt rules flyout. Covers
installation and\r\nupgrade actions and also the content shown in
Overview and Investigation\r\nGuide tabs.\r\n\r\n<img width=\"1301\"
alt=\"Scherm­afbeelding 2023-10-03 om 22 29
39\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/1aa7aa8b-3746-4767-82ad-f4910034d5eb\">\r\n\r\n---------\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"930b440c9b91a99a3b3398d5d4941d67c88d9128"}}]}]
BACKPORT-->

Co-authored-by: Nikita Indik <[email protected]>
hop-dev pushed a commit to hop-dev/kibana that referenced this pull request Oct 18, 2023
**Resolves: elastic#166160

## Summary

Adds a test plan for the prebuilt rules flyout. Covers installation and
upgrade actions and also the content shown in Overview and Investigation
Guide tabs.

<img width="1301" alt="Scherm­afbeelding 2023-10-03 om 22 29 39"
src="https://github.com/elastic/kibana/assets/15949146/1aa7aa8b-3746-4767-82ad-f4910034d5eb">

---------

Co-authored-by: Georgii Gorbachev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area release_note:skip Skip the PR/issue when compiling release notes Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. test-plan v8.11.0 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Security Solution] Write a test plan for the Overview and Investigation Guide tabs of prebuilt rule flyout
4 participants