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

[Cases] Fix export flaky test #112456

Merged

Conversation

jonathan-buttner
Copy link
Contributor

@jonathan-buttner jonathan-buttner commented Sep 16, 2021

Issues: #112207, #112353

This PR fixes some flaky tests for exporting cases. The issue was likely that the export route is returning the objects in different order that we were originally expecting. The solution is to search for the types of objects and then perform the checks on them.

I ran it through the flaky test runner here and it passed 👍
https://kibana-ci.elastic.co/job/kibana+flaky-test-suite-runner/1903/

@jonathan-buttner jonathan-buttner marked this pull request as ready for review September 16, 2021 21:39
@jonathan-buttner jonathan-buttner requested a review from a team as a code owner September 16, 2021 21:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@jonathan-buttner
Copy link
Contributor Author

@elasticmachine merge upstream

@jonathan-buttner
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@jonathan-buttner jonathan-buttner merged commit 69dd0ee into elastic:master Sep 20, 2021
@jonathan-buttner jonathan-buttner deleted the cases-fix-export-flaky-test branch September 20, 2021 16:46
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Sep 20, 2021
* Refactoring flaky test

* Forcing error in ci

* Removing fast fail

* remove .only

Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Sep 20, 2021
* Refactoring flaky test

* Forcing error in ci

* Removing fast fail

* remove .only

Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Jonathan Buttner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment