Skip to content

Stabilize GitHelper timeout regression test under full-suite load - #5032

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue4982
Aug 1, 2026
Merged

Stabilize GitHelper timeout regression test under full-suite load#5032
Widthdom merged 1 commit into
mainfrom
fix-issue4982

Conversation

@Widthdom

@Widthdom Widthdom commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep the fake diff-tree process blocked beyond the assertion window and record its PID
  • verify the 500 ms timeout reaps the fake Git process before returning within the shared 10-second wall-clock bound
  • document the full-suite timing contract and add a bilingual changelog fragment

Validation

  • dotnet restore CodeIndex.sln
  • dotnet build CodeIndex.sln -c Release --no-restore -p:UseSharedCompilation=false (0 warnings, 0 errors)
  • dotnet build CodeIndex.sln -c Debug --no-restore -p:UseSharedCompilation=false (0 warnings, 0 errors)
  • focused timeout test passed repeatedly, including under concurrent index/full-suite load
  • GitHelperTests: 77 passed on Release net8.0
  • full Release suite with 5-minute hang collection:
    • net9.0: 10,612 passed, 420 skipped, 0 failed
    • net8.0: the changed timeout test passed; 11,085 passed and 7 skipped, with one unrelated RunBatch_ParallelRefreshesDirectSessionsBetweenItems_Issue4872 load-time failure; its immediate isolated rerun passed
  • dotnet tools/CodeIndex.Changelog/bin/Release/net8.0/CodeIndex.Changelog.dll check (101 fragments validated)
  • cdidx status --check --json (index_matches_workspace: true, reference graph complete)
  • Codex adversarial review: No blocking/actionable issues found.

Documentation and changelog

  • updated the English and Japanese timeout-test guidance in TESTING_GUIDE.md
  • added changelog.d/unreleased/4982.internal.md

Follow-up candidates

  • None.

Fixes #4982

@Widthdom
Widthdom marked this pull request as ready for review August 1, 2026 21:47
@Widthdom
Widthdom merged commit a998977 into main Aug 1, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue4982 branch August 2, 2026 12:49
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.

Stabilize GitHelper timeout regression test under full-suite load

1 participant