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] Maintenance Window callout: Update tests to match the test plan #156532

Conversation

nikitaindik
Copy link
Contributor

@nikitaindik nikitaindik commented May 3, 2023

Summary

This PR updates Maintenance Window callout tests to match the test plan.

Maintenance Window callout is shown on the Rules Management page when there's a Maintenance Window running.

233624339-9c9b6e3e-9e5e-424d-9d19-9cd7d4e92259

Test plan (internal)

Maintenance Window Callout issue with more info about the feature

@nikitaindik nikitaindik self-assigned this May 3, 2023
@nikitaindik nikitaindik added backport:skip This commit does not require backporting Feature:Rule Management Security Solution Detection Rule Management area bug Fixes for quality problems that affect the customer experience Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detection Rule Management Security Detection Rule Management Team v8.8.0 labels May 3, 2023
@nikitaindik nikitaindik marked this pull request as ready for review May 3, 2023 13:04
@nikitaindik nikitaindik requested a review from a team as a code owner May 3, 2023 13:04
@nikitaindik nikitaindik requested a review from jpdjere May 3, 2023 13:04
@elasticmachine
Copy link
Contributor

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

@nikitaindik nikitaindik added the release_note:skip Skip the PR/issue when compiling release notes label May 3, 2023
Copy link
Contributor

@jpdjere jpdjere left a comment

Choose a reason for hiding this comment

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

Thanks for adding what we discussed in the test plan 👍

LGTM

@nikitaindik nikitaindik force-pushed the maintenance-window-callout-test-updates branch from ffcee30 to bcd290d Compare May 3, 2023 19:47
@nikitaindik nikitaindik changed the title Maintenance Window callout: Update tests to match the test plan [Security Solution] Maintenance Window callout: Update tests to match the test plan May 3, 2023
@nikitaindik nikitaindik enabled auto-merge (squash) May 3, 2023 20:06
@nikitaindik nikitaindik force-pushed the maintenance-window-callout-test-updates branch from bcd290d to ce7d396 Compare May 4, 2023 07:02
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 398 401 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 478 481 +3
total +5

History

  • 💔 Build #125311 failed bcd290d7f0ed4e8d7d4d99e521cc57c3adf642cb
  • 💔 Build #125144 failed ffcee30e97dfc8c5adcfd5286d77736046ffb1e7

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @nikitaindik

@nikitaindik nikitaindik merged commit 508ab65 into elastic:main May 4, 2023
@banderror banderror added the Team:Detections and Resp Security Detection Response Team label May 4, 2023
@elasticmachine
Copy link
Contributor

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

@banderror banderror added test test-coverage issues & PRs for improving code test coverage and removed backport:skip This commit does not require backporting bug Fixes for quality problems that affect the customer experience labels May 4, 2023
@sorenlouv
Copy link
Member

💚 All backports created successfully

Status Branch Result
8.8

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

Questions ?

Please refer to the Backport tool documentation

@nikitaindik
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.8

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

Questions ?

Please refer to the Backport tool documentation

