Skip to content

test: stabilize logFileContent on Windows CI#79

Merged
s2005 merged 1 commit intomainfrom
fix/windows-logfilecontent-test-flake
Feb 15, 2026
Merged

test: stabilize logFileContent on Windows CI#79
s2005 merged 1 commit intomainfrom
fix/windows-logfilecontent-test-flake

Conversation

@s2005
Copy link
Copy Markdown
Owner

@s2005 s2005 commented Feb 15, 2026

Summary

  • replace file-exists polling with content-readiness polling in tests/unit/logFileContent.test.ts
  • wait for expected header/output markers before asserting
  • increase wait timeout to better tolerate slower Windows filesystem timing

Why

storeLog() writes files asynchronously, so checking only file existence can read partial content on Windows runners and produce intermittent failures.

Validation

  • npm test -- --runInBand tests/unit/logFileContent.test.ts
  • 80 repeated runs of the same test file all passed locally

@s2005 s2005 merged commit 0e5a780 into main Feb 15, 2026
3 checks passed
@s2005 s2005 deleted the fix/windows-logfilecontent-test-flake branch February 15, 2026 19:17
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.

1 participant