Skip to content

Fix flaky HTTP request logger test - #2596

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue2419
May 24, 2026
Merged

Fix flaky HTTP request logger test#2596
Widthdom merged 1 commit into
mainfrom
fix-issue2419

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Extended the HTTP MCP request logger test grace period for async log delivery under full-suite load.
  • Added timeout diagnostics that show the request log records observed before failure.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~HttpMcpTransportTests
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check

Full dotnet test was attempted, but an unrelated trimmed publish test failed because ILLink could not write src/CodeIndex/obj/Debug/net8.0/osx-arm64/linked/cdidx.pdb while another process held it.

Documentation and changelog

  • Added changelog fragment changelog.d/unreleased/2419.fixed.md.
  • No documentation update needed; this is a test stability fix only.

Adversarial review

  • Manual adversarial review found no blocking/actionable issues.
  • Codex adversarial review was attempted, but the usable diff-based run was blocked by the Codex usage limit.

Fixes #2419

@Widthdom
Widthdom merged commit bd01bb4 into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue2419 branch May 24, 2026 09:21
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 HttpTransport request logger test records only two requests in full suite

1 participant