Skip to content

test: make Windows maintenance concurrency deterministic - #251

Merged
khaira777 merged 1 commit into
mainfrom
fix/maintenance-test-synchronization
Aug 11, 2026
Merged

test: make Windows maintenance concurrency deterministic#251
khaira777 merged 1 commit into
mainfrom
fix/maintenance-test-synchronization

Conversation

@khaira777

Copy link
Copy Markdown
Contributor

Summary

  • replace the fixed 20 ms maintenance delay with an explicit promise gate
  • keep maintenance active until the concurrent route assertion completes
  • preserve the 503-before-auth regression coverage without changing production code

Validation

  • npm run test:database-tools-api
  • npm test
  • npm run build
  • npm run lint
  • focused maintenance test repeated 20 times

Fixes the Windows-only 200 !== 503 flake in database-maintenance-lock.test.ts.

@khaira777
khaira777 requested a review from itsbkm as a code owner August 11, 2026 04:56
@khaira777
khaira777 merged commit a98e21e into main Aug 11, 2026
9 checks passed
@khaira777
khaira777 deleted the fix/maintenance-test-synchronization branch August 11, 2026 05:03
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