-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Conversation
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'); |
There was a problem hiding this comment.
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.
…-ref HEAD~1..HEAD --fix'
@elasticmachine merge upstream |
Pinging @elastic/kibana-gis (Team:Geo) |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
code review
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
* [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)
💚 All backports created successfully
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 |
* [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>
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