test: fix config path handling in test environment

Co-Authored-By: Michael Latman <[email protected]>

src/utils/__tests__/config-manager.test.ts

@@ -1,4 +1,4 @@
-import { jest, describe, it, expect, beforeEach } from '@jest/globals';
+import { jest, describe, it, expect, beforeEach, afterEach } from '@jest/globals';
 import { ConfigManager } from '../config-manager';
 import { Package } from '../../types/package';
 import fs from 'fs';
@@ -11,9 +11,19 @@ jest.mock('path');
 describe('ConfigManager', () => {
   beforeEach(() => {
     jest.clearAllMocks();
-    jest.spyOn(fs, 'existsSync').mockReturnValue(true);
+
+    // Mock fs operations
+    jest.spyOn(fs, 'existsSync').mockImplementation(() => true);
     jest.spyOn(fs, 'readFileSync').mockReturnValue('{"mcpServers":{}}');
-    jest.spyOn(path, 'dirname').mockReturnValue('/mock/path');
+    jest.spyOn(fs, 'writeFileSync').mockImplementation(() => undefined);
+    jest.spyOn(fs, 'mkdirSync').mockImplementation(() => undefined);
+
+    // Mock path operations
+    jest.spyOn(path, 'dirname').mockReturnValue('/tmp');
+  });
+
+  afterEach(() => {
+    jest.restoreAllMocks();
   });
 
   describe('installPackage', () => {

src/utils/config-manager.ts

@@ -23,17 +23,19 @@ export class ConfigManager {
     private static preferencesPath: string;
 
     static {
-        if (process.platform === 'win32') {
+        const isTest = process.env.NODE_ENV === 'test';
+        if (isTest) {
+            this.configPath = '/tmp/test-mcp-config.json';
+            this.preferencesPath = '/tmp/test-mcp-preferences.json';
+        } else if (process.platform === 'win32') {
             const appData = process.env.APPDATA || path.join(os.homedir(), 'AppData', 'Roaming');
             this.configPath = path.join(appData, 'Claude', 'claude_desktop_config.json');
             this.preferencesPath = path.join(appData, 'mcp-get', 'preferences.json');
         } else if (process.platform === 'darwin') {
-            // macOS
             const homeDir = os.homedir();
             this.configPath = path.join(homeDir, 'Library', 'Application Support', 'Claude', 'claude_desktop_config.json');
             this.preferencesPath = path.join(homeDir, '.mcp-get', 'preferences.json');
         } else {
-            // Linux
             const homeDir = os.homedir();
             const configDir = process.env.XDG_CONFIG_HOME || path.join(homeDir, '.config');
             this.configPath = path.join(configDir, 'Claude', 'claude_desktop_config.json');
@@ -112,7 +114,6 @@ export class ConfigManager {
             env: envVars
         };
 
-        // Add command and args based on runtime
         if (pkg.runtime === 'node') {
             serverConfig.command = 'npx';
             serverConfig.args = ['-y', pkg.name];

test/setup.ts

@@ -0,0 +1,15 @@
+import { jest } from '@jest/globals';
+import fs from 'fs';
+import path from 'path';
+
+// Set NODE_ENV to 'test'
+process.env.NODE_ENV = 'test';
+
+// Ensure test directory exists
+const testConfigDir = path.dirname('/tmp/test-mcp-config.json');
+if (!fs.existsSync(testConfigDir)) {
+    fs.mkdirSync(testConfigDir, { recursive: true });
+}
+
+// Mock console.error to keep test output clean
+console.error = jest.fn();