@akiojin/gwt 4.10.0 → 4.12.0

package/src/cli/ui/__tests__/hooks/useGitData.test.ts

@@ -1,197 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, vi, afterEach } from "vitest";
-import { renderHook, act, waitFor } from "@testing-library/react";
-import { Window } from "happy-dom";
-
-// モジュールをモック
-vi.mock("../../../../git.js", () => ({
-  getAllBranches: vi.fn(),
-  hasUnpushedCommitsInRepo: vi.fn(),
-  getRepositoryRoot: vi.fn(),
-  fetchAllRemotes: vi.fn(),
-  collectUpstreamMap: vi.fn(),
-  getBranchDivergenceStatuses: vi.fn(),
-  hasUncommittedChanges: vi.fn(),
-}));
-
-vi.mock("../../../../worktree.js", () => ({
-  listAdditionalWorktrees: vi.fn(),
-}));
-
-vi.mock("../../../../github.js", () => ({
-  getPullRequestByBranch: vi.fn(),
-}));
-
-vi.mock("../../../../config/index.js", () => ({
-  getLastToolUsageMap: vi.fn(),
-}));
-
-import { useGitData } from "../../hooks/useGitData.js";
-import {
-  getAllBranches,
-  getRepositoryRoot,
-  fetchAllRemotes,
-  collectUpstreamMap,
-  getBranchDivergenceStatuses,
-} from "../../../../git.js";
-import { listAdditionalWorktrees } from "../../../../worktree.js";
-import { getLastToolUsageMap } from "../../../../config/index.js";
-
-const mockGetAllBranches = getAllBranches as ReturnType<typeof vi.fn>;
-const mockGetRepositoryRoot = getRepositoryRoot as ReturnType<typeof vi.fn>;
-const mockFetchAllRemotes = fetchAllRemotes as ReturnType<typeof vi.fn>;
-const mockCollectUpstreamMap = collectUpstreamMap as ReturnType<typeof vi.fn>;
-const mockGetBranchDivergenceStatuses =
-  getBranchDivergenceStatuses as ReturnType<typeof vi.fn>;
-const mockListAdditionalWorktrees = listAdditionalWorktrees as ReturnType<
-  typeof vi.fn
->;
-const mockGetLastToolUsageMap = getLastToolUsageMap as ReturnType<typeof vi.fn>;
-
-describe("useGitData", () => {
-  beforeEach(() => {
-    // Setup happy-dom
-    const window = new Window();
-    globalThis.window = window as unknown as typeof globalThis.window;
-    globalThis.document =
-      window.document as unknown as typeof globalThis.document;
-
-    // Reset all mocks
-    vi.clearAllMocks();
-
-    // Default mock implementations
-    mockGetRepositoryRoot.mockResolvedValue("/mock/repo");
-    mockFetchAllRemotes.mockResolvedValue(undefined);
-    mockGetAllBranches.mockResolvedValue([
-      {
-        name: "main",
-        type: "local",
-        isCurrent: true,
-        lastCommitDate: "2025-01-01",
-      },
-      {
-        name: "develop",
-        type: "local",
-        isCurrent: false,
-        lastCommitDate: "2025-01-02",
-      },
-    ]);
-    mockListAdditionalWorktrees.mockResolvedValue([]);
-    mockCollectUpstreamMap.mockResolvedValue(new Map());
-    mockGetBranchDivergenceStatuses.mockResolvedValue([]);
-    mockGetLastToolUsageMap.mockResolvedValue(new Map());
-  });
-
-  afterEach(() => {
-    vi.restoreAllMocks();
-  });
-
-  describe("キャッシュ機構", () => {
-    it("初回マウント時にGitデータを取得する", async () => {
-      const { result } = renderHook(() => useGitData());
-
-      // 初回ロード中
-      expect(result.current.loading).toBe(true);
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // Gitデータが取得されていることを確認
-      expect(mockGetAllBranches).toHaveBeenCalledTimes(1);
-      expect(result.current.branches).toHaveLength(2);
-    });
-
-    it("refresh()を呼び出すとGitデータを再取得する", async () => {
-      const { result } = renderHook(() => useGitData());
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // 初回ロードで1回呼ばれている
-      expect(mockGetAllBranches).toHaveBeenCalledTimes(1);
-
-      // refresh()を呼び出す
-      await act(async () => {
-        result.current.refresh();
-      });
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // refresh後に再度呼ばれている（forceRefresh=true）
-      expect(mockGetAllBranches).toHaveBeenCalledTimes(2);
-    });
-
-    it("キャッシュ済みの場合、再マウント時にGitデータを再取得しない", async () => {
-      // 注: useGitData は内部で useRef を使ってキャッシュ状態を管理
-      // 同一コンポーネント内での再レンダリングではキャッシュが効く
-      const { result, rerender } = renderHook(() => useGitData());
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // 初回ロードで1回呼ばれている
-      expect(mockGetAllBranches).toHaveBeenCalledTimes(1);
-
-      // 再レンダリング
-      rerender();
-
-      // キャッシュされているため追加の呼び出しはない
-      expect(mockGetAllBranches).toHaveBeenCalledTimes(1);
-    });
-  });
-
-  describe("lastUpdated", () => {
-    it("データ取得成功後にlastUpdatedが更新される", async () => {
-      const { result } = renderHook(() => useGitData());
-
-      // 初期状態ではnull
-      expect(result.current.lastUpdated).toBeNull();
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // ロード完了後にlastUpdatedが設定される
-      expect(result.current.lastUpdated).toBeInstanceOf(Date);
-    });
-  });
-
-  describe("エラーハンドリング", () => {
-    it("getAllBranchesがエラーを投げた場合、フォールバック値（空配列）が使用される", async () => {
-      // withTimeout がエラーをキャッチしてフォールバック値を返すため、
-      // エラー状態にはならず、空配列が設定される
-      mockGetAllBranches.mockRejectedValue(new Error("Git error"));
-
-      const { result } = renderHook(() => useGitData());
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // エラーはキャッチされ、フォールバック値が使用される
-      expect(result.current.error).toBeNull();
-      expect(result.current.branches).toHaveLength(0);
-    });
-
-    it("fetchAllRemotesがエラーを投げてもローカル表示は継続される", async () => {
-      mockFetchAllRemotes.mockRejectedValue(new Error("Network error"));
-
-      const { result } = renderHook(() => useGitData());
-
-      await waitFor(() => {
-        expect(result.current.loading).toBe(false);
-      });
-
-      // リモート取得失敗でもローカルブランチは表示される
-      expect(result.current.branches).toHaveLength(2);
-      expect(result.current.error).toBeNull();
-    });
-  });
-});

