test(windows): stabilize subscription lock contention - #2662
Merged
wgqqqqq merged 1 commit intoAug 29, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Stabilize the cross-process subscription-auth file-lock regression test on loaded Windows runners.
The child process now confirms an actual OS-level lock contention before notifying the parent, replacing the previous fixed 100 ms scheduling assumption. Test-only timeout headroom covers Windows process startup, antivirus scanning, and metadata
sync_alllatency without changing production lock behavior.Type and Areas
Type: regression fix / test
Areas: Rust, AI adapters, subscription authentication, Windows CI
Motivation / Impact
The Windows Rust matrix intermittently failed
separate_process_waits_for_metadata_commit_before_reconciliationbecause the test announced child startup before the child had attempted the lock and used overlapping five-second timeouts.No direct user-facing change. Production file-lock behavior is unchanged.
Verification
pnpm run fmt:rsgit diff --checkcargo test --locked -p bitfun-ai-adapters --features subscription-auth --lib subscription_auth: 63 passedRun subscription authentication tests: passedRust Build Check (windows-latest): passedRust / CLI Validation: passedThe fork run had an unrelated Web UI assertion failure in
ConfigPageLayout.test.tsx; all Rust and CLI matrix jobs and the Rust/CLI summary gate passed.Reviewer Notes
This only changes code under
#[cfg(test)]. No persisted formats, product behavior, or remote-scenario behavior are affected.Checklist