@akiojin/gwt 4.10.0 → 4.12.0

package/src/cli/ui/__tests__/SKIPPED_TESTS.md

@@ -1,119 +0,0 @@
-# Skipped Tests
-
-## Overview
-
-Some tests have been temporarily renamed with `.skip` extension to exclude them from the test suite. These tests pass when run in isolation but fail during parallel execution due to limitations in bun's vitest implementation.
-
-## Skipped Test Files
-
-### 1. `useGitData.test.ts.skip`
-
-- **Original location**: `src/ui/__tests__/hooks/useGitData.test.ts`
-- **Tests**: 6 tests (auto-refresh tests were removed earlier, 6 basic tests remain)
-- **Reason**: Timer-based and mock state conflicts in parallel execution
-- **Coverage**: Basic functionality is tested in other test files
-
-### 2. `realtimeUpdate.test.tsx.skip`
-
-- **Original location**: `src/ui/__tests__/integration/realtimeUpdate.test.tsx`
-- **Tests**: 5 tests (auto-refresh integration)
-- **Reason**: setInterval timing precision issues in parallel runs
-- **Coverage**: Auto-refresh functionality works correctly in production
-
-### 3. `branchList.test.tsx.skip`
-
-- **Original location**: `src/ui/__tests__/integration/branchList.test.tsx`
-- **Tests**: 5 tests (branch list integration)
-- **Reason**: Mock state conflicts with other parallel tests
-- **Coverage**: Component functionality tested in unit tests
-
-### 4. `realtimeUpdate.acceptance.test.tsx.skip`
-
-- **Original location**: `src/ui/__tests__/acceptance/realtimeUpdate.acceptance.test.tsx`
-- **Tests**: 4 acceptance tests
-- **Reason**: Timer-based acceptance criteria in parallel execution
-- **Coverage**: Same functionality as integration tests
-
-### 5. `branchList.acceptance.test.tsx.skip`
-
-- **Original location**: `src/ui/__tests__/acceptance/branchList.acceptance.test.tsx`
-- **Tests**: 2 acceptance tests (performance with 20+, 100+ branches)
-- **Reason**: Heavy load tests causing resource conflicts
-- **Coverage**: Performance validated in manual testing
-
-## Total Impact
-
-- **Skipped**: 22 tests
-- **Remaining**: 307 tests (100% pass rate)
-- **Test Coverage**: 81.78% (unchanged)
-
-## Technical Details
-
-### Why These Tests Fail in Parallel
-
-1. **Bun's vitest limitations**:
-   - No support for `pool` configuration options
-   - No support for `retry` option
-   - Limited control over test execution order
-
-2. **Timer precision**:
-   - `setInterval` and `setTimeout` behavior varies in parallel execution
-   - Tests expect specific timing (100ms, 300ms) which becomes unreliable
-
-3. **Mock state management**:
-   - Global mocks (getAllBranches, listAdditionalWorktrees) conflict between tests
-   - happy-dom environment state leaks between parallel tests
-
-### Verification
-
-All skipped tests pass when run individually:
-
-```bash
-# Examples of individual runs that pass
-bun test src/ui/__tests__/hooks/useGitData.test.ts.skip
-bun test src/ui/__tests__/integration/realtimeUpdate.test.tsx.skip
-bun test src/ui/__tests__/integration/branchList.test.tsx.skip
-```
-
-## Future Actions
-
-These tests can be re-enabled when:
-
-1. Bun's vitest adds support for sequential execution options
-2. Tests are rewritten to avoid timer dependencies
-3. Mock state management is refactored for better isolation
-
-## Running Skipped Tests Manually
-
-To run these tests locally for verification:
-
-```bash
-# Rename files temporarily
-for f in src/ui/__tests__/**/*.skip; do
-  mv "$f" "${f%.skip}"
-done
-
-# Run specific test file
-bun test src/ui/__tests__/hooks/useGitData.test.ts
-
-# Rename back
-for f in src/ui/__tests__/**/*.test.{ts,tsx}; do
-  if [[ ! -f "$f" ]]; then continue; fi
-  case "$(basename "$f")" in
-    useGitData.test.ts|realtimeUpdate.*|branchList.*)
-      mv "$f" "$f.skip"
-      ;;
-  esac
-done
-```
-
-## Conclusion
-
-The decision to skip these tests is pragmatic:
-
-- **Production code works correctly** (all skipped tests pass in isolation)
-- **Core functionality is tested** (307 stable tests remain)
-- **Test suite is reliable** (100% pass rate)
-- **CI/CD can proceed** (no flaky test failures)
-
-The skipped tests document known limitations of the test environment, not bugs in the implementation.

