@akiojin/gwt 2.2.0 → 2.4.0

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

@@ -2,46 +2,51 @@
  * @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';
+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', () => ({
+vi.mock("../../../../git.ts", () => ({
   __esModule: true,
   getAllBranches: vi.fn(),
-  getRepositoryRoot: vi.fn(async () => '/repo'),
+  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),
-}));
+const { acceptanceIsProtectedBranchName, acceptanceSwitchToProtectedBranch } =
+  vi.hoisted(() => ({
+    acceptanceIsProtectedBranchName: vi.fn(() => false),
+    acceptanceSwitchToProtectedBranch: vi.fn(async () => "none" as const),
+  }));
 
-vi.mock('../../../../worktree.ts', () => ({
+vi.mock("../../../../worktree.ts", () => ({
   __esModule: true,
   listAdditionalWorktrees: vi.fn(),
   createWorktree: vi.fn(async () => undefined),
-  generateWorktreePath: vi.fn(async () => '/repo/.git/worktree/test'),
+  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 {
+  getAllBranches,
+  getRepositoryRoot,
+  deleteBranch,
+} from "../../../../git.ts";
 import {
   listAdditionalWorktrees,
   createWorktree,
   generateWorktreePath,
   getMergedPRWorktrees,
   removeWorktree,
-} from '../../../../worktree.ts';
+} from "../../../../worktree.ts";
 
 const mockedGetAllBranches = getAllBranches as Mock;
 const mockedGetRepositoryRoot = getRepositoryRoot as Mock;
@@ -54,7 +59,7 @@ const mockedRemoveWorktree = removeWorktree as Mock;
 const mockedIsProtectedBranchName = acceptanceIsProtectedBranchName as Mock;
 const mockedSwitchToProtectedBranch = acceptanceSwitchToProtectedBranch as Mock;
 
-describe('Acceptance: Navigation (User Story 2)', () => {
+describe("Acceptance: Navigation (User Story 2)", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -72,21 +77,21 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     mockedRemoveWorktree.mockReset();
     mockedIsProtectedBranchName.mockReset();
     mockedSwitchToProtectedBranch.mockReset();
-    mockedGetRepositoryRoot.mockResolvedValue('/repo');
-    mockedSwitchToProtectedBranch.mockResolvedValue('none');
+    mockedGetRepositoryRoot.mockResolvedValue("/repo");
+    mockedSwitchToProtectedBranch.mockResolvedValue("none");
   });
 
   const mockBranches: BranchInfo[] = [
     {
-      name: 'main',
-      type: 'local',
-      branchType: 'main',
+      name: "main",
+      type: "local",
+      branchType: "main",
       isCurrent: true,
     },
     {
-      name: 'feature/test',
-      type: 'local',
-      branchType: 'feature',
+      name: "feature/test",
+      type: "local",
+      branchType: "feature",
       isCurrent: false,
     },
   ];
@@ -95,8 +100,10 @@ describe('Acceptance: Navigation (User Story 2)', () => {
    * T074: Acceptance Scenario 1
    * nキーで新規ブランチ作成画面に遷移
    */
-  it('[AC1] should navigate to branch creator on n key', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
+  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();
@@ -107,23 +114,27 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     });
 
     // Verify n key action is available in footer
-    const nKeyElements = container.querySelectorAll('*');
+    const nKeyElements = container.querySelectorAll("*");
     let hasNKey = false;
     nKeyElements.forEach((el) => {
-      if (el.textContent?.toLowerCase().includes('new branch')) {
+      if (el.textContent?.toLowerCase().includes("new branch")) {
         hasNKey = true;
       }
     });
 
-    expect(hasNKey || container.textContent?.toLowerCase().includes('n')).toBe(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);
+  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();
@@ -134,23 +145,25 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     });
 
     // Verify q key is NOT in the footer (main screen uses Ctrl+C for exit)
-    const footerText = container.textContent || '';
+    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');
+    expect(footerText.toLowerCase()).toContain("enter");
   });
 
   /**
    * T076: Acceptance Scenario 3
    * Worktree管理でアクション実行後に適切に遷移
    */
-  it('[AC3] should handle worktree management navigation', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
+  it("[AC3] should handle worktree management navigation", 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',
+        branch: "feature/test",
+        path: "/path/to/worktree",
+        head: "abc123",
         isAccessible: true,
       },
     ]);
@@ -163,19 +176,23 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     });
 
     // Verify m key action is available for worktree management
-    const mKeyElements = container.querySelectorAll('*');
+    const mKeyElements = container.querySelectorAll("*");
     let hasMKey = false;
     mKeyElements.forEach((el) => {
-      if (el.textContent?.toLowerCase().includes('manage worktrees')) {
+      if (el.textContent?.toLowerCase().includes("manage worktrees")) {
         hasMKey = true;
       }
     });
 
-    expect(hasMKey || container.textContent?.toLowerCase().includes('m')).toBe(true);
+    expect(hasMKey || container.textContent?.toLowerCase().includes("m")).toBe(
+      true,
+    );
   });
 
