@akiojin/gwt 4.9.1 → 4.11.6

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

@@ -1,253 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, vi, afterEach } 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('Branch List Integration', () => {
-  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();
-  });
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('should render full application with branch list', 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 { getByText } = render(<App onExit={onExit} />);
-
-    // Wait for async data loading
-    await waitFor(() => {
-      expect(getByText(/Claude Worktree/i)).toBeDefined();
-    });
-
-    // Verify header
-    expect(getByText(/Claude Worktree/i)).toBeDefined();
-
-    // Verify stats
-    expect(getByText(/Local:/)).toBeDefined();
-
-    // Verify branches are displayed
-    expect(getByText(/main/)).toBeDefined();
-    expect(getByText(/feature\/test/)).toBeDefined();
-
-    // Verify footer
-    expect(getByText(/Quit/i)).toBeDefined();
-  });
-
-  it('should display statistics correctly', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
-        isCurrent: true,
-      },
-      {
-        name: 'feature/a',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-        worktree: {
-          path: '/path/a',
-          locked: false,
-          prunable: false,
-        },
-      },
-      {
-        name: 'origin/main',
-        type: 'remote',
-        branchType: 'main',
-        isCurrent: false,
-      },
-    ];
-
-    (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} />);
-
-    // Wait for async data loading
-    await waitFor(() => {
-      expect(getByText(/Local:/)).toBeDefined();
-    });
-
-    // Verify statistics
-    expect(getByText(/Local:/)).toBeDefined();
-    expect(getAllByText(/2/).length).toBeGreaterThan(0); // 2 local branches
-
-    expect(getByText(/Remote:/)).toBeDefined();
-    expect(getAllByText(/1/).length).toBeGreaterThan(0); // 1 remote branch + 1 worktree
-
-    expect(getByText(/Worktrees:/)).toBeDefined();
-  });
-
-  it('should handle empty branch list', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText } = render(<App onExit={onExit} />);
-
-    // Wait for async data loading
-    await waitFor(() => {
-      expect(getByText(/No branches found/i)).toBeDefined();
-    });
-
-    expect(getByText(/No branches found/i)).toBeDefined();
-  });
-
-  it('should handle loading state', async () => {
-    // Mock a slow response
-    (getAllBranches as ReturnType<typeof vi.fn>).mockImplementation(
-      () =>
-        new Promise((resolve) =>
-          setTimeout(() => resolve([]), 100)
-        )
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText, queryByText } = render(<App onExit={onExit} />);
-
-    // Initially should show loading
-    // Note: In happy-dom, useEffect may run synchronously, so we check for either loading or loaded state
-    const hasLoading = queryByText(/Loading/i);
-    if (hasLoading) {
-      expect(hasLoading).toBeDefined();
-    }
-
-    // Wait for data to load
-    await waitFor(() => {
-      expect(queryByText(/Loading/i)).toBeNull();
-    });
-  });
-
-  it('should handle error state', async () => {
-    const error = new Error('Failed to fetch branches');
-    (getAllBranches as ReturnType<typeof vi.fn>).mockRejectedValue(error);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText } = render(<App onExit={onExit} />);
-
-    // Wait for error to appear
-    await waitFor(() => {
-      expect(getByText(/Error:/i)).toBeDefined();
-    });
-
-    expect(getByText(/Error:/i)).toBeDefined();
-    expect(getByText(/Failed to fetch branches/i)).toBeDefined();
-  });
-
-  it('should display branch icons correctly', async () => {
-    const mockBranches: BranchInfo[] = [
-      {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
-        isCurrent: true,
-      },
-      {
-        name: 'feature/test',
-        type: 'local',
-        branchType: 'feature',
-        isCurrent: false,
-      },
-      {
-        name: 'hotfix/urgent',
-        type: 'local',
-        branchType: 'hotfix',
-        isCurrent: false,
-      },
-    ];
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-
-    const onExit = vi.fn();
-    const { getByText } = render(<App onExit={onExit} />);
-
-    // Wait for async data loading
-    await waitFor(() => {
-      expect(getByText(/⚡/)).toBeDefined();
-    });
-
-    // Check for branch type icons
-    expect(getByText(/⚡/)).toBeDefined(); // main icon
-    expect(getByText(/⭐/)).toBeDefined(); // current branch icon
-    expect(getByText(/✨/)).toBeDefined(); // feature icon
-    expect(getByText(/🔥/)).toBeDefined(); // hotfix icon
-  });
-
-  it('should integrate all components correctly', 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 { container } = render(<App onExit={onExit} />);
-
-    // Wait for rendering
-    await waitFor(() => {
-      expect(container.textContent).toContain('Claude Worktree');
-    });
-
-    // Verify all major sections are present
-    expect(container.textContent).toContain('Claude Worktree'); // Header
-    expect(container.textContent).toContain('Local:'); // Stats
-    expect(container.textContent).toContain('main'); // Branch list
-    expect(container.textContent).toContain('Quit'); // Footer
-  });
-});

