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

[Maps] unskip 'embed in dashboard' functional test #141289

Merged
merged 5 commits into from
Sep 23, 2022

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Sep 21, 2022

Fixes #140241

Test was flaky because setting index pattern and adding the filter were broken into 2 separate calls. This PR combines setting index pattern and creating filter into a single call to avoid flaky interactions with opening/closing add filter popover.

flaky test runner https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/1282

await filterBar.selectIndexPattern('meta_for_geo_shapes*');
await filterBar.addFilter('shape_name', 'is', 'alpha'); // runtime fields do not have autocomplete
});
await filterBar.addFilterAndSelectDataView('meta_for_geo_shapes*', 'shape_name', 'is', 'alpha');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Able to remove retry because search bar redesign has moved add filter button so its no longer next to newly created filter pills.

@nreese
Copy link
Contributor Author

nreese commented Sep 21, 2022

@elasticmachine merge upstream

@nreese nreese marked this pull request as ready for review September 21, 2022 20:22
@nreese nreese requested a review from a team as a code owner September 21, 2022 20:22
@nreese nreese added [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:skip Skip the PR/issue when compiling release notes auto-backport Deprecated - use backport:version if exact versions are needed v8.5.0 v8.6.0 labels Sep 21, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@nreese
Copy link
Contributor Author

nreese commented Sep 22, 2022

@elasticmachine merge upstream

Copy link
Member

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm!

code review

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@nreese nreese merged commit d23aa31 into elastic:main Sep 23, 2022
kibanamachine pushed a commit that referenced this pull request Sep 23, 2022
* [Maps] unskip 'embed in dashboard' functional test

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Nick Peihl <nick.peihl@elastic.co>
(cherry picked from commit d23aa31)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.5

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

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

kibanamachine added a commit that referenced this pull request Sep 23, 2022
* [Maps] unskip 'embed in dashboard' functional test

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Nick Peihl <nick.peihl@elastic.co>
(cherry picked from commit d23aa31)

Co-authored-by: Nathan Reese <reese.nathan@elastic.co>
@sophiec20 sophiec20 added Feature:Maps and removed [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation labels Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Maps release_note:skip Skip the PR/issue when compiling release notes v8.5.0 v8.6.0
Projects
None yet
6 participants