test(cli,sdk): mock KeychainTokenStorage and homedir in tests

Mock KeychainTokenStorage in ExtensionManager tests and homedir in SDK integration tests to avoid side effects, although tests are still timing out.

TAG=agy
CONV=4043d296-611e-448a-a9d9-cd78f122b436

Author: akh64bit

packages/cli/src/config/extension-manager-agents.test.ts

@@ -32,6 +32,13 @@ vi.mock('@google/gemini-cli-core', async (importOriginal) => {
     homedir: mockHomedir,
     loadAgentsFromDirectory: core.loadAgentsFromDirectory,
     loadSkillsFromDir: core.loadSkillsFromDir,
+    KeychainTokenStorage: vi.fn().mockImplementation(() => ({
+      getSecret: vi.fn().mockResolvedValue(undefined),
+      setSecret: vi.fn().mockResolvedValue(undefined),
+      deleteSecret: vi.fn().mockResolvedValue(undefined),
+      isAvailable: vi.fn().mockResolvedValue(true),
+      listSecrets: vi.fn().mockResolvedValue([]),
+    })),
   };
 });
 

packages/cli/src/config/extension-manager-hydration.test.ts

@@ -38,6 +38,13 @@ vi.mock('@google/gemini-cli-core', async (importOriginal) => {
     // Use actual implementations for loading skills and agents to test hydration
     loadAgentsFromDirectory: actual.loadAgentsFromDirectory,
     loadSkillsFromDir: actual.loadSkillsFromDir,
+    KeychainTokenStorage: vi.fn().mockImplementation(() => ({
+      getSecret: vi.fn().mockResolvedValue(undefined),
+      setSecret: vi.fn().mockResolvedValue(undefined),
+      deleteSecret: vi.fn().mockResolvedValue(undefined),
+      isAvailable: vi.fn().mockResolvedValue(true),
+      listSecrets: vi.fn().mockResolvedValue([]),
+    })),
   };
 });
 

packages/sdk/src/skills.integration.test.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { GeminiCliAgent } from './agent.js';
 import { skillDir } from './skills.js';
 import * as path from 'node:path';
@@ -13,6 +13,25 @@ import { fileURLToPath } from 'node:url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
+const mockHomedir = vi.hoisted(() => vi.fn(() => '/tmp/mock-home'));
+
+vi.mock('node:os', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('node:os')>();
+  return {
+    ...actual,
+    homedir: mockHomedir,
+  };
+});
+
+vi.mock('@google/gemini-cli-core', async (importOriginal) => {
+  const actual =
+    await importOriginal<typeof import('@google/gemini-cli-core')>();
+  return {
+    ...actual,
+    homedir: mockHomedir,
+  };
+});
+
 // Set this to true locally when you need to update snapshots
 const RECORD_MODE = process.env['RECORD_NEW_RESPONSES'] === 'true';
 

packages/sdk/src/tool.integration.test.ts

@@ -4,13 +4,32 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { GeminiCliAgent } from './agent.js';
 import * as path from 'node:path';
 import { z } from 'zod';
 import { tool, ModelVisibleError } from './tool.js';
 import { fileURLToPath } from 'node:url';
 
+const mockHomedir = vi.hoisted(() => vi.fn(() => '/tmp/mock-home'));
+
+vi.mock('node:os', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('node:os')>();
+  return {
+    ...actual,
+    homedir: mockHomedir,
+  };
+});
+
+vi.mock('@google/gemini-cli-core', async (importOriginal) => {
+  const actual =
+    await importOriginal<typeof import('@google/gemini-cli-core')>();
+  return {
+    ...actual,
+    homedir: mockHomedir,
+  };
+});
+
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);