move mouse to close obstructing tooltip#59214
Conversation
LeeDr
added
test_ui_functional
v8.0.0
release_note:skip
|
Pinging @elastic/kibana-app (Team:KibanaApp) |
|
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
| } | ||
|
|
||
| async clickLoadSavedSearchButton() { | ||
| await testSubjects.moveMouseTo('discoverOpenButton'); |
There was a problem hiding this comment.
We used to have this internally but looks like it was commented out:
There was a problem hiding this comment.
I commented it out during Leadfoot -> Webdriver migration, since many tests were failing on moveMouseTo.
The plan was to find a reason for that issue, but the easiest way was to keep it commented out.
Now I think we need to remove it to avoid confusion.
There was a problem hiding this comment.
I saw people adding moveMouseTo in some test/method explicitly and it looks reasonable and easy to follow.
There was a problem hiding this comment.
@stacey-gammon would you like me to remove the commented out moveMouseTo line in this PR?
There was a problem hiding this comment.
up to you, just was pointing it out, changes LGTM!
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
…re/files-and-filetree * 'master' of github.com:elastic/kibana: (254 commits) Convert discover_page to ts, remove redundunt methods (elastic#59312) [Fix for Vis Editor] Revert setting time field to empty string when it's undefined (elastic#58873) Delete legacy search endpoint (elastic#59341) [Uptime] Improve duration chart (elastic#58404) [Snapshot & Restore] NP migration (elastic#59109) [ML] Add support for date_nanos time field in anomaly job wizard (elastic#59017) Revert "Makes alerting and actions optional properties for interface RequestH… (elastic#59264)" Change remote_clusters ID to remoteClusters (elastic#59246) Makes alerting and actions optional properties for interface RequestH… (elastic#59264) Clean up date histogram agg type. (elastic#58805) [ML] Management: fix license unsubscribe (elastic#59365) Remove documentation for server.cors settings (elastic#59096) Edit alert flyout (elastic#58964) [SIEM] Fix rule delete/duplicate actions (elastic#59306) move mouse to close obstructing tooltip (elastic#59214) Reset page after deleting (elastic#59310) Make sure phrases input filter triggers autosuggestons (elastic#59299) Add loading count source for http requests (elastic#59245) Revert "[ML] Transforms: Deprecate custom KibanaContext. (elastic#59133)" Expose metrics service to public API (elastic#59294) ... # Conflicts: # src/plugins/console/public/application/containers/editor/legacy/console_editor/editor.tsx
5 participants
Summary
This is another PR to fix a test "visualize app visualize lab mode disabling does not break loading saved searches" which passes HEADLESS (like in Jenkins) but fails reliably for me when run locally with a visible browser. The problem in this case is a tooltip obstructing the use of the Discover Open menu item.
The fix, only implemented on the one discover_page method that is breaking this test, is to move the mouse to that menu item before clicking it.
The failure looks like this;
Checklist
Delete any items that are not applicable to this PR.