diff --git a/src/logging/diagnostic.test.ts b/src/logging/diagnostic.test.ts index c7cd72f5c08..10c666432c1 100644 --- a/src/logging/diagnostic.test.ts +++ b/src/logging/diagnostic.test.ts @@ -1,3 +1,4 @@ +import fs from "node:fs"; import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import { getDiagnosticSessionStateCountForTest, @@ -34,3 +35,21 @@ describe("diagnostic session state pruning", () => { expect(getDiagnosticSessionStateCountForTest()).toBe(2000); }); }); + +describe("logger import side effects", () => { + afterEach(() => { + vi.restoreAllMocks(); + vi.useRealTimers(); + }); + + it("does not mkdir at import time", async () => { + vi.useRealTimers(); + vi.resetModules(); + + const mkdirSpy = vi.spyOn(fs, "mkdirSync"); + + await import("./logger.js"); + + expect(mkdirSpy).not.toHaveBeenCalled(); + }); +}); diff --git a/src/logging/logger.import-side-effects.test.ts b/src/logging/logger.import-side-effects.test.ts deleted file mode 100644 index b0e8c2b9729..00000000000 --- a/src/logging/logger.import-side-effects.test.ts +++ /dev/null @@ -1,16 +0,0 @@ -import fs from "node:fs"; -import { afterEach, describe, expect, it, vi } from "vitest"; - -describe("logger import side effects", () => { - afterEach(() => { - vi.restoreAllMocks(); - }); - - it("does not mkdir at import time", async () => { - const mkdirSpy = vi.spyOn(fs, "mkdirSync"); - - await import("./logger.js"); - - expect(mkdirSpy).not.toHaveBeenCalled(); - }); -});