Skip to content

[SIEM] Fixes flyout flaky test#68683

Merged
MadameSheema merged 2 commits intoelastic:masterfrom
MadameSheema:flyout-button-flaky
Jun 10, 2020
Merged

[SIEM] Fixes flyout flaky test#68683
MadameSheema merged 2 commits intoelastic:masterfrom
MadameSheema:flyout-button-flaky

Conversation

@MadameSheema
Copy link
Copy Markdown
Contributor

Summary

In this PR we are fixing the flyout flaky test.

The test has been executed 30 times in local emulating Jenkins environment and passes all the time.

@MadameSheema MadameSheema added Team:SIEM v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.8.0 labels Jun 9, 2020
@MadameSheema MadameSheema requested review from a team as code owners June 9, 2020 18:15
@MadameSheema MadameSheema self-assigned this Jun 9, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/siem (Team:SIEM)

…flyout_button.spec.ts

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>
Copy link
Copy Markdown
Contributor

@andrew-goldstein andrew-goldstein left a comment

Choose a reason for hiding this comment

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

Thanks @MadameSheema!
The node x-pack/plugins/security_solution/scripts/loop_cypress_tests.js script isn't working for me locally, which is likely an environment issue with my laptop, so I ran the test manually via yarn cypress:open.
LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@MadameSheema MadameSheema merged commit 3eced03 into elastic:master Jun 10, 2020
@MadameSheema MadameSheema deleted the flyout-button-flaky branch June 10, 2020 10:01
MadameSheema added a commit to MadameSheema/kibana that referenced this pull request Jun 10, 2020
* fixes flaky test

* Update x-pack/plugins/security_solution/cypress/integration/timeline_flyout_button.spec.ts

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jun 10, 2020
* master:
  [Ingest Manager] Document fleet configuration flags (elastic#66736)
  [Metrics UI] Fix fatal error when no data is available for Metric Threshold Alerts (elastic#68678)
  [SIEM][Detections Engine] - Exceptions viewer cleanup (elastic#68651)
  [SIEM] Fixes flyout flaky test (elastic#68683)
  [SIEM] Cypress renaming (elastic#68659)
MadameSheema added a commit that referenced this pull request Jun 10, 2020
* fixes flaky test

* Update x-pack/plugins/security_solution/cypress/integration/timeline_flyout_button.spec.ts

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>

Co-authored-by: patrykkopycinski <contact@patrykkopycinski.com>
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Sep 23, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

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

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: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.8.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants