@akiojin/gwt 2.2.0 → 2.3.0

package/src/cli/ui/__tests__/components/App.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 { App } from '../../components/App.js';
-import { Window } from 'happy-dom';
-import type { BranchInfo } from '../../types.js';
-import * as useGitDataModule from '../../hooks/useGitData.js';
+import {
+  describe,
+  it,
+  expect,
+  beforeEach,
+  afterEach,
+  afterAll,
+  vi,
+} from "vitest";
+import { act, render } 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 * 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('App', () => {
+describe("App", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -27,20 +35,20 @@ describe('App', () => {
 
   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,
     },
   ];
 
-  it('should render BranchListScreen when data is loaded', () => {
+  it("should render BranchListScreen when data is loaded", () => {
     useGitDataSpy.mockReturnValue({
       branches: mockBranches,
       loading: false,
@@ -58,7 +66,7 @@ describe('App', () => {
     expect(getByText(/feature\/test/)).toBeDefined();
   });
 
-  it('should show loading state initially', async () => {
+  it("should show loading state initially", async () => {
     useGitDataSpy.mockReturnValue({
       branches: [],
       loading: true,
@@ -69,7 +77,7 @@ describe('App', () => {
 
     const onExit = vi.fn();
     const { queryByText, getByText } = render(
-      <App onExit={onExit} loadingIndicatorDelay={10} />
+      <App onExit={onExit} loadingIndicatorDelay={10} />,
     );
 
     expect(queryByText(/Loading Git information/i)).toBeNull();
@@ -81,8 +89,8 @@ describe('App', () => {
     expect(getByText(/Loading Git information/i)).toBeDefined();
   });
 
-  it('should show error state when Git data fails to load', () => {
-    const error = new Error('Failed to fetch branches');
+  it("should show error state when Git data fails to load", () => {
+    const error = new Error("Failed to fetch branches");
     useGitDataSpy.mockReturnValue({
       branches: [],
       loading: false,
@@ -98,29 +106,29 @@ describe('App', () => {
     expect(getByText(/Failed to fetch branches/i)).toBeDefined();
   });
 
-  it('should calculate statistics from branches', () => {
+  it("should calculate statistics from branches", () => {
     const branchesWithWorktree: BranchInfo[] = [
       {
-        name: 'main',
-        type: 'local',
-        branchType: 'main',
+        name: "main",
+        type: "local",
+        branchType: "main",
         isCurrent: true,
       },
       {
-        name: 'feature/a',
-        type: 'local',
-        branchType: 'feature',
+        name: "feature/a",
+        type: "local",
+        branchType: "feature",
         isCurrent: false,
         worktree: {
-          path: '/path/a',
+          path: "/path/a",
           locked: false,
           prunable: false,
         },
       },
       {
-        name: 'origin/main',
-        type: 'remote',
-        branchType: 'main',
+        name: "origin/main",
+        type: "remote",
+        branchType: "main",
         isCurrent: false,
       },
     ];
@@ -144,7 +152,7 @@ describe('App', () => {
     expect(getByText(/Worktrees:/)).toBeDefined();
   });
 
-  it('should call onExit when branch is selected', () => {
+  it("should call onExit when branch is selected", () => {
     useGitDataSpy.mockReturnValue({
       branches: mockBranches,
       loading: false,
@@ -161,7 +169,7 @@ describe('App', () => {
     // which is covered in integration tests
   });
 
-  it('should handle empty branch list', () => {
+  it("should handle empty branch list", () => {
     useGitDataSpy.mockReturnValue({
       branches: [],
       loading: false,
@@ -176,7 +184,7 @@ describe('App', () => {
     expect(getByText(/No branches found/i)).toBeDefined();
   });
 
-  it('should wrap with ErrorBoundary', () => {
+  it("should wrap with ErrorBoundary", () => {
     // This test verifies ErrorBoundary is present
     // Actual error catching is tested separately
     useGitDataSpy.mockReturnValue({
@@ -193,7 +201,7 @@ describe('App', () => {
     expect(container).toBeDefined();
   });
 
-  it('should format branch items with icons', () => {
+  it("should format branch items with icons", () => {
     useGitDataSpy.mockReturnValue({
       branches: mockBranches,
       loading: false,
@@ -209,8 +217,8 @@ describe('App', () => {
     expect(getByText(/⚡/)).toBeDefined();
   });
 
-  describe('BranchActionSelectorScreen integration', () => {
-    it('should show BranchActionSelectorScreen after branch selection', () => {
+  describe("BranchActionSelectorScreen integration", () => {
+    it("should show BranchActionSelectorScreen after branch selection", () => {
       useGitDataSpy.mockReturnValue({
         branches: mockBranches,
         loading: false,

package/src/cli/ui/__tests__/components/common/Confirm.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 { Confirm } from '../../../components/common/Confirm.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 { Confirm } from "../../../components/common/Confirm.js";
+import { Window } from "happy-dom";
 
-describe('Confirm', () => {
+describe("Confirm", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,47 +15,60 @@ describe('Confirm', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render the message', () => {
+  it("should render the message", () => {
     const onConfirm = vi.fn();
-    const { getByText } = render(<Confirm message="Are you sure?" onConfirm={onConfirm} />);
+    const { getByText } = render(
+      <Confirm message="Are you sure?" onConfirm={onConfirm} />,
+    );
 
-    expect(getByText('Are you sure?')).toBeDefined();
+    expect(getByText("Are you sure?")).toBeDefined();
   });
 
-  it('should render Yes and No options', () => {
+  it("should render Yes and No options", () => {
     const onConfirm = vi.fn();
-    const { getByText } = render(<Confirm message="Continue?" onConfirm={onConfirm} />);
+    const { getByText } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} />,
+    );
 
-    expect(getByText('Yes')).toBeDefined();
-    expect(getByText('No')).toBeDefined();
+    expect(getByText("Yes")).toBeDefined();
+    expect(getByText("No")).toBeDefined();
   });
 
-  it('should render custom Yes and No labels', () => {
+  it("should render custom Yes and No labels", () => {
     const onConfirm = vi.fn();
     const { getByText } = render(
-      <Confirm message="Delete?" onConfirm={onConfirm} yesLabel="Confirm" noLabel="Cancel" />
+      <Confirm
+        message="Delete?"
+        onConfirm={onConfirm}
+        yesLabel="Confirm"
+        noLabel="Cancel"
+      />,
     );
 
-    expect(getByText('Confirm')).toBeDefined();
-    expect(getByText('Cancel')).toBeDefined();
+    expect(getByText("Confirm")).toBeDefined();
+    expect(getByText("Cancel")).toBeDefined();
   });
 
-  it('should default to Yes option', () => {
+  it("should default to Yes option", () => {
     const onConfirm = vi.fn();
-    const { container } = render(<Confirm message="Continue?" onConfirm={onConfirm} />);
+    const { container } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} />,
+    );
 
     // Verify component renders without error
     expect(container).toBeDefined();
   });
 
-  it('should accept defaultNo prop to default to No', () => {
+  it("should accept defaultNo prop to default to No", () => {
     const onConfirm = vi.fn();
-    const { container } = render(<Confirm message="Continue?" onConfirm={onConfirm} defaultNo />);
+    const { container } = render(
+      <Confirm message="Continue?" onConfirm={onConfirm} defaultNo />,
+    );
 
     expect(container).toBeDefined();
   });
 
-  it('should call onConfirm with false by default when rendered', () => {
+  it("should call onConfirm with false by default when rendered", () => {
     const onConfirm = vi.fn();
     render(<Confirm message="Continue?" onConfirm={onConfirm} />);
 

package/src/cli/ui/__tests__/components/common/ErrorBoundary.test.tsx

@@ -1,22 +1,22 @@
 /**
  * @vitest-environment happy-dom
  */
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { render } from '@testing-library/react';
-import React from 'react';
-import { ErrorBoundary } from '../../../components/common/ErrorBoundary.js';
-import { Text, Box } from 'ink';
-import { Window } from 'happy-dom';
+import { describe, it, expect, beforeEach, vi } from "vitest";
+import { render } from "@testing-library/react";
+import React from "react";
+import { ErrorBoundary } from "../../../components/common/ErrorBoundary.js";
+import { Text, Box } from "ink";
+import { Window } from "happy-dom";
 
 // Component that throws an error
 const ThrowError = ({ shouldThrow }: { shouldThrow: boolean }) => {
   if (shouldThrow) {
-    throw new Error('Test error message');
+    throw new Error("Test error message");
   }
   return <Text>No error</Text>;
 };
 
-describe('ErrorBoundary', () => {
+describe("ErrorBoundary", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -24,31 +24,31 @@ describe('ErrorBoundary', () => {
     globalThis.document = window.document as any;
 
     // Suppress console.error for expected errors in tests
-    vi.spyOn(console, 'error').mockImplementation(() => {});
+    vi.spyOn(console, "error").mockImplementation(() => {});
   });
 
-  it('should render children when no error occurs', () => {
+  it("should render children when no error occurs", () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={false} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
-    expect(getByText('No error')).toBeDefined();
+    expect(getByText("No error")).toBeDefined();
   });
 
-  it('should catch errors and display error message', () => {
+  it("should catch errors and display error message", () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Error:/)).toBeDefined();
     expect(getByText(/Test error message/)).toBeDefined();
   });
 
-  it('should display custom fallback when provided', () => {
+  it("should display custom fallback when provided", () => {
     const CustomFallback = ({ error }: { error: Error }) => (
       <Box>
         <Text color="red">Custom Error: {error.message}</Text>
@@ -58,18 +58,18 @@ describe('ErrorBoundary', () => {
     const { getByText } = render(
       <ErrorBoundary fallback={CustomFallback}>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Custom Error:/)).toBeDefined();
     expect(getByText(/Test error message/)).toBeDefined();
   });
 
-  it('should reset error state when children change', () => {
+  it("should reset error state when children change", () => {
     const { rerender, getByText } = render(
       <ErrorBoundary>
         <ThrowError shouldThrow={true} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     // Error is shown
@@ -79,14 +79,14 @@ describe('ErrorBoundary', () => {
     rerender(
       <ErrorBoundary>
         <ThrowError shouldThrow={false} />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     // Original children should be rendered
-    expect(getByText('No error')).toBeDefined();
+    expect(getByText("No error")).toBeDefined();
   });
 
-  it('should handle errors with no message', () => {
+  it("should handle errors with no message", () => {
     const ThrowNoMessage = () => {
       throw new Error();
     };
@@ -94,7 +94,7 @@ describe('ErrorBoundary', () => {
     const { getByText } = render(
       <ErrorBoundary>
         <ThrowNoMessage />
-      </ErrorBoundary>
+      </ErrorBoundary>,
     );
 
     expect(getByText(/Error:/)).toBeDefined();

package/src/cli/ui/__tests__/components/common/Input.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 { Input } from '../../../components/common/Input.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 { Input } from "../../../components/common/Input.js";
+import { Window } from "happy-dom";
 
-describe('Input', () => {
+describe("Input", () => {
   beforeEach(() => {
     // Setup happy-dom
     const window = new Window();
@@ -15,35 +15,42 @@ describe('Input', () => {
     globalThis.document = window.document as any;
   });
 
-  it('should render with value', () => {
+  it("should render with value", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
-    const { container } = render(<Input value="test" onChange={onChange} onSubmit={onSubmit} />);
+    const { container } = render(
+      <Input value="test" onChange={onChange} onSubmit={onSubmit} />,
+    );
 
     expect(container).toBeDefined();
   });
 
-  it('should render with placeholder', () => {
+  it("should render with placeholder", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
-      <Input value="" onChange={onChange} onSubmit={onSubmit} placeholder="Enter text..." />
+      <Input
+        value=""
+        onChange={onChange}
+        onSubmit={onSubmit}
+        placeholder="Enter text..."
+      />,
     );
 
-    expect(getByText('Enter text...')).toBeDefined();
+    expect(getByText("Enter text...")).toBeDefined();
   });
 
-  it('should render with label', () => {
+  it("should render with label", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
-      <Input value="" onChange={onChange} onSubmit={onSubmit} label="Name:" />
+      <Input value="" onChange={onChange} onSubmit={onSubmit} label="Name:" />,
     );
 
-    expect(getByText('Name:')).toBeDefined();
+    expect(getByText("Name:")).toBeDefined();
   });
 
-  it('should render label and placeholder together', () => {
+  it("should render label and placeholder together", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { getByText } = render(
@@ -53,24 +60,24 @@ describe('Input', () => {
         onSubmit={onSubmit}
         label="Branch name:"
         placeholder="feature/..."
-      />
+      />,
     );
 
-    expect(getByText('Branch name:')).toBeDefined();
-    expect(getByText('feature/...')).toBeDefined();
+    expect(getByText("Branch name:")).toBeDefined();
+    expect(getByText("feature/...")).toBeDefined();
   });
 
-  it('should accept mask prop for password input', () => {
+  it("should accept mask prop for password input", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     const { container } = render(
-      <Input value="secret" onChange={onChange} onSubmit={onSubmit} mask="*" />
+      <Input value="secret" onChange={onChange} onSubmit={onSubmit} mask="*" />,
     );
 
     expect(container).toBeDefined();
   });
 
-  it('should call onChange when value changes', () => {
+  it("should call onChange when value changes", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     render(<Input value="" onChange={onChange} onSubmit={onSubmit} />);
@@ -80,7 +87,7 @@ describe('Input', () => {
     expect(onChange).not.toHaveBeenCalled();
   });
 
-  it('should call onSubmit when submitted', () => {
+  it("should call onSubmit when submitted", () => {
     const onChange = vi.fn();
     const onSubmit = vi.fn();
     render(<Input value="test" onChange={onChange} onSubmit={onSubmit} />);

package/src/cli/ui/__tests__/components/common/LoadingIndicator.test.tsx

@@ -1,11 +1,11 @@
 /**
  * @vitest-environment happy-dom
  */
-import React from 'react';
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { act, render } from '@testing-library/react';
-import { LoadingIndicator } from '../../../components/common/LoadingIndicator.js';
-import { Window } from 'happy-dom';
+import React from "react";
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+import { act, render } from "@testing-library/react";
+import { LoadingIndicator } from "../../../components/common/LoadingIndicator.js";
+import { Window } from "happy-dom";
 
 const advanceTimersBy = async (ms: number) => {
   await act(async () => {
@@ -25,57 +25,63 @@ afterEach(() => {
   vi.useRealTimers();
 });
 
-describe('LoadingIndicator', () => {
+describe("LoadingIndicator", () => {
   const getSpinnerText = (container: HTMLElement) => {
-    return container.querySelector('ink-text')?.textContent ?? '';
+    return container.querySelector("ink-text")?.textContent ?? "";
   };
 
   const getMessageText = (container: HTMLElement) => {
-    const texts = container.querySelectorAll('ink-text');
-    return texts.length > 1 ? texts[1]?.textContent ?? '' : '';
+    const texts = container.querySelectorAll("ink-text");
+    return texts.length > 1 ? (texts[1]?.textContent ?? "") : "";
   };
 
-  it('does not render before the delay elapses', async () => {
+  it("does not render before the delay elapses", async () => {
     const { container } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={50} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={50} />,
     );
 
-    expect(container.textContent).toBe('');
+    expect(container.textContent).toBe("");
 
     await advanceTimersBy(20);
 
-    expect(container.textContent).toBe('');
+    expect(container.textContent).toBe("");
   });
 
-  it('renders after the delay elapses', async () => {
+  it("renders after the delay elapses", async () => {
     const { container } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={30} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={30} />,
     );
 
     await advanceTimersBy(30);
 
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(getMessageText(container)).toContain("Loading data");
   });
 
-  it('stops rendering when loading becomes false', async () => {
+  it("stops rendering when loading becomes false", async () => {
     const { container, rerender } = render(
-      <LoadingIndicator isLoading={true} message="Loading data" delay={10} />
+      <LoadingIndicator isLoading={true} message="Loading data" delay={10} />,
     );
 
     await advanceTimersBy(10);
 
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(getMessageText(container)).toContain("Loading data");
 
     await act(async () => {
-      rerender(<LoadingIndicator isLoading={false} message="Loading data" delay={10} />);
+      rerender(
+        <LoadingIndicator
+          isLoading={false}
+          message="Loading data"
+          delay={10}
+        />,
+      );
       await vi.advanceTimersByTimeAsync(0);
     });
 
-    expect(container.textContent).toBe('');
+    expect(container.textContent).toBe("");
   });
 
-  it('cycles through spinner frames over time', async () => {
-    const customFrames = ['.', '..', '...'];
+  it("cycles through spinner frames over time", async () => {
+    const customFrames = [".", "..", "..."];
     const { container } = render(
       <LoadingIndicator
         isLoading={true}
@@ -83,7 +89,7 @@ describe('LoadingIndicator', () => {
         delay={0}
         interval={5}
         frames={customFrames}
-      />
+      />,
     );
 
     await advanceTimersBy(0);
@@ -100,28 +106,28 @@ describe('LoadingIndicator', () => {
 
     expect(secondFrame).not.toEqual(firstFrame);
     expect(thirdFrame).not.toEqual(secondFrame);
-    expect(customFrames).toContain(firstFrame ?? '');
-    expect(customFrames).toContain(secondFrame ?? '');
-    expect(customFrames).toContain(thirdFrame ?? '');
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(customFrames).toContain(firstFrame ?? "");
+    expect(customFrames).toContain(secondFrame ?? "");
+    expect(customFrames).toContain(thirdFrame ?? "");
+    expect(getMessageText(container)).toContain("Loading data");
   });
 
-  it('keeps rendering even when only a single frame is provided', async () => {
+  it("keeps rendering even when only a single frame is provided", async () => {
     const { container } = render(
       <LoadingIndicator
         isLoading={true}
         message="Loading data"
         delay={0}
         interval={10}
-        frames={['*']}
-      />
+        frames={["*"]}
+      />,
     );
 
     await advanceTimersBy(0);
-    expect(getSpinnerText(container)).toBe('*');
+    expect(getSpinnerText(container)).toBe("*");
 
     await advanceTimersBy(30);
-    expect(getSpinnerText(container)).toBe('*');
-    expect(getMessageText(container)).toContain('Loading data');
+    expect(getSpinnerText(container)).toBe("*");
+    expect(getMessageText(container)).toContain("Loading data");
   });
 });