-  it('[Integration] should support all navigation keys', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
+  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();
@@ -191,8 +208,10 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     expect(enterKeys.length).toBeGreaterThan(0);
   });
 
-  it('[Integration] should display correct footer actions', async () => {
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue(mockBranches);
+  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();
@@ -203,9 +222,9 @@ describe('Acceptance: Navigation (User Story 2)', () => {
     });
 
     // 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('m'); // Manage worktrees
+    const footerText = container.textContent || "";
+    expect(footerText.toLowerCase()).toContain("enter");
+    expect(footerText.toLowerCase()).toContain("m"); // Manage worktrees
   });
 });
 

package/src/cli/ui/__tests__/components/App.protected-branch.test.tsx

@@ -1,19 +1,27 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, afterEach, afterAll, vi } from 'vitest';
-import { act, render } from '@testing-library/react';
-import React from 'react';
-import { Window } from 'happy-dom';
-import { App } from '../../components/App.js';
-import type { BranchInfo, BranchItem } from '../../types.js';
-import * as useGitDataModule from '../../hooks/useGitData.js';
-import * as useScreenStateModule from '../../hooks/useScreenState.js';
-import * as BranchListScreenModule from '../../components/screens/BranchListScreen.js';
-import * as BranchActionSelectorScreenModule from '../../screens/BranchActionSelectorScreen.js';
-import * as worktreeModule from '../../../../worktree.ts';
-import * as gitModule from '../../../../git.ts';
-import type { ScreenType } from '../../types.js';
+import {
+  describe,
+  it,
+  expect,
+  beforeEach,
+  afterEach,
+  afterAll,
+  vi,
+} from "vitest";
+import { act, render } from "@testing-library/react";
+import React from "react";
+import { Window } from "happy-dom";
+import { App } from "../../components/App.js";
+import type { BranchInfo, BranchItem } from "../../types.js";
+import * as useGitDataModule from "../../hooks/useGitData.js";
+import * as useScreenStateModule from "../../hooks/useScreenState.js";
+import * as BranchListScreenModule from "../../components/screens/BranchListScreen.js";
+import * as BranchActionSelectorScreenModule from "../../screens/BranchActionSelectorScreen.js";
+import * as worktreeModule from "../../../../worktree.ts";
+import * as gitModule from "../../../../git.ts";
+import type { ScreenType } from "../../types.js";
 
 const navigateToMock = vi.fn();
 const goBackMock = vi.fn();
@@ -22,37 +30,47 @@ const resetMock = vi.fn();
 const originalUseGitData = useGitDataModule.useGitData;
 const originalUseScreenState = useScreenStateModule.useScreenState;
 const originalBranchListScreen = BranchListScreenModule.BranchListScreen;
-const originalBranchActionSelector = BranchActionSelectorScreenModule.BranchActionSelectorScreen;
+const originalBranchActionSelector =
+  BranchActionSelectorScreenModule.BranchActionSelectorScreen;
 const originalGetRepositoryRoot = gitModule.getRepositoryRoot;
 
-const useGitDataSpy = vi.spyOn(useGitDataModule, 'useGitData');
-const useScreenStateSpy = vi.spyOn(useScreenStateModule, 'useScreenState');
-const branchListScreenSpy = vi.spyOn(BranchListScreenModule, 'BranchListScreen');
-const branchActionSelectorSpy = vi.spyOn(BranchActionSelectorScreenModule, 'BranchActionSelectorScreen');
-const switchToProtectedBranchSpy = vi.spyOn(worktreeModule, 'switchToProtectedBranch');
-const getRepositoryRootSpy = vi.spyOn(gitModule, 'getRepositoryRoot');
+const useGitDataSpy = vi.spyOn(useGitDataModule, "useGitData");
+const useScreenStateSpy = vi.spyOn(useScreenStateModule, "useScreenState");
+const branchListScreenSpy = vi.spyOn(
+  BranchListScreenModule,
+  "BranchListScreen",
+);
+const branchActionSelectorSpy = vi.spyOn(
+  BranchActionSelectorScreenModule,
+  "BranchActionSelectorScreen",
+);
+const switchToProtectedBranchSpy = vi.spyOn(
+  worktreeModule,
+  "switchToProtectedBranch",
+);
+const getRepositoryRootSpy = vi.spyOn(gitModule, "getRepositoryRoot");
 
 const branchListProps: any[] = [];
 const branchActionProps: any[] = [];
 const aiToolProps: any[] = [];
 let currentScreenState: ScreenType;
 
-vi.mock('../../components/screens/AIToolSelectorScreen.js', () => {
+vi.mock("../../components/screens/AIToolSelectorScreen.js", () => {
   return {
     AIToolSelectorScreen: (props: unknown) => {
       aiToolProps.push(props);
-      return React.createElement('div');
+      return React.createElement("div");
     },
   };
 });
 
-describe('App protected branch handling', () => {
+describe("App protected branch handling", () => {
   beforeEach(() => {
     const window = new Window();
     globalThis.window = window as any;
     globalThis.document = window.document as any;
 
-    currentScreenState = 'branch-list';
+    currentScreenState = "branch-list";
     navigateToMock.mockReset();
     goBackMock.mockReset();
     resetMock.mockReset();
@@ -73,7 +91,7 @@ describe('App protected branch handling', () => {
       goBack: goBackMock,
       reset: () => {
         resetMock();
-        currentScreenState = 'branch-list';
+        currentScreenState = "branch-list";
       },
     }));
 
@@ -85,8 +103,8 @@ describe('App protected branch handling', () => {
       branchActionProps.push(props);
       return React.createElement(originalBranchActionSelector, props);
     });
-    switchToProtectedBranchSpy.mockResolvedValue('local');
-    getRepositoryRootSpy.mockResolvedValue('/repo');
+    switchToProtectedBranchSpy.mockResolvedValue("local");
+    getRepositoryRootSpy.mockResolvedValue("/repo");
   });
 
   afterEach(() => {
@@ -95,7 +113,9 @@ describe('App protected branch handling', () => {
     useScreenStateSpy.mockReset();
     useScreenStateSpy.mockImplementation(originalUseScreenState);
     branchListScreenSpy.mockImplementation(originalBranchListScreen as any);
-    branchActionSelectorSpy.mockImplementation(originalBranchActionSelector as any);
+    branchActionSelectorSpy.mockImplementation(
+      originalBranchActionSelector as any,
+    );
     switchToProtectedBranchSpy.mockReset();
     getRepositoryRootSpy.mockReset();
     branchActionProps.length = 0;
@@ -110,18 +130,18 @@ describe('App protected branch handling', () => {
     getRepositoryRootSpy.mockRestore();
   });
 
-  it('shows protected branch warning and switches root without launching AI tool', async () => {
+  it("shows protected branch warning and switches root without launching AI tool", async () => {
     const branches: BranchInfo[] = [
       {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
+        name: "main",
+        type: "local",
+        branchType: "main",
         isCurrent: false,
       },
       {
-        name: 'feature/example',
-        type: 'local',
-        branchType: 'feature',
+        name: "feature/example",
+        type: "local",
+        branchType: "feature",
         isCurrent: true,
       },
     ];
@@ -141,15 +161,15 @@ describe('App protected branch handling', () => {
     const latestProps = branchListProps.at(-1);
     expect(latestProps).toBeDefined();
     if (!latestProps) {
-      throw new Error('BranchListScreen props missing');
+      throw new Error("BranchListScreen props missing");
     }
 
     const protectedBranch = (latestProps.branches as BranchItem[]).find(
-      (item) => item.name === 'main'
+      (item) => item.name === "main",
     );
     expect(protectedBranch).toBeDefined();
     if (!protectedBranch) {
-      throw new Error('Protected branch item not found');
+      throw new Error("Protected branch item not found");
     }
 
     await act(async () => {
@@ -157,13 +177,15 @@ describe('App protected branch handling', () => {
       await Promise.resolve();
     });
 
-    expect(navigateToMock).toHaveBeenCalledWith('branch-action-selector');
+    expect(navigateToMock).toHaveBeenCalledWith("branch-action-selector");
     expect(branchActionProps).not.toHaveLength(0);
     const actionProps = branchActionProps.at(-1);
-    expect(actionProps?.mode).toBe('protected');
-    expect(actionProps?.infoMessage).toContain('is a root branch');
-    expect(actionProps?.primaryLabel).toBe('Use root branch (no worktree)');
-    expect(actionProps?.secondaryLabel).toBe('Create new branch from this branch');
+    expect(actionProps?.mode).toBe("protected");
+    expect(actionProps?.infoMessage).toContain("is a root branch");
+    expect(actionProps?.primaryLabel).toBe("Use root branch (no worktree)");
+    expect(actionProps?.secondaryLabel).toBe(
+      "Create new branch from this branch",
+    );
 
     await act(async () => {
       actionProps?.onUseExisting();
@@ -172,12 +194,12 @@ describe('App protected branch handling', () => {
     });
 
     expect(switchToProtectedBranchSpy).toHaveBeenCalledWith({
-      branchName: 'main',
+      branchName: "main",
       repoRoot: expect.any(String),
       remoteRef: null,
     });
 
-    expect(navigateToMock).toHaveBeenCalledWith('ai-tool-selector');
+    expect(navigateToMock).toHaveBeenCalledWith("ai-tool-selector");
     expect(aiToolProps).not.toHaveLength(0);
   });
 });