Skip to content

Commit c38684b

Browse files
authored
fix(test): skip watchRecursive suite on win32
The daemon is explicitly deferred on Windows (v0.8.0 release notes), and Bun's fs.watch on win32 hard-crashes the test process mid-suite — which silently prevented the remaining ~260 tests (ingest, search, parsers — all of which DO ship on Windows) from running at all. Skipping the unshipped feature's suite restores real Windows coverage for everything that is shipped.
1 parent 9c07d02 commit c38684b

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

test/daemon-watcher.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,9 @@ const settle = (ms = 250) => new Promise((r) => setTimeout(r, ms));
2929

3030
// Tests --------------------------------------------------------------
3131

32-
describe("watchRecursive", () => {
32+
// The daemon is deferred on Windows (see release notes); Bun's fs.watch on
33+
// win32 hard-crashes the test process, so skip the suite there entirely.
34+
describe.skipIf(process.platform === "win32")("watchRecursive", () => {
3335
test("throws if root does not exist", () => {
3436
expect(() =>
3537
watchRecursive(join(tmpRoot, "does-not-exist"), () => {}),

0 commit comments

Comments
 (0)