package/src/cli/ui/__tests__/hooks/useGitData.test.ts.skip

@@ -1,228 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { renderHook, waitFor } from '@testing-library/react';
-import { useGitData } from '../../hooks/useGitData.js';
-import { Window } from 'happy-dom';
-import type { BranchInfo } from '../../types.js';
-// Mock git.js and worktree.js
-vi.mock('../../../git.js', () => ({
-  getAllBranches: vi.fn(),
-}));
-
-vi.mock('../../../worktree.js', () => ({
-  listAdditionalWorktrees: vi.fn(),
-}));
-
-import { getAllBranches } from '../../../git.js';
-import { listAdditionalWorktrees } from '../../../worktree.js';
-
-describe('useGitData', () => {
-  beforeEach(() => {
-    // Setup happy-dom
-    const window = new Window();
-    globalThis.window = window as any;
-    globalThis.document = window.document as any;
-
-    // Reset mocks
-    (getAllBranches as ReturnType<typeof vi.fn>).mockReset();
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockReset();
-  });
-
-  it('should initialize with loading state', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const { result } = renderHook(() => useGitData());
-
-    // In test environment, useEffect runs synchronously, so loading may already be false
-    // Check that it eventually becomes false and data is loaded
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    expect(result.current.branches).toEqual([]);
-    expect(result.current.worktrees).toEqual([]);
-    expect(result.current.error).toBeNull();
-  });
-
-  it('should load branches and worktrees', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
-        isCurrent: true,
-      },
-      {
-        name: 'feature/test',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-    ];
-
-    const mockWorktrees = [
-      {
-        path: '/path/to/worktree',
-        branch: 'feature/test',
-        head: 'abc123',
-        isAccessible: true,
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue(mockWorktrees);
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    expect(result.current.branches).toHaveLength(2);
-    expect(result.current.branches[1].worktree).toBeDefined();
-    expect(result.current.branches[1].worktree?.path).toBe('/path/to/worktree');
-    expect(result.current.worktrees).toHaveLength(1);
-    expect(result.current.worktrees[0].branch).toBe('feature/test');
-    expect(result.current.error).toBeNull();
-  });
-
-  it('should handle errors', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockRejectedValue(new Error('Git error'));
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    expect(result.current.error).toBeDefined();
-    expect(result.current.error?.message).toBe('Git error');
-    expect(result.current.branches).toEqual([]);
-    expect(result.current.worktrees).toEqual([]);
-  });
-
-  it('should support manual refresh', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
-        isCurrent: true,
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    expect(result.current.branches).toHaveLength(1);
-
-    // Add a new branch
-    const updatedBranches: BranchInfo[] = [
-      ...mockBranches,
-      {
-        name: 'feature/new',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(updatedBranches);
-
-    // Trigger refresh
-    result.current.refresh();
-
-    await waitFor(() => {
-      expect(result.current.branches).toHaveLength(2);
-    });
-  });
-
-  it('should match worktrees to branches by name', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'feature/a',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-      {
-        name: 'feature/b',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-      {
-        name: 'feature/c',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-    ];
-
-    const mockWorktrees = [
-      {
-        path: '/path/a',
-        branch: 'feature/a',
-        head: 'aaa',
-        isAccessible: true,
-      },
-      {
-        path: '/path/c',
-        branch: 'feature/c',
-        head: 'ccc',
-        isAccessible: false,
-        invalidReason: 'Path does not exist',
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue(mockWorktrees);
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    const branchA = result.current.branches.find((b) => b.name === 'feature/a');
-    const branchB = result.current.branches.find((b) => b.name === 'feature/b');
-    const branchC = result.current.branches.find((b) => b.name === 'feature/c');
-
-    expect(branchA?.worktree).toBeDefined();
-    expect(branchA?.worktree?.path).toBe('/path/a');
-
-    expect(branchB?.worktree).toBeUndefined();
-
-    expect(branchC?.worktree).toBeDefined();
-    expect(branchC?.worktree?.path).toBe('/path/c');
-  });
-
-  it('should set lastUpdated after loading data', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(() => {
-      expect(result.current.loading).toBe(false);
-    });
-
-    expect(result.current.lastUpdated).toBeInstanceOf(Date);
-  });
-
-  // Note: Auto-refresh tests removed due to flakiness in parallel execution
-  // The auto-refresh functionality is thoroughly tested in:
-  // - src/ui/__tests__/integration/realtimeUpdate.test.tsx
-  // - src/ui/__tests__/acceptance/realtimeUpdate.acceptance.test.tsx
-  // These tests pass when run in isolation but fail in parallel due to timer precision issues
-});

