Replace fixed delay with exponential backoff in PlainExecutable_ExtensionMode_SupportedDebugMode_RunsInIde test #12211
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the flaky test
Aspire.Hosting.Tests.Dcp.DcpExecutorTests.PlainExecutable_ExtensionMode_SupportedDebugMode_RunsInIdeby replacing a fixed 2-second delay with exponential backoff polling.Problem
The test was failing intermittently on macOS CI runners because it used a fixed 2-second delay after calling
RunApplicationAsync()before checking if 2 Executables had been created:This approach had multiple issues:
Solution
Replaced the fixed delay with exponential backoff polling using the existing
AsyncTestHelpers.AssertIsTrueRetryAsyncutility:Benefits:
AsyncTestHelpersutility as other tests in the codebaseTesting
Verified that the related test
PlainExecutable_ExtensionMode_UnsupportedDebugMode_RunsInProcessstill passes with the changes.Resolves #[issue-number]
Co-authored-by: @karolz-ms
Original prompt
Fixes #12133
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.