package/src/cli/ui/__tests__/acceptance/branchList.acceptance.test.tsx.skip

@@ -1,239 +0,0 @@
-/**
- * @vitest-environment happy-dom
- * Acceptance tests for User Story 1: Branch List Display and Selection
- */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render, waitFor } from '@testing-library/react';
-import React from 'react';
-import { App } from '../../components/App.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('Acceptance: Branch List (User Story 1)', () => {
-  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();
-  });
-
-  /**
-   * T047: Acceptance Scenario 1
-   * 1秒以内に全画面レイアウトが表示される
-   */
-  it('[AC1] should display full-screen layout within 1 second', 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 startTime = Date.now();
-    const onExit = vi.fn();
-    const { getByText, container } = render(<App onExit={onExit} />);
-
-    // Wait for full layout to be rendered
-    await waitFor(
-      () => {
-        expect(getByText(/Claude Worktree/i)).toBeDefined(); // Header
-        expect(getByText(/Local:/)).toBeDefined(); // Stats
-        expect(getByText(/main/)).toBeDefined(); // Branch list
-        expect(getByText(/Quit/i)).toBeDefined(); // Footer
-      },
-      { timeout: 1000 }
-    );
-
-    const renderTime = Date.now() - startTime;
-    expect(renderTime).toBeLessThan(1000); // Should render within 1 second
-  });
-
-  /**
-   * T048: Acceptance Scenario 2
-   * 20個以上のブランチでスクロールがスムーズに動作
-   */
-  it('[AC2] should handle smooth scrolling with 20+ branches', async () => {
-    // Generate 25 branches
-    const mockBranches: BranchInfo[] = Array.from({ length: 25 }, (_, i) => ({
-      name: `feature/branch-${i + 1}`,
-      type: 'local' as const,
-      branchType: 'feature' as const,
-      isCurrent: i === 0,
-    }));
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    // Wait for rendering
-    await waitFor(() => {
-      expect(container.textContent).toContain('feature/branch-1');
-    });
-
-    // Verify all branches are in the DOM (even if not visible)
-    // Note: Actual scrolling behavior is handled by ink-select-input
-    expect(container.textContent).toBeTruthy();
-  });
-
-  /**
-   * T049: Acceptance Scenario 3
-   * ターミナルリサイズで表示行数が自動調整される
-   */
-  it('[AC3] should adjust display rows on terminal resize', 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([]);
-
-    // Mock initial terminal size
-    const originalRows = process.stdout.rows;
-    process.stdout.rows = 30;
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    // Wait for initial render
-    await waitFor(() => {
-      expect(container.textContent).toContain('Claude Worktree');
-    });
-
-    // Simulate terminal resize
-    process.stdout.rows = 20;
-
-    // In a real terminal, this would trigger a resize event
-    // For this test, we just verify the component can handle different sizes
-    expect(container).toBeDefined();
-
-    // Restore original size
-    process.stdout.rows = originalRows;
-  });
-
-  /**
-   * T050: Acceptance Scenario 4
-   * ブランチ選択とEnterキーで処理開始
-   */
-  it('[AC4] should trigger onExit when branch is selected', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
-        isCurrent: true,
-      },
-      {
-        name: 'feature/test',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    // Wait for rendering
-    await waitFor(() => {
-      expect(container.textContent).toContain('main');
-    });
-
-    // Note: Actual key input simulation requires ink's input handling
-    // This test verifies that the onExit callback is properly wired
-    expect(onExit).toBeDefined();
-  });
-
-  /**
-   * T051: Acceptance Scenario 5
-   * qキーでアプリケーション終了
-   */
-  it('[AC5] should support quit action with q key', 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 onExit = vi.fn();
-    const { getAllByText } = render(<App onExit={onExit} />);
-
-    // Wait for rendering
-    await waitFor(() => {
-      expect(getAllByText(/Quit/i).length).toBeGreaterThan(0);
-    });
-
-    // Verify quit action is displayed
-    expect(getAllByText(/q/i).length).toBeGreaterThan(0);
-    expect(getAllByText(/Quit/i).length).toBeGreaterThan(0);
-
-    // Note: Actual 'q' key press requires ink's input handling
-    // This test verifies that the quit action is properly displayed
-  });
-
-  /**
-   * Additional: Performance test for large branch lists
-   */
-  it('[Performance] should handle 100+ branches efficiently', async () => {
-    // Generate 100 branches
-    const mockBranches: BranchInfo[] = Array.from({ length: 100 }, (_, i) => ({
-      name: `feature/branch-${i + 1}`,
-      type: 'local' as const,
-      branchType: 'feature' as const,
-      isCurrent: i === 0,
-    }));
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const startTime = Date.now();
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    // Wait for rendering
-    await waitFor(() => {
-      expect(container.textContent).toContain('feature/branch-1');
-    });
-
-    const renderTime = Date.now() - startTime;
-
-    // Should render 100 branches within reasonable time (< 2 seconds)
-    expect(renderTime).toBeLessThan(2000);
-  });
-});

package/src/cli/ui/__tests__/acceptance/navigation.acceptance.test.tsx

@@ -1,225 +0,0 @@
-/**
- * @vitest-environment happy-dom
- * Acceptance tests for User Story 2: Sub-screen Navigation
- */
-import { describe, it, expect, beforeEach, afterAll, vi } from "vitest";
-import type { Mock } from "vitest";
-import { render, waitFor } from "@testing-library/react";
-import React from "react";
-import { App } from "../../components/App.js";
-import { Window } from "happy-dom";
-import type { BranchInfo } from "../../types.js";
-
-// Mock git.ts and worktree.ts
-vi.mock("../../../../git.ts", () => ({
-  __esModule: true,
-  getAllBranches: vi.fn(),
-  getRepositoryRoot: vi.fn(async () => "/repo"),
-  deleteBranch: vi.fn(async () => undefined),
-}));
-
-const { acceptanceIsProtectedBranchName, acceptanceSwitchToProtectedBranch } =
-  vi.hoisted(() => ({
-    acceptanceIsProtectedBranchName: vi.fn(() => false),
-    acceptanceSwitchToProtectedBranch: vi.fn(async () => "none" as const),
-  }));
-
-vi.mock("../../../../worktree.ts", () => ({
-  __esModule: true,
-  listAdditionalWorktrees: vi.fn(),
-  createWorktree: vi.fn(async () => undefined),
-  generateWorktreePath: vi.fn(async () => "/repo/.git/worktree/test"),
-  getMergedPRWorktrees: vi.fn(async () => []),
-  removeWorktree: vi.fn(async () => undefined),
-  isProtectedBranchName: acceptanceIsProtectedBranchName,
-  switchToProtectedBranch: acceptanceSwitchToProtectedBranch,
-}));
-
-import {
-  getAllBranches,
-  getRepositoryRoot,
-  deleteBranch,
-} from "../../../../git.ts";
-import {
-  listAdditionalWorktrees,
-  createWorktree,
-  generateWorktreePath,
-  getMergedPRWorktrees,
-  removeWorktree,
-} from "../../../../worktree.ts";
-
-const mockedGetAllBranches = getAllBranches as Mock;
-const mockedGetRepositoryRoot = getRepositoryRoot as Mock;
-const mockedDeleteBranch = deleteBranch as Mock;
-const mockedListAdditionalWorktrees = listAdditionalWorktrees as Mock;
-const mockedCreateWorktree = createWorktree as Mock;
-const mockedGenerateWorktreePath = generateWorktreePath as Mock;
-const mockedGetMergedPRWorktrees = getMergedPRWorktrees as Mock;
-const mockedRemoveWorktree = removeWorktree as Mock;
-const mockedIsProtectedBranchName = acceptanceIsProtectedBranchName as Mock;
-const mockedSwitchToProtectedBranch = acceptanceSwitchToProtectedBranch as Mock;
-
-describe("Acceptance: Navigation (User Story 2)", () => {
-  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 mocks
-    mockedGetAllBranches.mockReset();
-    mockedListAdditionalWorktrees.mockReset();
-    mockedGetRepositoryRoot.mockReset();
-    mockedDeleteBranch.mockReset();
-    mockedCreateWorktree.mockReset();
-    mockedGenerateWorktreePath.mockReset();
-    mockedGetMergedPRWorktrees.mockReset();
-    mockedRemoveWorktree.mockReset();
-    mockedIsProtectedBranchName.mockReset();
-    mockedSwitchToProtectedBranch.mockReset();
-    mockedGetRepositoryRoot.mockResolvedValue("/repo");
-    mockedSwitchToProtectedBranch.mockResolvedValue("none");
-  });
-
-  const mockBranches: BranchInfo[] = [
-    {
-      name: "main",
-      type: "local",
-      branchType: "main",
-      isCurrent: true,
-    },
-    {
-      name: "feature/test",
-      type: "local",
-      branchType: "feature",
-      isCurrent: false,
-    },
-  ];
-
-  /**
-   * T074: Acceptance Scenario 1
-   * nキーで新規ブランチ作成画面に遷移
-   */
-  it("[AC1] should navigate to branch creator on n key", async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockBranches,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText, container } = render(<App onExit={onExit} />);
-
-    await waitFor(() => {
-      expect(getByText(/gwt - Branch Selection/i)).toBeDefined();
-    });
-
-    // Verify n key action is available in footer
-    const nKeyElements = container.querySelectorAll("*");
-    let hasNKey = false;
-    nKeyElements.forEach((el) => {
-      if (el.textContent?.toLowerCase().includes("new branch")) {
-        hasNKey = true;
-      }
-    });
-
-    expect(hasNKey || container.textContent?.toLowerCase().includes("n")).toBe(
-      true,
-    );
-  });
-
-  /**
-   * T075: Acceptance Scenario 2
-   * メイン画面にはqキーが存在しない（終了はCtrl+Cのみ）
-   */
-  it("[AC2] should not have q key on main screen", async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockBranches,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    await waitFor(() => {
-      expect(container).toBeDefined();
-    });
-
-    // Verify q key is NOT in the footer (main screen uses Ctrl+C for exit)
-    const footerText = container.textContent || "";
-    // Main screen should not have 'q' for quit, but should have other keys
-    expect(footerText.toLowerCase()).not.toMatch(/\[q\]/);
-    expect(footerText.toLowerCase()).toContain("enter");
-  });
-
-  /**
-   * T076: Acceptance Scenario 3
-   * ブランチリスト画面でフッターアクションが表示される
-   */
-  it("[AC3] should display footer actions on branch list screen", async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockBranches,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([
-      {
-        branch: "feature/test",
-        path: "/path/to/worktree",
-        head: "abc123",
-        isAccessible: true,
-      },
-    ]);
-
-    const onExit = vi.fn();
-    const { getByText, container } = render(<App onExit={onExit} />);
-
-    await waitFor(() => {
-      expect(getByText(/gwt - Branch Selection/i)).toBeDefined();
-    });
-
-    // Verify footer actions are available
-    const footerText = container.textContent || "";
-    expect(footerText.toLowerCase()).toContain("enter");
-  });
-
-  it("[Integration] should support all navigation keys", async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockBranches,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText, getAllByText } = render(<App onExit={onExit} />);
-
-    await waitFor(() => {
-      expect(getByText(/gwt - Branch Selection/i)).toBeDefined();
-    });
-
-    // Verify navigation keys are available (main screen doesn't have q key)
-    const enterKeys = getAllByText(/enter/i);
-
-    expect(enterKeys.length).toBeGreaterThan(0);
-  });
-
-  it("[Integration] should display correct footer actions", async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockBranches,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    await waitFor(() => {
-      expect(container).toBeDefined();
-    });
-
-    // Verify footer has multiple action keys (main screen doesn't have q key)
-    const footerText = container.textContent || "";
-    expect(footerText.toLowerCase()).toContain("enter");
-    expect(footerText.toLowerCase()).toContain("r"); // Refresh
-  });
-});
-
-afterAll(() => {
-  vi.restoreAllMocks();
-});