@akiojin/gwt 2.2.0 → 2.3.0

package/src/cli/ui/__tests__/components/screens/WorktreeManagerScreen.test.tsx

@@ -1,13 +1,13 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { WorktreeManagerScreen } from '../../../components/screens/WorktreeManagerScreen.js';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { WorktreeManagerScreen } from "../../../components/screens/WorktreeManagerScreen.js";
+import { Window } from "happy-dom";
 
-describe('WorktreeManagerScreen', () => {
+describe("WorktreeManagerScreen", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -17,69 +17,85 @@ describe('WorktreeManagerScreen', () => {
 
   const mockWorktrees = [
     {
-      branch: 'feature/test-1',
-      path: '/path/to/worktree-1',
+      branch: "feature/test-1",
+      path: "/path/to/worktree-1",
       isAccessible: true,
     },
     {
-      branch: 'feature/test-2',
-      path: '/path/to/worktree-2',
+      branch: "feature/test-2",
+      path: "/path/to/worktree-2",
       isAccessible: true,
     },
   ];
 
-  it('should render header with title', () => {
+  it("should render header with title", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <WorktreeManagerScreen worktrees={mockWorktrees} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={mockWorktrees}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/Worktree Manager/i)).toBeDefined();
   });
 
-  it('should render worktree list', () => {
+  it("should render worktree list", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <WorktreeManagerScreen worktrees={mockWorktrees} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={mockWorktrees}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/feature\/test-1/)).toBeDefined();
     expect(getByText(/feature\/test-2/)).toBeDefined();
   });
 
-  it('should render footer with actions', () => {
+  it("should render footer with actions", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getAllByText } = render(
-      <WorktreeManagerScreen worktrees={mockWorktrees} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={mockWorktrees}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
     expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
   });
 
-  it('should handle empty worktree list', () => {
+  it("should handle empty worktree list", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText } = render(
-      <WorktreeManagerScreen worktrees={[]} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={[]}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(getByText(/No worktrees found/i)).toBeDefined();
   });
 
-  it('should display inaccessible worktrees differently', () => {
+  it("should display inaccessible worktrees differently", () => {
     const worktreesWithInaccessible = [
       {
-        branch: 'feature/accessible',
-        path: '/path/accessible',
+        branch: "feature/accessible",
+        path: "/path/accessible",
         isAccessible: true,
       },
       {
-        branch: 'feature/inaccessible',
-        path: '/path/inaccessible',
+        branch: "feature/inaccessible",
+        path: "/path/inaccessible",
         isAccessible: false,
       },
     ];
@@ -91,21 +107,25 @@ describe('WorktreeManagerScreen', () => {
         worktrees={worktreesWithInaccessible}
         onBack={onBack}
         onSelect={onSelect}
-      />
+      />,
     );
 
     expect(getByText(/feature\/accessible/)).toBeDefined();
     expect(getByText(/feature\/inaccessible/)).toBeDefined();
   });
 
-  it('should use terminal height for layout calculation', () => {
+  it("should use terminal height for layout calculation", () => {
     const originalRows = process.stdout.rows;
     process.stdout.rows = 30;
 
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { container } = render(
-      <WorktreeManagerScreen worktrees={mockWorktrees} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={mockWorktrees}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     expect(container).toBeDefined();
@@ -113,11 +133,15 @@ describe('WorktreeManagerScreen', () => {
     process.stdout.rows = originalRows;
   });
 
-  it('should display worktree count in stats', () => {
+  it("should display worktree count in stats", () => {
     const onBack = vi.fn();
     const onSelect = vi.fn();
     const { getByText, getAllByText } = render(
-      <WorktreeManagerScreen worktrees={mockWorktrees} onBack={onBack} onSelect={onSelect} />
+      <WorktreeManagerScreen
+        worktrees={mockWorktrees}
+        onBack={onBack}
+        onSelect={onSelect}
+      />,
     );
 
     // Check for worktree count

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

@@ -2,20 +2,28 @@
  * @vitest-environment happy-dom
  * Edge case tests for UI components
  */
-import { describe, it, expect, beforeEach, afterEach, afterAll, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { App } from '../../components/App.js';
-import { BranchListScreen } from '../../components/screens/BranchListScreen.js';
-import { Window } from 'happy-dom';
-import type { BranchInfo, BranchItem, Statistics } from '../../types.js';
-import * as useGitDataModule from '../../hooks/useGitData.js';
+import {
+  describe,
+  it,
+  expect,
+  beforeEach,
+  afterEach,
+  afterAll,
+  vi,
+} from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { App } from "../../components/App.js";
+import { BranchListScreen } from "../../components/screens/BranchListScreen.js";
+import { Window } from "happy-dom";
+import type { BranchInfo, BranchItem, Statistics } from "../../types.js";
+import * as useGitDataModule from "../../hooks/useGitData.js";
 
 const mockRefresh = vi.fn();
 const originalUseGitData = useGitDataModule.useGitData;
-const useGitDataSpy = vi.spyOn(useGitDataModule, 'useGitData');
+const useGitDataSpy = vi.spyOn(useGitDataModule, "useGitData");
 
-describe('Edge Cases Integration Tests', () => {
+describe("Edge Cases Integration Tests", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -31,7 +39,7 @@ describe('Edge Cases Integration Tests', () => {
   /**
    * T091: Terminal size極小（10行以下）の動作確認
    */
-  it('[T091] should handle minimal terminal size (10 rows)', () => {
+  it("[T091] should handle minimal terminal size (10 rows)", () => {
     // Save original rows
     const originalRows = process.stdout.rows;
 
@@ -39,9 +47,9 @@ describe('Edge Cases Integration Tests', () => {
     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' },
+      { 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 = {
@@ -54,7 +62,11 @@ describe('Edge Cases Integration Tests', () => {
 
     const onSelect = vi.fn();
     const { container } = render(
-      <BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
     );
 
     // Should render without crashing
@@ -64,12 +76,12 @@ describe('Edge Cases Integration Tests', () => {
     process.stdout.rows = originalRows;
   });
 
-  it('[T091] should handle extremely small terminal (5 rows)', () => {
+  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' },
+      { name: "main", label: "main", value: "main" },
     ];
 
     const mockStats: Statistics = {
@@ -82,7 +94,11 @@ describe('Edge Cases Integration Tests', () => {
 
     const onSelect = vi.fn();
     const { getByText } = render(
-      <BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
     );
 
     // Header should still be visible
@@ -94,15 +110,15 @@ describe('Edge Cases Integration Tests', () => {
   /**
    * T092: 非常に長いブランチ名の表示確認
    */
-  it('[T092] should handle very long branch names', () => {
+  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';
+      "feature/very-long-branch-name-that-exceeds-normal-terminal-width-and-should-be-handled-gracefully";
 
     const mockBranches: BranchItem[] = [
       {
-        name: 'main',
-        label: 'main',
-        value: 'main',
+        name: "main",
+        label: "main",
+        value: "main",
         latestCommitTimestamp: 1_700_000_000,
       },
       {
@@ -123,27 +139,33 @@ describe('Edge Cases Integration Tests', () => {
 
     const onSelect = vi.fn();
     const { container } = render(
-      <BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
     );
 
     // Long branch name should be displayed (Ink will handle wrapping/truncation)
     expect(container.textContent).toMatch(/\d{4}-\d{2}-\d{2} \d{2}:\d{2}/);
   });
 
-  it('[T092] should handle branch names with special characters', () => {
+  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/改善-日本語',
+      "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 mockBranches: BranchItem[] = specialBranchNames.map(
+      (name, index) => ({
+        name,
+        label: name,
+        value: name,
+        latestCommitTimestamp: 1_700_001_000 + index * 60,
+      }),
+    );
 
     const mockStats: Statistics = {
       localCount: mockBranches.length,
@@ -155,7 +177,11 @@ describe('Edge Cases Integration Tests', () => {
 
     const onSelect = vi.fn();
     const { container } = render(
-      <BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
     );
 
     // All special branch names should be displayed
@@ -167,13 +193,13 @@ describe('Edge Cases Integration Tests', () => {
   /**
    * T093: Error Boundary動作確認
    */
-  it('[T093] should catch errors in App component', async () => {
+  it("[T093] should catch errors in App component", async () => {
     // Mock useGitData to throw an error after initial render
     let callCount = 0;
     useGitDataSpy.mockImplementation(() => {
       callCount++;
       if (callCount > 1) {
-        throw new Error('Simulated error');
+        throw new Error("Simulated error");
       }
       return {
         branches: [],
@@ -192,8 +218,8 @@ describe('Edge Cases Integration Tests', () => {
     expect(container).toBeDefined();
   });
 
-  it('[T093] should display error message when data loading fails', () => {
-    const testError = new Error('Test error: Failed to load Git data');
+  it("[T093] should display error message when data loading fails", () => {
+    const testError = new Error("Test error: Failed to load Git data");
     useGitDataSpy.mockReturnValue({
       branches: [],
       worktrees: [],
@@ -211,7 +237,7 @@ describe('Edge Cases Integration Tests', () => {
     expect(getByText(/Failed to load Git data/i)).toBeDefined();
   });
 
-  it('[T093] should handle empty branches list gracefully', () => {
+  it("[T093] should handle empty branches list gracefully", () => {
     useGitDataSpy.mockReturnValue({
       branches: [],
       worktrees: [],
@@ -231,11 +257,11 @@ describe('Edge Cases Integration Tests', () => {
   /**
    * Additional edge cases
    */
-  it('should handle large number of worktrees', () => {
+  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,
+      type: "local" as const,
+      branchType: "feature" as const,
       isCurrent: false,
     }));
 
@@ -259,14 +285,14 @@ describe('Edge Cases Integration Tests', () => {
     expect(container).toBeDefined();
   });
 
-  it('should handle terminal resize gracefully', () => {
+  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' },
+      { name: "main", label: "main", value: "main" },
     ];
 
     const mockStats: Statistics = {
@@ -279,7 +305,11 @@ describe('Edge Cases Integration Tests', () => {
 
     const onSelect = vi.fn();
     const { container, rerender } = render(
-      <BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
     );
 
     expect(container).toBeDefined();
@@ -288,7 +318,13 @@ describe('Edge Cases Integration Tests', () => {
     process.stdout.rows = 15;
 
     // Re-render
-    rerender(<BranchListScreen branches={mockBranches} stats={mockStats} onSelect={onSelect} />);
+    rerender(
+      <BranchListScreen
+        branches={mockBranches}
+        stats={mockStats}
+        onSelect={onSelect}
+      />,
+    );
 
     expect(container).toBeDefined();