test: update tests to work with safeWriteJson

Updated tests to work with safeWriteJson instead of direct fs.writeFile calls:

- Updated importExport.test.ts to expect safeWriteJson calls instead of fs.writeFile
- Fixed McpHub.test.ts by properly mocking fs/promises module:
  - Moved jest.mock() to the top of the file before any imports
  - Added mock implementations for all fs functions used by safeWriteJson
  - Updated the test setup to work with the mocked fs module

All tests now pass successfully.

Signed-off-by: Eric Wheeler <[email protected]>

Author: Eric Wheeler

src/core/config/__tests__/importExport.test.ts

@@ -407,11 +407,10 @@ describe("importExport", () => {
 				contextProxy: mockContextProxy,
 			})
 
-			expect(fs.writeFile).toHaveBeenCalledWith(
-				"/mock/path/roo-code-settings.json",
-				JSON.stringify({ providerProfiles: mockProviderProfiles, globalSettings: mockGlobalSettings }, null, 2),
-				"utf-8",
-			)
+			expect(safeWriteJson).toHaveBeenCalledWith("/mock/path/roo-code-settings.json", {
+				providerProfiles: mockProviderProfiles,
+				globalSettings: mockGlobalSettings,
+			})
 		})
 
 		it("should handle errors during the export process", async () => {

src/services/mcp/__tests__/McpHub.test.ts

@@ -3,8 +3,35 @@ import type { ClineProvider } from "../../../core/webview/ClineProvider"
 import type { ExtensionContext, Uri } from "vscode"
 import { ServerConfigSchema } from "../McpHub"
 
-const fs = require("fs/promises")
 const { McpHub } = require("../McpHub")
+const path = require("path")
+
+// Mock fs/promises before importing anything that uses it
+jest.mock("fs/promises", () => ({
+	access: jest.fn().mockResolvedValue(undefined),
+	writeFile: jest.fn().mockResolvedValue(undefined),
+	readFile: jest.fn().mockResolvedValue("{}"),
+	unlink: jest.fn().mockResolvedValue(undefined),
+	rename: jest.fn().mockResolvedValue(undefined),
+	lstat: jest.fn().mockImplementation(() =>
+		Promise.resolve({
+			isDirectory: () => true,
+		}),
+	),
+	mkdir: jest.fn().mockResolvedValue(undefined),
+}))
+
+// Import the mocked fs
+const fs = require("fs/promises")
+
+// Mock safeWriteJson
+jest.mock("../../../utils/safeWriteJson", () => ({
+	safeWriteJson: jest.fn(async (filePath, data) => {
+		// Instead of trying to write to the file system, just call fs.writeFile mock
+		// This avoids the complex file locking and temp file operations
+		return fs.writeFile(filePath, JSON.stringify(data), "utf8")
+	}),
+}))
 
 jest.mock("vscode", () => ({
 	workspace: {
@@ -43,6 +70,9 @@ describe("McpHub", () => {
 		// Mock console.error to suppress error messages during tests
 		console.error = jest.fn()
 
+		// Reset the mock implementations before each test
+		jest.clearAllMocks()
+
 		const mockUri: Uri = {
 			scheme: "file",
 			authority: "",