package/src/cli/ui/__tests__/hooks/useScreenState.test.ts

@@ -1,147 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach } from "vitest";
-import { renderHook, act } from "@testing-library/react";
-import { useScreenState } from "../../hooks/useScreenState.js";
-import type { ScreenType } from "../../types.js";
-import { Window } from "happy-dom";
-
-describe("useScreenState", () => {
-  beforeEach(() => {
-    // Setup happy-dom
-    const window = new Window();
-    globalThis.window = window as unknown as typeof globalThis.window;
-    globalThis.document =
-      window.document as unknown as typeof globalThis.document;
-  });
-  it("should initialize with branch-list as active screen", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-
-  it("should navigate to a new screen", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    act(() => {
-      result.current.navigateTo("branch-creator");
-    });
-
-    expect(result.current.currentScreen).toBe("branch-creator");
-  });
-
-  it("should navigate back to previous screen", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    act(() => {
-      result.current.navigateTo("branch-creator");
-    });
-
-    expect(result.current.currentScreen).toBe("branch-creator");
-
-    act(() => {
-      result.current.goBack();
-    });
-
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-
-  it("should maintain screen history", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    act(() => {
-      result.current.navigateTo("branch-creator");
-    });
-
-    act(() => {
-      result.current.navigateTo("ai-tool-selector");
-    });
-
-    expect(result.current.currentScreen).toBe("ai-tool-selector");
-
-    act(() => {
-      result.current.goBack();
-    });
-
-    expect(result.current.currentScreen).toBe("branch-creator");
-
-    act(() => {
-      result.current.goBack();
-    });
-
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-
-  it("should not go back when at initial screen", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    expect(result.current.currentScreen).toBe("branch-list");
-
-    act(() => {
-      result.current.goBack();
-    });
-
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-
-  it("should handle multiple navigations correctly", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    const screens: ScreenType[] = [
-      "branch-creator",
-      "ai-tool-selector",
-      "model-selector",
-      "execution-mode-selector",
-    ];
-
-    screens.forEach((screen) => {
-      act(() => {
-        result.current.navigateTo(screen);
-      });
-    });
-
-    expect(result.current.currentScreen).toBe("execution-mode-selector");
-
-    // Go back through all screens
-    act(() => {
-      result.current.goBack();
-    });
-    expect(result.current.currentScreen).toBe("model-selector");
-
-    act(() => {
-      result.current.goBack();
-    });
-    expect(result.current.currentScreen).toBe("ai-tool-selector");
-
-    act(() => {
-      result.current.goBack();
-    });
-    expect(result.current.currentScreen).toBe("branch-creator");
-
-    act(() => {
-      result.current.goBack();
-    });
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-
-  it("should reset to initial screen", () => {
-    const { result } = renderHook(() => useScreenState());
-
-    act(() => {
-      result.current.navigateTo("branch-creator");
-    });
-
-    act(() => {
-      result.current.navigateTo("ai-tool-selector");
-    });
-
-    expect(result.current.currentScreen).toBe("ai-tool-selector");
-
-    act(() => {
-      result.current.reset();
-    });
-
-    expect(result.current.currentScreen).toBe("branch-list");
-  });
-});

package/src/cli/ui/__tests__/hooks/useTerminalSize.test.ts

@@ -1,99 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import { renderHook, act } from "@testing-library/react";
-import { useTerminalSize } from "../../hooks/useTerminalSize.js";
-import { Window } from "happy-dom";
-
-describe("useTerminalSize", () => {
-  const originalRows = process.stdout.rows;
-  const originalColumns = process.stdout.columns;
-
-  beforeEach(() => {
-    // Setup happy-dom
-    const window = new Window();
-    globalThis.window = window as unknown as typeof globalThis.window;
-    globalThis.document =
-      window.document as unknown as typeof globalThis.document;
-
-    // デフォルト値を設定
-    process.stdout.rows = 24;
-    process.stdout.columns = 80;
-  });
-
-  afterEach(() => {
-    // 元の値に戻す
-    process.stdout.rows = originalRows;
-    process.stdout.columns = originalColumns;
-    vi.restoreAllMocks();
-  });
-
-  it("should return current terminal size", () => {
-    const { result } = renderHook(() => useTerminalSize());
-
-    expect(result.current.rows).toBe(24);
-    expect(result.current.columns).toBe(80);
-  });
-
-  it("should use default values when stdout properties are undefined", () => {
-    process.stdout.rows = undefined as any;
-    process.stdout.columns = undefined as any;
-
-    const { result } = renderHook(() => useTerminalSize());
-
-    expect(result.current.rows).toBe(24); // デフォルト値
-    expect(result.current.columns).toBe(80); // デフォルト値
-  });
-
-  it("should update size when resize event is emitted", () => {
-    const { result } = renderHook(() => useTerminalSize());
-
-    expect(result.current.rows).toBe(24);
-    expect(result.current.columns).toBe(80);
-
-    act(() => {
-      process.stdout.rows = 30;
-      process.stdout.columns = 120;
-      process.stdout.emit("resize");
-    });
-
-    expect(result.current.rows).toBe(30);
-    expect(result.current.columns).toBe(120);
-  });
-
-  it("should cleanup resize listener on unmount", () => {
-    const removeListenerSpy = vi.spyOn(process.stdout, "removeListener");
-
-    const { unmount } = renderHook(() => useTerminalSize());
-
-    unmount();
-
-    expect(removeListenerSpy).toHaveBeenCalledWith(
-      "resize",
-      expect.any(Function),
-    );
-  });
-
-  it("should handle multiple resize events", () => {
-    const { result } = renderHook(() => useTerminalSize());
-
-    act(() => {
-      process.stdout.rows = 40;
-      process.stdout.columns = 100;
-      process.stdout.emit("resize");
-    });
-
-    expect(result.current.rows).toBe(40);
-    expect(result.current.columns).toBe(100);
-
-    act(() => {
-      process.stdout.rows = 50;
-      process.stdout.columns = 150;
-      process.stdout.emit("resize");
-    });
-
-    expect(result.current.rows).toBe(50);
-    expect(result.current.columns).toBe(150);
-  });
-});