nikitaindik added a commit to nikitaindik/kibana that referenced this pull request May 4, 2023
… the test plan (elastic#156532)

## Summary

This PR updates Maintenance Window callout tests to match the test plan.

Maintenance Window callout is shown on the Rules Management page when
there's a Maintenance Window running.

<img width="1260" alt="233624339-9c9b6e3e-9e5e-424d-9d19-9cd7d4e92259"
src="https://user-images.githubusercontent.com/15949146/235923634-53113720-d58a-4a15-b66b-b2008f80f035.png">

[**Test plan**
(internal)](https://docs.google.com/document/d/1NTTbV7FemZR8LVwWBbDW3Jji58Wy2-n-YGycD698RR8/edit?usp=sharing
)

[**Maintenance Window Callout
issue**](elastic#155099) with more info
about the feature

(cherry picked from commit 508ab65)
nikitaindik added a commit that referenced this pull request May 4, 2023
… match the test plan (#156532) (#156727)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[Security Solution] Maintenance Window callout: Update tests to match
the test plan (#156532)](#156532)

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

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

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"nikita.indik@elastic.co"},"sourceCommit":{"committedDate":"2023-05-04T08:07:42Z","message":"[Security
Solution] Maintenance Window callout: Update tests to match the test
plan (#156532)\n\n## Summary\r\n\r\nThis PR updates Maintenance Window
callout tests to match the test plan.\r\n\r\nMaintenance Window callout
is shown on the Rules Management page when\r\nthere's a Maintenance
Window running.\r\n\r\n<img width=\"1260\"
alt=\"233624339-9c9b6e3e-9e5e-424d-9d19-9cd7d4e92259\"\r\nsrc=\"https://user-images.githubusercontent.com/15949146/235923634-53113720-d58a-4a15-b66b-b2008f80f035.png\">\r\n\r\n[**Test
plan**\r\n(internal)](https://docs.google.com/document/d/1NTTbV7FemZR8LVwWBbDW3Jji58Wy2-n-YGycD698RR8/edit?usp=sharing\r\n)\r\n\r\n[**Maintenance
Window
Callout\r\nissue**](#155099)
with more info\r\nabout the
feature","sha":"508ab6521995eb044b16f6be15839d64de8c39e5","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","test-coverage","Team:Detections
and Resp","Team: SecuritySolution","Feature:Rule
Management","Team:Detection
Rules","v8.8.0","v8.9.0"],"number":156532,"url":"https://github.com/elastic/kibana/pull/156532","mergeCommit":{"message":"[Security
Solution] Maintenance Window callout: Update tests to match the test
plan (#156532)\n\n## Summary\r\n\r\nThis PR updates Maintenance Window
callout tests to match the test plan.\r\n\r\nMaintenance Window callout
is shown on the Rules Management page when\r\nthere's a Maintenance
Window running.\r\n\r\n<img width=\"1260\"
alt=\"233624339-9c9b6e3e-9e5e-424d-9d19-9cd7d4e92259\"\r\nsrc=\"https://user-images.githubusercontent.com/15949146/235923634-53113720-d58a-4a15-b66b-b2008f80f035.png\">\r\n\r\n[**Test
plan**\r\n(internal)](https://docs.google.com/document/d/1NTTbV7FemZR8LVwWBbDW3Jji58Wy2-n-YGycD698RR8/edit?usp=sharing\r\n)\r\n\r\n[**Maintenance
Window
Callout\r\nissue**](#155099)
with more info\r\nabout the
feature","sha":"508ab6521995eb044b16f6be15839d64de8c39e5"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156532","number":156532,"mergeCommit":{"message":"[Security
Solution] Maintenance Window callout: Update tests to match the test
plan (#156532)\n\n## Summary\r\n\r\nThis PR updates Maintenance Window
callout tests to match the test plan.\r\n\r\nMaintenance Window callout
is shown on the Rules Management page when\r\nthere's a Maintenance
Window running.\r\n\r\n<img width=\"1260\"
alt=\"233624339-9c9b6e3e-9e5e-424d-9d19-9cd7d4e92259\"\r\nsrc=\"https://user-images.githubusercontent.com/15949146/235923634-53113720-d58a-4a15-b66b-b2008f80f035.png\">\r\n\r\n[**Test
plan**\r\n(internal)](https://docs.google.com/document/d/1NTTbV7FemZR8LVwWBbDW3Jji58Wy2-n-YGycD698RR8/edit?usp=sharing\r\n)\r\n\r\n[**Maintenance
Window
Callout\r\nissue**](#155099)
with more info\r\nabout the
feature","sha":"508ab6521995eb044b16f6be15839d64de8c39e5"}}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Rule Management Security Solution Detection Rule Management 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 test-coverage issues & PRs for improving code test coverage v8.8.0 v8.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants