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/selenium race conditions #45

Merged
merged 2 commits into from
Jul 13, 2018

Conversation

aarranz
Copy link
Member

@aarranz aarranz commented Jul 13, 2018

Current code has some race conditions on the selenium tests, making those tests randomly fail. This PR request fixes this by removing the use of the implicitly_wait method (directly discouraged by Selenium) and adding the required explicit waits.

@aarranz aarranz requested a review from fdelavega July 13, 2018 10:46
@aarranz aarranz force-pushed the fix/selenium-race-conditions branch from c07b6ac to 887e915 Compare July 13, 2018 10:52
Copy link
Contributor

@fdelavega fdelavega left a comment

Choose a reason for hiding this comment

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

For me it looks good, although we have a random error with an invalid interger we need to look at

@aarranz aarranz force-pushed the fix/selenium-race-conditions branch 3 times, most recently from b224788 to 94db176 Compare July 13, 2018 16:47
@aarranz aarranz force-pushed the fix/selenium-race-conditions branch from 94db176 to fd5c692 Compare July 13, 2018 18:21
@aarranz aarranz merged commit 5e2920f into conwetlab:master Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants