-
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
[Discover] Fix time zone switch functional test in cloud env #76396
[Discover] Fix time zone switch functional test in cloud env #76396
Conversation
Pinging @elastic/kibana-app (Team:KibanaApp) |
async pressTabKey() { | ||
await browser.pressKeys(browser.keys.TAB); | ||
} | ||
|
||
// Pause the browser at a certain place for debugging | ||
// Not meant for usage in CI, only for dev-usage | ||
async pause() { |
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.
this is now completely off topic, but I was just excited to find a pause function for the functional tests 🥳
@@ -54,6 +54,11 @@ export function QueryBarProvider({ getService, getPageObjects }: FtrProviderCont | |||
}); | |||
} | |||
|
|||
public async clearQuery(): Promise<void> { | |||
await this.setQuery(''); | |||
await PageObjects.common.pressTabKey(); |
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.
moving focus by tab closes the query bar suggestions, also the time picker is then displayed again, which simplifies the code of the failing Discover test
@elasticmachine merge upstream |
…al-test-timezone-switch
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
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
…#76396) # Conflicts: # test/functional/page_objects/common_page.ts
Summary
The overlapping of the query bar with the index pattern selector caused a test fail in cloud. Setting the query bar to '' triggered displaying suggestions, overlapping the index pattern selector. So this PR adds a new function to clear the query, which is followed by a
TAB
to move focus to the next tabbable element.Here's the test run to prove that it no longer fails:
https://internal-ci.elastic.co/job/elastic+estf-cloud-kibana-flaky-test-runner/105/
Fixes #76166
Checklist