Skip to content

[Security Solution] Cypress template creation#80180

Merged
angorayc merged 24 commits intoelastic:masterfrom
angorayc:cypress-template-creation
Oct 15, 2020
Merged

[Security Solution] Cypress template creation#80180
angorayc merged 24 commits intoelastic:masterfrom
angorayc:cypress-template-creation

Conversation

@angorayc
Copy link
Copy Markdown
Contributor

@angorayc angorayc commented Oct 12, 2020

Summary

This PR is to fix the failing cypress test for template creation:
#79967
#79389

@angorayc angorayc added release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Oct 13, 2020
@angorayc angorayc marked this pull request as ready for review October 14, 2020 15:11
@angorayc angorayc requested review from a team as code owners October 14, 2020 15:11
@angorayc angorayc added the Team:Threat Hunting Security Solution Threat Hunting Team label Oct 14, 2020
@angorayc
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@MadameSheema MadameSheema left a comment

Choose a reason for hiding this comment

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

I think we cannot delete that methods because if not, we are not properly testing all the things we need.

@angorayc
Copy link
Copy Markdown
Contributor Author

filter
@MadameSheema I found that when adding a new filter to timeline without selecting from the suggestion list, the operation field remains disabled sometimes, so I guess that's why the timeline creation test failed occasionally because of that.
Therefor I'm changing the behaviour of adding the query field here:

cy.get(TIMELINE_FILTER_FIELD).type(`${filter.field}{downarrow}{enter}`);

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

Copy link
Copy Markdown
Contributor

@MadameSheema MadameSheema left a comment

Choose a reason for hiding this comment

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

Lots of thanks for fixing this @angorayc :)

@angorayc angorayc merged commit 8d053fd into elastic:master Oct 15, 2020
angorayc added a commit to angorayc/kibana that referenced this pull request Oct 15, 2020
* init tests

* fix cypress test

* remove console

* fix functional test

* update functional test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
angorayc added a commit to angorayc/kibana that referenced this pull request Oct 15, 2020
* init tests

* fix cypress test

* remove console

* fix functional test

* update functional test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
angorayc added a commit that referenced this pull request Oct 15, 2020
* init tests

* fix cypress test

* remove console

* fix functional test

* update functional test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
angorayc added a commit that referenced this pull request Oct 15, 2020
* init tests

* fix cypress test

* remove console

* fix functional test

* update functional test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Oct 16, 2020
* master: (115 commits)
  [ML] Transforms/DF Analytics: Fix data grid column sorting. (elastic#80618)
  added brace import to vis editor (elastic#80652)
  Fix error rate sorting in services list (elastic#80764)
  Emit info log when using custom registry URL (elastic#80768)
  [Reporting] Config Schema Validation for rules[N].protocol strings (elastic#80766)
  Add Storybook a11y addon (elastic#80069)
  Fix anomaly alert selection text (elastic#80746)
  [Security Solution] [Maps] Kibana index pattern, comma bug fix (elastic#80208)
  [kbn/optimizer] tweak split chunks options (elastic#80444)
  update template to use the new team label (elastic#80748)
  [Security Solution] Fix the Field dropdown in Timeline data providers resets when scrolled (elastic#80718)
  Adjusts observability alerting perms to require "all" (elastic#79896)
  [Security Solutions][Detection Engine] Fixes pre-packaged rules which contain exception lists to not overwrite user defined lists   (elastic#80592)
  [data.ui] Fix flaky test & lazy loading rendering artifacts. (elastic#80612)
  Licensed feature usage for connectors (elastic#77679)
  [Security Solution] Cypress template creation (elastic#80180)
  [APM] Hide service if only data is from ML (elastic#80145)
  Fix role mappings test for ESS (elastic#80604)
  [Maps] Add support for envelope (elastic#80614)
  [Security Solution] Update button text according to status (elastic#80389)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team v7.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants