From 975f28a57c059b0189d2d898a1055b09f6ea5960 Mon Sep 17 00:00:00 2001
From: decolua <decoluadt@gmail.com>
Date: Sat, 26 Sep 2026 11:40:08 +0700
Subject: [PATCH] test(zed): isolate zed-live-models suite DB via temp DATA_DIR

Seeding via createProviderConnection wrote zed-live-* accounts into the
real ~/.9router DB, showing up as junk accounts in the running dashboard.
Set DATA_DIR to a mkdtemp dir before dynamic-importing the DB-backed
modules and clean it up afterwards, matching the pattern in
compatible-provider-connections.test.js.

Co-Authored-By: Claude Code <noreply@anthropic.com>
---
 tests/unit/zed-live-models.test.js | 27 +++++++++++++++++++++++----
 1 file changed, 23 insertions(+), 4 deletions(-)

diff --git a/tests/unit/zed-live-models.test.js b/tests/unit/zed-live-models.test.js
index 1b999044..eaaea2fe 100644
--- a/tests/unit/zed-live-models.test.js
+++ b/tests/unit/zed-live-models.test.js
@@ -1,9 +1,10 @@
 // Route-level acceptance for the Zed live-model wiring:
 //   GET /api/providers/[connectionId]/models  →  resolveZedModels  →  UI rows
-// RUN WITH AN ISOLATED DB:  DATA_DIR=$(mktemp -d) npx vitest run ...
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import { GET } from "@/app/api/providers/[id]/models/route.js";
-import { createProviderConnection } from "@/models/index.js";
+// Self-isolating: DATA_DIR points at a temp dir so seeding never touches ~/.9router.
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+import { describe, it, expect, beforeAll, afterAll, beforeEach, afterEach, vi } from "vitest";
 
 // Transport stub BELOW resolveZedModels: proxyAwareFetch captures the native
 // fetch at import time, so stubbing globalThis.fetch cannot intercept it.
@@ -72,6 +73,24 @@ afterEach(() => {
   vi.restoreAllMocks();
 });
 
+// Imports must be dynamic so DATA_DIR is set before the DB layer loads.
+const originalDataDir = process.env.DATA_DIR;
+let GET;
+let createProviderConnection;
+
+beforeAll(async () => {
+  process.env.DATA_DIR = fs.mkdtempSync(path.join(os.tmpdir(), "9router-zed-live-"));
+  vi.resetModules();
+  ({ GET } = await import("@/app/api/providers/[id]/models/route.js"));
+  ({ createProviderConnection } = await import("@/models/index.js"));
+});
+
+afterAll(() => {
+  fs.rmSync(process.env.DATA_DIR, { recursive: true, force: true });
+  if (originalDataDir === undefined) delete process.env.DATA_DIR;
+  else process.env.DATA_DIR = originalDataDir;
+});
+
 async function seedZed(n) {
   return createProviderConnection({
     provider: "zed",