Skip to content

SE-1917 Replace hacky sleeps with webdriver waits - #275

Merged
samuelallan72 merged 3 commits into
masterfrom
samuel/fix-tests
Apr 21, 2020
Merged

SE-1917 Replace hacky sleeps with webdriver waits#275
samuelallan72 merged 3 commits into
masterfrom
samuel/fix-tests

Conversation

@samuelallan72

@samuelallan72 samuelallan72 commented Apr 21, 2020

Copy link
Copy Markdown
Member

The tests don't appear to be flaky on my machine with python 3 any more, but this at least removes the hacky manual sleeps where possible. Instead it relies on webdriver waiting for changes to happen (element to exist, become visible, etc.). This will be faster and more reliable.

Test instructions:

  • sanity check the code to make sure the test semantics haven't been changed
  • verify that tests in CI reliably pass

Reviewer:

@samuelallan72 samuelallan72 self-assigned this Apr 21, 2020

@pomegranited pomegranited left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 LGTM @swalladge !

  • I tested this by ensuring the automated tests had passed consistently.
  • I read through the code changes, and checked all the places in the tests where sleep is still invoked, to ensure there were no better options available.
  • I checked for accessibility issues test changes only
  • Includes documentation -- adds code comments where sleep was still required.
  • I made sure any change in configuration variables is reflected in the corresponding
    client's configuration-secure repository.
    N/A

@samuelallan72
samuelallan72 merged commit 3badc1c into master Apr 21, 2020
@samuelallan72
samuelallan72 deleted the samuel/fix-tests branch April 21, 2020 09:21
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.

2 participants