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

[Files management] Fix flaky functional test #162146

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

sebelga
Copy link
Contributor

@sebelga sebelga commented Jul 18, 2023

There is still some flakiness in the File management test. Probably due to the 300ms delay to render the Empty prompt.

I updated the test to be more resilient.

Fixes #160178

@sebelga sebelga marked this pull request as ready for review July 18, 2023 13:45
@sebelga sebelga self-assigned this Jul 18, 2023
@sebelga sebelga added the Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) label Jul 18, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@sebelga sebelga added the test label Jul 18, 2023
@sebelga sebelga requested a review from Dosant July 18, 2023 13:46
@sebelga sebelga enabled auto-merge (squash) July 18, 2023 13:47
@sebelga sebelga added the release_note:skip Skip the PR/issue when compiling release notes label Jul 18, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

cc @sebelga

@sebelga sebelga merged commit fde21b1 into elastic:main Jul 18, 2023
@kibanamachine kibanamachine added v8.10.0 backport:skip This commit does not require backporting labels Jul 18, 2023
@sebelga sebelga deleted the fix-flaky-test branch July 18, 2023 14:41
@sebelga sebelga added backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Jul 20, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 20, 2023
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.9

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jul 20, 2023
# Backport

This will backport the following commits from `main` to `8.9`:
- [[Files management] Fix flaky functional test
(#162146)](#162146)

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

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

<!--BACKPORT [{"author":{"name":"Sébastien
Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2023-07-18T14:40:51Z","message":"[Files
management] Fix flaky functional test
(#162146)","sha":"fde21b15de69831e5b2b9f3df919b9d6d13ccebd","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["test","release_note:skip","Team:SharedUX","backport:prev-minor","v8.10.0"],"number":162146,"url":"https://github.com/elastic/kibana/pull/162146","mergeCommit":{"message":"[Files
management] Fix flaky functional test
(#162146)","sha":"fde21b15de69831e5b2b9f3df919b9d6d13ccebd"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/162146","number":162146,"mergeCommit":{"message":"[Files
management] Fix flaky functional test
(#162146)","sha":"fde21b15de69831e5b2b9f3df919b9d6d13ccebd"}}]}]
BACKPORT-->

Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
@mistic
Copy link
Member

mistic commented Jul 25, 2023

This PR didn't make it into the latest BC of 8.9.0. Updating the labels.

@mistic mistic added v8.9.1 and removed v8.9.0 labels Jul 25, 2023
ThomThomson pushed a commit to ThomThomson/kibana that referenced this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) test v8.9.1 v8.10.0
Projects
None yet
6 participants