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 #8792 chore(nimbus): add wait to review buttons in tests #8829

Merged
merged 1 commit into from
May 15, 2023

Conversation

jaredlockhart
Copy link
Collaborator

@jaredlockhart jaredlockhart commented May 15, 2023

Because

  • We've been seeing some intermittent integration test failures in the remote settings tests
  • The tests were attempting to click the review buttons without waiting for them to be rendered

This commit

  • Adds waits to the review clicks
  • Increase kinto retries
  • Folds overlapping tests together to reduce test run time

Because

* We've been seeing some intermittent integration test failures in the remote settings tests
* The tests were attempting to click the review buttons without waiting for them to be rendered

This commit

* Adds waits to the review clicks
* Increase kinto retries
* Folds overlapping tests together to reduce test run time
Copy link
Contributor

@b4handjr b4handjr left a comment

Choose a reason for hiding this comment

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

More red than green, love it

@jaredlockhart jaredlockhart added this pull request to the merge queue May 15, 2023
Merged via the queue into main with commit 497635c May 15, 2023
@jaredlockhart jaredlockhart deleted the 8792 branch May 15, 2023 21:14
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