test(config): check settings import laziness in a fresh process

Author: bwachman

scripts/settings-lazy-probe.ts

@@ -0,0 +1,7 @@
+import { schemaMaterializations } from "../src/schema/lazy.js";
+
+const before = schemaMaterializations();
+await import("../src/config/settings.js");
+process.stdout.write(
+  JSON.stringify({ before, afterImport: schemaMaterializations() }),
+);

src/config/settings-lazy.test.ts

@@ -1,22 +1,26 @@
-import { mkdtemp, rm, writeFile } from "node:fs/promises";
-import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { expect, test } from "bun:test";
 
-import { schemaMaterializations } from "../schema/lazy.js";
+const settingsLazinessProbe = join(
+  import.meta.dirname,
+  "../../scripts/settings-lazy-probe.ts",
+);
 
-test("importing settings does not build schemas; a malformed file fails when read", async () => {
-  const before = schemaMaterializations();
-  const settings = await import("./settings.js");
-  expect(schemaMaterializations()).toBe(before);
+test("importing settings does not build schemas", async () => {
+  const child = Bun.spawn([process.execPath, settingsLazinessProbe], {
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+  const [exitCode, stdout, stderr] = await Promise.all([
+    child.exited,
+    new Response(child.stdout).text(),
+    new Response(child.stderr).text(),
+  ]);
 
-  const dir = await mkdtemp(join(tmpdir(), "lazy-settings-"));
-  try {
-    const path = join(dir, "settings.json");
-    await writeFile(path, JSON.stringify({ providers: "nope" }));
-    await expect(settings.loadSettings(path)).rejects.toThrow();
-    expect(schemaMaterializations()).toBeGreaterThan(before);
-  } finally {
-    await rm(dir, { recursive: true, force: true });
-  }
+  expect(exitCode, stderr).toBe(0);
+  const materializations = JSON.parse(stdout) as {
+    before: number;
+    afterImport: number;
+  };
+  expect(materializations.afterImport).toBe(materializations.before);
 });