From cf29cd137ed1f1374380285f97044adb9637749c Mon Sep 17 00:00:00 2001
From: YeonGyu-Kim <code.yeon.gyu@gmail.com>
Date: Sat, 7 Feb 2026 20:06:58 +0900
Subject: [PATCH] test: isolate user-level MCP config test from real homedir

---
 .../claude-code-mcp-loader/loader.test.ts     | 64 +++++++++----------
 1 file changed, 32 insertions(+), 32 deletions(-)

diff --git a/src/features/claude-code-mcp-loader/loader.test.ts b/src/features/claude-code-mcp-loader/loader.test.ts
index 67de9127e..8faf65718 100644
--- a/src/features/claude-code-mcp-loader/loader.test.ts
+++ b/src/features/claude-code-mcp-loader/loader.test.ts
@@ -1,7 +1,7 @@
-import { describe, it, expect, beforeEach, afterEach } from "bun:test"
+import { describe, it, expect, beforeEach, afterEach, mock } from "bun:test"
 import { mkdirSync, writeFileSync, rmSync } from "fs"
 import { join } from "path"
-import { tmpdir, homedir } from "os"
+import { tmpdir } from "os"
 
 const TEST_DIR = join(tmpdir(), "mcp-loader-test-" + Date.now())
 
@@ -160,36 +160,36 @@ describe("getSystemMcpServerNames", () => {
      }
    })
 
-   it("reads user-level MCP config from ~/.claude.json", async () => {
-     // given
-     const userConfigPath = join(homedir(), ".claude.json")
-     const userMcpConfig = {
-       mcpServers: {
-         "user-server": {
-           command: "npx",
-           args: ["user-mcp-server"],
-         },
-       },
-     }
-     
-     // Create user config if it doesn't exist
-     const originalCwd = process.cwd()
-     process.chdir(TEST_DIR)
-     
-     try {
-       // Write user config temporarily
-       writeFileSync(userConfigPath, JSON.stringify(userMcpConfig))
+    it("reads user-level MCP config from ~/.claude.json", async () => {
+      // given
+      const userConfigPath = join(TEST_DIR, ".claude.json")
+      const userMcpConfig = {
+        mcpServers: {
+          "user-server": {
+            command: "npx",
+            args: ["user-mcp-server"],
+          },
+        },
+      }
 
-       // when
-       const { getSystemMcpServerNames } = await import("./loader")
-       const names = getSystemMcpServerNames()
+      const originalCwd = process.cwd()
+      process.chdir(TEST_DIR)
 
-       // then
-       expect(names.has("user-server")).toBe(true)
-     } finally {
-       process.chdir(originalCwd)
-       // Clean up user config
-       rmSync(userConfigPath, { force: true })
-     }
-   })
+      try {
+        mock.module("os", () => ({
+          homedir: () => TEST_DIR,
+          tmpdir,
+        }))
+
+        writeFileSync(userConfigPath, JSON.stringify(userMcpConfig))
+
+        const { getSystemMcpServerNames } = await import("./loader")
+        const names = getSystemMcpServerNames()
+
+        expect(names.has("user-server")).toBe(true)
+      } finally {
+        process.chdir(originalCwd)
+        rmSync(userConfigPath, { force: true })
+      }
+    })
 })