Skip to content

removed ConfigureAwait(false) from all tests - #3051

Merged
JLoze (JoshLozensky) merged 1 commit into
masterfrom
lozensky/EnableAwaitConfigurationForAllTests
Oct 1, 2024
Merged

removed ConfigureAwait(false) from all tests#3051
JLoze (JoshLozensky) merged 1 commit into
masterfrom
lozensky/EnableAwaitConfigurationForAllTests

Conversation

@JoshLozensky

Copy link
Copy Markdown
Contributor

Per the guidance in the XUnit docs, removing await configurations can cause unexpected behavior as the test framework is not able to control the awaited processes without these configurations. This has led to flaky tests through intermittent race conditions.

@JoshLozensky
JLoze (JoshLozensky) requested a review from a team as a code owner October 1, 2024 03:09
@JoshLozensky
JLoze (JoshLozensky) merged commit 0c7277b into master Oct 1, 2024
@JoshLozensky
JLoze (JoshLozensky) deleted the lozensky/EnableAwaitConfigurationForAllTests branch October 1, 2024 17:01
@JoshLozensky JLoze (JoshLozensky) linked an issue Oct 1, 2024 that may be closed by this pull request
This was referenced Dec 4, 2025
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.

Flaky unit tests

5 participants