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

Fixed async tests sometimes waiting the full timeout even though the … #276

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

chrisdp
Copy link
Collaborator

@chrisdp chrisdp commented Feb 13, 2024

…test group is completed

This issue was cased do to the flag getting flipped to true in the moments between us checking if it was done and when we started waiting for the field to change. I updated the logic to be more tailored to looping only while the value is false. Rather then the value not matching the initial value.

@chrisdp chrisdp added the bug Something isn't working label Feb 13, 2024
@chrisdp chrisdp merged commit f462b5d into master Feb 13, 2024
10 checks passed
@chrisdp chrisdp deleted the bugfix/async-tests-waiting-full-timeout branch February 13, 2024 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants