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 Solutions] Adds additional cypress tests and utils to value based lists #83026

Merged
merged 3 commits into from
Nov 11, 2020

Conversation

FrankHassanabad
Copy link
Contributor

@FrankHassanabad FrankHassanabad commented Nov 10, 2020

Summary

Adds additional cypress tests and utils around value lists so that the percent of test driven development (TDD) possibilities will be easier for us to accomplish around bug fixes towards the next release.

  • Changes and adds data test subjects within value based lists
  • Fixes a bug where the list fixtures were not being converted to base64 before uploads within Cypress
  • Adds a CIDR fixture
  • Adds tests for export, delete, close and open the dialog, and all the value based list types for the current modal.

Checklist

@FrankHassanabad FrankHassanabad changed the title value lists cypress tests and utils [Security Solutions] Adds additional cypress tests and cypress utils to value based lists Nov 10, 2020
@FrankHassanabad FrankHassanabad changed the title [Security Solutions] Adds additional cypress tests and cypress utils to value based lists [Security Solutions] Adds additional cypress tests and utils to value based lists Nov 10, 2020
@FrankHassanabad FrankHassanabad self-assigned this Nov 10, 2020
@FrankHassanabad FrankHassanabad added Feature:Detection Rules Anything related to Security Solution's Detection Rules 7.11.0 v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Nov 10, 2020
@FrankHassanabad FrankHassanabad marked this pull request as ready for review November 10, 2020 16:42
@FrankHassanabad FrankHassanabad requested review from a team as code owners November 10, 2020 16:42
@FrankHassanabad FrankHassanabad added the Team:Detections and Resp Security Detection Response Team label Nov 10, 2020
Copy link
Contributor

@peluja1012 peluja1012 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the added coverage!

@@ -0,0 +1,2 @@
192.168.100.14/24

Copy link
Contributor

@peluja1012 peluja1012 Nov 10, 2020

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, that's not needed. Thanks, removed it.

@jen-huang jen-huang added v7.11.0 and removed 7.11.0 labels Nov 10, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 7.8MB 7.8MB +74.0B

Distributable file count

id before after diff
default 42772 42774 +2

History

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

@FrankHassanabad FrankHassanabad merged commit 34c80e5 into elastic:master Nov 11, 2020
@FrankHassanabad FrankHassanabad deleted the cypress-tests-lists branch November 11, 2020 05:02
FrankHassanabad added a commit to FrankHassanabad/kibana that referenced this pull request Nov 11, 2020
… based lists (elastic#83026)

## Summary

Adds additional cypress tests and utils around value lists so that the percent of test driven development (TDD) possibilities will be easier for us to accomplish around bug fixes towards the next release.

* Changes and adds data test subjects within value based lists
* Fixes a bug where the list fixtures were not being converted to base64 before uploads within Cypress
* Adds a [CIDR](https://en.wikipedia.org/wiki/Classless_Inter-Domain_Routing) fixture
* Adds tests for export, delete, close and open the dialog, and all the value based list types for the current modal. 

### Checklist

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
FrankHassanabad added a commit that referenced this pull request Nov 11, 2020
… based lists (#83026) (#83143)

## Summary

Adds additional cypress tests and utils around value lists so that the percent of test driven development (TDD) possibilities will be easier for us to accomplish around bug fixes towards the next release.

* Changes and adds data test subjects within value based lists
* Fixes a bug where the list fixtures were not being converted to base64 before uploads within Cypress
* Adds a [CIDR](https://en.wikipedia.org/wiki/Classless_Inter-Domain_Routing) fixture
* Adds tests for export, delete, close and open the dialog, and all the value based list types for the current modal. 

### Checklist

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Detection Rules Anything related to Security Solution's Detection Rules release_note:skip Skip the PR/issue when compiling release notes Team:Detections and Resp Security Detection Response Team v7.11.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants