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 control chained test #20642

Merged
merged 7 commits into from
Jul 17, 2018
Merged

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Jul 10, 2018

fixes #20633

The test was failing because some times, clearComboBox did not clear the combo box. This caused the assertion statement to fail. This PR wraps the clear in a retry that will attempt to clear again if the the previous attempts were not successful.

@nreese nreese added test Feature:Input Control Input controls visualization labels Jul 10, 2018
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@nreese
Copy link
Contributor Author

nreese commented Jul 11, 2018

passed first time with 25 runs

jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@timroes
Copy link
Contributor

timroes commented Jul 11, 2018

Jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@timroes
Copy link
Contributor

timroes commented Jul 11, 2018

Jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@timroes
Copy link
Contributor

timroes commented Jul 11, 2018

Jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@timroes
Copy link
Contributor

timroes commented Jul 11, 2018

Jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@nreese
Copy link
Contributor Author

nreese commented Jul 11, 2018

Jenkins, test this

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@nreese nreese requested a review from timroes July 16, 2018 22:36
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Code looks good for me, and it seems test are running rather stable.

@nreese nreese merged commit 2053441 into elastic:master Jul 17, 2018
nreese added a commit to nreese/kibana that referenced this pull request Jul 17, 2018
* add check that combobox was successfully cleared

* run test 25 times

* checkin the correct files this time

* re-enable skipped test

* remove extra test loaders
nreese added a commit that referenced this pull request Jul 17, 2018
* add check that combobox was successfully cleared

* run test 25 times

* checkin the correct files this time

* re-enable skipped test

* remove extra test loaders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Input Control Input controls visualization test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky input control nested test
3 participants