Skip to content

Test: fix flaky test#957

Merged
dtivel merged 1 commit intomainfrom
dtivel/fix-flaky-test
Dec 2, 2025
Merged

Test: fix flaky test#957
dtivel merged 1 commit intomainfrom
dtivel/fix-flaky-test

Conversation

@dtivel
Copy link
Collaborator

@dtivel dtivel commented Nov 28, 2025

Resolve #956

This change fixes an intermittent IOException in Crypt32Tests.ShouldTimestampData where the test attempted to read the test timestamp server's log file before it was fully closed. This eliminates the race condition where tests could access files still held open by the timestamp service.

  • Made RespondAsync(...) truly async with async Task and await in finally block
  • Created async WriteAsync(...) method using await using and async write operations

@dtivel dtivel requested a review from a team as a code owner November 28, 2025 15:41
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.

Test: fix flaky test

3 participants