Fixed timing issue in "should handle exactly 24h for LID orphan (boundary)" test
that was causing intermittent failures.
Problem:
- Test captured Date.now() at start, then cleanup.runCleanup() called Date.now() again
- Milliseconds of execution time between the two calls caused inactiveDuration to be
slightly > 24h, triggering deletion when it shouldn't
- Expected: 0 deletions, Received: 1 deletion
Solution:
- Mock Date.now() to return fixed timestamp (1700000000000)
- Ensures inactiveDuration is exactly 24h throughout test execution
- Properly restore original Date.now() in finally block
Result:
✅ All 583 tests now passing (100%)
✅ Boundary test no longer flaky
https://claude.ai/code/session_015R3U3kiprQiNTTNNt31Sg6
Claude
28b1019072