package/src/cli/ui/__tests__/integration/edgeCases.test.tsx

@@ -1,436 +0,0 @@
-/**
- * @vitest-environment happy-dom
- * Edge case tests for UI components
- */
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import { render } from "@testing-library/react";
-import React from "react";
-import { App as AppComponent } from "../../components/App.js";
-import * as BranchListScreenModule from "../../components/screens/BranchListScreen.js";
-import type { BranchListScreenProps } from "../../components/screens/BranchListScreen.js";
-import { Window } from "happy-dom";
-import type { BranchInfo, BranchItem, Statistics } from "../../types.js";
-
-const mockRefresh = vi.fn();
-const branchListProps: BranchListScreenProps[] = [];
-const useGitDataMock = vi.fn();
-const useProfilesMock = vi.fn();
-const useToolStatusMock = vi.fn();
-const originalStdoutRows = process.stdout.rows;
-const originalStdoutColumns = process.stdout.columns;
-
-vi.mock("ink", async () => {
-  const ReactImport = await import("react");
-  const Box = ({ children }: { children?: ReactImport.ReactNode }) =>
-    ReactImport.createElement("div", null, children);
-  const Text = ({ children }: { children?: ReactImport.ReactNode }) =>
-    ReactImport.createElement("span", null, children);
-  return {
-    Box,
-    Text,
-    useApp: () => ({ exit: vi.fn() }),
-    useInput: () => {},
-    useStdout: () => ({ stdout: process.stdout, write: vi.fn() }),
-  };
-});
-
-vi.mock("../../hooks/useGitData.js", () => ({
-  useGitData: (...args: unknown[]) => useGitDataMock(...args),
-}));
-
-vi.mock("../../hooks/useProfiles.js", () => ({
-  useProfiles: (...args: unknown[]) => useProfilesMock(...args),
-}));
-
-vi.mock("../../hooks/useToolStatus.js", () => ({
-  useToolStatus: (...args: unknown[]) => useToolStatusMock(...args),
-}));
-
-vi.mock("../../../utils.js", () => ({
-  getPackageVersion: vi.fn(async () => "0.0.0-test"),
-}));
-
-vi.mock("../../../git.js", () => ({
-  getRepositoryRoot: vi.fn(async () => "/repo"),
-  deleteBranch: vi.fn(async () => undefined),
-}));
-
-vi.mock("../../../config/index.js", () => ({
-  loadSession: vi.fn(async () => null),
-}));
-
-vi.mock("../../components/screens/BranchListScreen.js", () => ({
-  BranchListScreen: (props: BranchListScreenProps) => {
-    branchListProps.push(props);
-    return (
-      <div>
-        <div>BranchList</div>
-        {props.error && <div>Error: {props.error.message}</div>}
-        <div>
-          Local:{props.stats?.localCount ?? 0} Remote:
-          {props.stats?.remoteCount ?? 0} Worktrees:
-          {props.stats?.worktreeCount ?? 0}
-        </div>
-        <ul>
-          {props.branches.map((b) => (
-            <li key={b.name}>{b.name}</li>
-          ))}
-        </ul>
-      </div>
-    );
-  },
-}));
-
-describe("Edge Cases Integration Tests", () => {
-  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
-    vi.clearAllMocks();
-    useGitDataMock.mockReset();
-    useProfilesMock.mockReset();
-    useToolStatusMock.mockReset();
-    branchListProps.length = 0;
-    useGitDataMock.mockReturnValue({
-      branches: [],
-      worktrees: [],
-      loading: false,
-      error: null,
-      refresh: mockRefresh,
-      lastUpdated: null,
-    });
-
-    useProfilesMock.mockReturnValue({
-      profiles: null,
-      loading: false,
-      error: null,
-      activeProfileName: null,
-      activeProfile: null,
-      refresh: vi.fn(),
-      setActiveProfile: vi.fn(),
-      createProfile: vi.fn(),
-      updateProfile: vi.fn(),
-      deleteProfile: vi.fn(),
-      updateEnvVar: vi.fn(),
-      deleteEnvVar: vi.fn(),
-    });
-
-    useToolStatusMock.mockReturnValue({
-      tools: [],
-      loading: false,
-      error: null,
-      refresh: vi.fn(),
-    });
-  });
-
-  afterEach(() => {
-    process.stdout.rows = originalStdoutRows;
-    process.stdout.columns = originalStdoutColumns;
-  });
-
-  /**
-   * T091: Terminal size極小（10行以下）の動作確認
-   */
-  it("[T091] should handle minimal terminal size (10 rows)", () => {
-    // Save original rows
-    const originalRows = process.stdout.rows;
-
-    // Set minimal terminal size
-    process.stdout.rows = 10;
-
-    const mockBranches: BranchItem[] = [
-      { name: "main", label: "main", value: "main" },
-      { name: "feature/a", label: "feature/a", value: "feature/a" },
-      { name: "feature/b", label: "feature/b", value: "feature/b" },
-    ];
-
-    const mockStats: Statistics = {
-      localCount: 3,
-      remoteCount: 0,
-      worktreeCount: 0,
-      changesCount: 0,
-      lastUpdated: new Date(),
-    };
-
-    const onSelect = vi.fn();
-    const { container } = render(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    // Should render without crashing
-    expect(container).toBeDefined();
-
-    // Restore original rows
-    process.stdout.rows = originalRows;
-  });
-
-  it("[T091] should handle extremely small terminal (5 rows)", () => {
-    const originalRows = process.stdout.rows;
-    process.stdout.rows = 5;
-
-    const mockBranches: BranchItem[] = [
-      { name: "main", label: "main", value: "main" },
-    ];
-
-    const mockStats: Statistics = {
-      localCount: 1,
-      remoteCount: 0,
-      worktreeCount: 0,
-      changesCount: 0,
-      lastUpdated: new Date(),
-    };
-
-    const onSelect = vi.fn();
-    const { container } = render(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(container).toBeDefined();
-    expect(branchListProps.at(-1)?.branches).toHaveLength(1);
-
-    process.stdout.rows = originalRows;
-  });
-
-  /**
-   * T092: 非常に長いブランチ名の表示確認
-   */
-  it("[T092] should handle very long branch names", () => {
-    const longBranchName =
-      "feature/very-long-branch-name-that-exceeds-normal-terminal-width-and-should-be-handled-gracefully";
-
-    const mockBranches: BranchItem[] = [
-      {
-        name: "main",
-        label: "main",
-        value: "main",
-        latestCommitTimestamp: 1_700_000_000,
-      },
-      {
-        name: longBranchName,
-        label: longBranchName,
-        value: longBranchName,
-        latestCommitTimestamp: 1_700_000_600,
-      },
-    ];
-
-    const mockStats: Statistics = {
-      localCount: 2,
-      remoteCount: 0,
-      worktreeCount: 0,
-      changesCount: 0,
-      lastUpdated: new Date(),
-    };
-
-    const onSelect = vi.fn();
-    const { container } = render(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(container.textContent).toContain(longBranchName);
-    expect(
-      branchListProps.at(-1)?.branches?.some((b) => b.name === longBranchName),
-    ).toBe(true);
-  });
-
-  it("[T092] should handle branch names with special characters", () => {
-    const specialBranchNames = [
-      "feature/bug-fix-#123",
-      "hotfix/issue@456",
-      "release/v1.0.0-beta.1",
-      "feature/改善-日本語",
-    ];
-
-    const mockBranches: BranchItem[] = specialBranchNames.map(
-      (name, index) => ({
-        name,
-        label: name,
-        value: name,
-        latestCommitTimestamp: 1_700_001_000 + index * 60,
-      }),
-    );
-
-    const mockStats: Statistics = {
-      localCount: mockBranches.length,
-      remoteCount: 0,
-      worktreeCount: 0,
-      changesCount: 0,
-      lastUpdated: new Date(),
-    };
-
-    const onSelect = vi.fn();
-    const { container } = render(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    // All special branch names should be displayed
-    specialBranchNames.forEach((name) => {
-      expect(container.textContent).toContain(name);
-    });
-  });
-
-  /**
-   * T093: Error Boundary動作確認
-   * Note: Skipped because React 18 with async useEffect makes error boundary
-   * testing unreliable in testing-library. The error is thrown but not caught
-   * by the test framework correctly.
-   */
-  it.skip("[T093] should catch errors in App component", () => {
-    // Mock useGitData to throw an error after initial render
-    let callCount = 0;
-    useGitDataMock.mockImplementation(() => {
-      callCount++;
-      if (callCount > 1) {
-        throw new Error("Simulated error");
-      }
-      return {
-        branches: [],
-        worktrees: [],
-        loading: false,
-        error: null,
-        refresh: mockRefresh,
-        lastUpdated: null,
-      };
-    });
-
-    const onExit = vi.fn();
-    const { container } = render(<AppComponent onExit={onExit} />);
-
-    // Initial render should work
-    expect(container).toBeDefined();
-  });
-
-  it("[T093] should display error message when data loading fails", () => {
-    const testError = new Error("Test error: Failed to load Git data");
-    useGitDataMock.mockReturnValue({
-      branches: [],
-      worktrees: [],
-      loading: false,
-      error: testError,
-      refresh: mockRefresh,
-      lastUpdated: null,
-    });
-
-    const onExit = vi.fn();
-    const { getByText } = render(<AppComponent onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    expect(branchListProps.at(-1)?.error).toBe(testError);
-    // Error should be displayed via stubbed screen
-    expect(getByText(/Error:/i)).toBeDefined();
-    expect(getByText(/Failed to load Git data/i)).toBeDefined();
-  });
-
-  it("[T093] should handle empty branches list gracefully", () => {
-    useGitDataMock.mockReturnValue({
-      branches: [],
-      worktrees: [],
-      loading: false,
-      error: null,
-      refresh: mockRefresh,
-      lastUpdated: null,
-    });
-
-    const onExit = vi.fn();
-    const { container } = render(<AppComponent onExit={onExit} />);
-
-    // Should render without error even with no branches
-    expect(container).toBeDefined();
-  });
-
-  /**
-   * Additional edge cases
-   */
-  it("should handle large number of worktrees", () => {
-    const mockBranches: BranchInfo[] = Array.from({ length: 50 }, (_, i) => ({
-      name: `feature/branch-${i}`,
-      type: "local" as const,
-      branchType: "feature" as const,
-      isCurrent: false,
-    }));
-
-    useGitDataMock.mockReturnValue({
-      branches: mockBranches,
-      worktrees: Array.from({ length: 30 }, (_, i) => ({
-        branch: `feature/branch-${i}`,
-        path: `/path/to/worktree-${i}`,
-        head: `commit-${i}`,
-        isAccessible: true,
-      })),
-      loading: false,
-      error: null,
-      refresh: mockRefresh,
-      lastUpdated: new Date(),
-    });
-
-    const onExit = vi.fn();
-    const { container } = render(<AppComponent onExit={onExit} />);
-
-    expect(container).toBeDefined();
-  });
-
-  it("should handle terminal resize gracefully", () => {
-    const originalRows = process.stdout.rows;
-
-    // Start with normal size
-    process.stdout.rows = 30;
-
-    const mockBranches: BranchItem[] = [
-      { name: "main", label: "main", value: "main" },
-    ];
-
-    const mockStats: Statistics = {
-      localCount: 1,
-      remoteCount: 0,
-      worktreeCount: 0,
-      changesCount: 0,
-      lastUpdated: new Date(),
-    };
-
-    const onSelect = vi.fn();
-    const { container, rerender } = render(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(container).toBeDefined();
-
-    // Simulate terminal resize
-    process.stdout.rows = 15;
-
-    // Re-render
-    rerender(
-      <BranchListScreenModule.BranchListScreen
-        branches={mockBranches}
-        stats={mockStats}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(container).toBeDefined();
-
-    process.stdout.rows = originalRows;
-  });
-});