perf(test): fold logger import side-effects test into diagnostic suite
This commit is contained in:
parent
166cf6a3e0
commit
412c1d0af1
@ -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();
|
||||
});
|
||||
});
|
||||
|
||||
@ -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();
|
||||
});
|
||||
});
|
||||
Loading…
x
Reference in New Issue
Block a user