Files
Ikraam GhoorandClaude Opus 4.6 bdb05afd55 Fixed flaky CI tests caused by mock.module pollution
Bun's mock.module() is global and hoisted — it replaces modules for
ALL test files in the same process regardless of mock.restore() timing.
This caused byos-auth, scheduleStore, and dithering tests to alternate
between pass and fail depending on file load order.

Replaced mock.module() with constructor dependency injection on both
HttpRouter and Browser classes. Tests now pass mock implementations via
constructor parameters, eliminating cross-file contamination entirely.

Also installed ImageMagick 7 Q16-HDRI in CI (docker cp from production
image) and replaced describe.skip with runtime IM7 detection so the 33
dithering tests now run in CI instead of being permanently skipped.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-06 12:57:31 +00:00
..
2025-12-29 12:54:34 +00:00
2025-12-29 19:27:26 +00:00
2025-12-29 19:27:26 +00:00