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

fix flaky input controls functional test #31970

Merged
merged 1 commit into from
Feb 26, 2019

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Feb 25, 2019

fixes #31548

For some unknown reason, clicking an EUI combo box does not always open the menu options. The old retry logic did not include the click so once the menu did not open, the functional test would get stuck.

This PR updates the retry logic to include clicking the combo box if the first click failed to open the menu.

Below is a debug output of a test run that required a retry. The first click failed to open the menu but the retry got it on the second attempt

screen shot 2019-02-25 at 3 15 01 pm

@nreese nreese added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Feb 25, 2019
@nreese nreese requested a review from spalger February 25, 2019 22:19
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@nreese nreese merged commit 946b782 into elastic:master Feb 26, 2019
@nreese nreese added the v6.6.0 label Feb 26, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 26, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 26, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 26, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 26, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 26, 2019
@nreese nreese added the non-issue Indicates to automation that a pull request should not appear in the release notes label Mar 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
non-issue Indicates to automation that a pull request should not appear in the release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v6.5.0 v6.6.0 v6.7.0 v7.0.0 v7.2.0 v8.0.0
Projects
None yet
3 participants