From 9295dd1ed0de903a294692502acf36cfd4180818 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Tue, 27 May 2025 00:44:33 +0200
Subject: [PATCH] Fix server status test by properly mocking fs module
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Mock fs.readFile in beforeEach to return valid package.json
- Mock fs module with existsSync and accessSync for CLI checks

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 tests/unit/tools/list.test.ts | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/tests/unit/tools/list.test.ts b/tests/unit/tools/list.test.ts
index d3e8add..1935360 100644
--- a/tests/unit/tools/list.test.ts
+++ b/tests/unit/tools/list.test.ts
@@ -21,6 +21,14 @@ import {
 vi.mock("../../../src/utils/peekaboo-cli");
 vi.mock("../../../src/utils/server-status");
 vi.mock("fs/promises");
+vi.mock("fs", () => ({
+  existsSync: vi.fn(() => false),
+  accessSync: vi.fn(),
+  constants: {
+    X_OK: 1,
+    W_OK: 2,
+  },
+}));
 
 // Mock path and url functions to avoid import.meta.url issues in test environment
 // jest.mock('url', () => ({ // REMOVED
@@ -58,6 +66,8 @@ describe("List Tool", () => {
     vi.clearAllMocks();
     // Mock fs.access to always succeed by default
     mockFsAccess.mockResolvedValue(undefined);
+    // Mock fs.readFile to return a valid package.json
+    mockFsReadFile.mockResolvedValue(JSON.stringify({ version: "1.0.0" }));
   });
 
   describe("buildSwiftCliArgs", () => {