@akiojin/gwt 4.10.0 → 4.11.6

package/src/cli/ui/__tests__/components/App.test.tsx

@@ -1,365 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, vi } from "vitest";
-import { render } from "@testing-library/react";
-import React from "react";
-import { Window } from "happy-dom";
-import type { BranchInfo, BranchItem } from "../../types.js";
-import type { BranchListScreenProps } from "../../components/screens/BranchListScreen.js";
-import { App } from "../../components/App.js";
-
-const mockRefresh = vi.fn();
-const exitMock = vi.fn();
-const branchListProps: BranchListScreenProps[] = [];
-const useGitDataMock = vi.fn();
-const useProfilesMock = vi.fn();
-const useToolStatusMock = vi.fn();
-
-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: exitMock }),
-    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", () => {
-  return {
-    BranchListScreen: (props: BranchListScreenProps) => {
-      branchListProps.push(props);
-      return (
-        <div>
-          <div>gwt - Branch Selection</div>
-          <div>Local: {props.stats?.localCount ?? 0}</div>
-          <div>Remote: {props.stats?.remoteCount ?? 0}</div>
-          <div>Worktrees: {props.stats?.worktreeCount ?? 0}</div>
-          <div>Changes: {props.stats?.changesCount ?? 0}</div>
-          {props.loading && <div>Loading Git information</div>}
-          {props.error && <div>Error: {props.error.message}</div>}
-          {!props.loading && !props.error && props.branches.length === 0 && (
-            <div>No branches found</div>
-          )}
-          <ul>
-            {props.branches.map((branch) => (
-              <li
-                key={branch.name}
-              >{`${branch.icons?.join("") ?? ""} ${branch.name}`}</li>
-            ))}
-          </ul>
-        </div>
-      );
-    },
-  };
-});
-
-describe("App", () => {
-  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;
-
-    vi.clearAllMocks();
-    useGitDataMock.mockReset();
-    useProfilesMock.mockReset();
-    useToolStatusMock.mockReset();
-    branchListProps.length = 0;
-
-    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(),
-    });
-  });
-
-  const mockBranches: BranchInfo[] = [
-    {
-      name: "main",
-      type: "local",
-      branchType: "main",
-      isCurrent: true,
-    },
-    {
-      name: "feature/test",
-      type: "local",
-      branchType: "feature",
-      isCurrent: false,
-    },
-  ];
-
-  it("should render BranchListScreen when data is loaded", () => {
-    useGitDataMock.mockImplementation(() => ({
-      branches: mockBranches,
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    expect(props?.loading).toBe(false);
-    expect(props?.error).toBeNull();
-    const branchNames = props?.branches.map((b) => b.name);
-    expect(branchNames).toContain("main");
-    expect(branchNames).toContain("feature/test");
-  });
-
-  it("should show loading state initially", async () => {
-    useGitDataMock.mockImplementation(() => ({
-      branches: [],
-      loading: true,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} loadingIndicatorDelay={10} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    expect(props?.loading).toBe(true);
-    expect(props?.loadingIndicatorDelay).toBe(10);
-  });
-
-  it("should show error state when Git data fails to load", () => {
-    const error = new Error("Failed to fetch branches");
-    useGitDataMock.mockImplementation(() => ({
-      branches: [],
-      loading: false,
-      error,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    expect(props?.error?.message).toBe("Failed to fetch branches");
-    expect(props?.loading).toBe(false);
-  });
-
-  it("should calculate statistics from branches", () => {
-    const branchesWithWorktree: 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,
-      },
-    ];
-
-    useGitDataMock.mockImplementation(() => ({
-      branches: branchesWithWorktree,
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    expect(props?.stats.localCount).toBe(2);
-    expect(props?.stats.remoteCount).toBe(1);
-    expect(props?.stats.worktreeCount).toBe(1);
-  });
-
-  it("should render branch selection without triggering exit", () => {
-    useGitDataMock.mockImplementation(() => ({
-      branches: mockBranches,
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    expect(container).toBeDefined();
-    expect(onExit).not.toHaveBeenCalled();
-    // Note: Testing actual selection requires simulating user input,
-    // which is covered in integration tests
-  });
-
-  it("should handle empty branch list", () => {
-    useGitDataMock.mockImplementation(() => ({
-      branches: [],
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    expect(props?.branches).toHaveLength(0);
-    expect(props?.loading).toBe(false);
-    expect(props?.error).toBeNull();
-  });
-
-  it("should wrap with ErrorBoundary", () => {
-    // This test verifies ErrorBoundary is present
-    // Actual error catching is tested separately
-    useGitDataMock.mockImplementation(() => ({
-      branches: mockBranches,
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    const { container } = render(<App onExit={onExit} />);
-
-    expect(container).toBeDefined();
-  });
-
-  it("should format branch items with icons", () => {
-    useGitDataMock.mockImplementation(() => ({
-      branches: mockBranches,
-      loading: false,
-      error: null,
-      worktrees: [],
-      refresh: mockRefresh,
-    }));
-
-    const onExit = vi.fn();
-    render(<App onExit={onExit} />);
-
-    expect(branchListProps).not.toHaveLength(0);
-    const props = branchListProps.at(-1);
-    const main = props?.branches.find((b: BranchItem) => b.name === "main");
-    expect(main?.icons).toContain("⚡");
-  });
-
-  describe("BranchActionSelectorScreen integration", () => {
-    // TODO: replace placeholder assertions with real integration checks.
-    it("should show BranchActionSelectorScreen after branch selection", () => {
-      useGitDataMock.mockImplementation(() => ({
-        branches: mockBranches,
-        loading: false,
-        error: null,
-        worktrees: [],
-        refresh: mockRefresh,
-      }));
-
-      const onExit = vi.fn();
-      const { container } = render(<App onExit={onExit} />);
-
-      // TODO: verify BranchActionSelectorScreen appears
-      expect(container).toBeDefined();
-    });
-
-    it('should navigate to AI tool selector when "use existing" is selected', () => {
-      useGitDataMock.mockImplementation(() => ({
-        branches: mockBranches,
-        loading: false,
-        error: null,
-        worktrees: [],
-        refresh: mockRefresh,
-      }));
-
-      const onExit = vi.fn();
-      const { container } = render(<App onExit={onExit} />);
-
-      // TODO: verify navigation to AIToolSelectorScreen
-      expect(container).toBeDefined();
-    });
-
-    it('should navigate to branch creator when "create new" is selected', () => {
-      useGitDataMock.mockImplementation(() => ({
-        branches: mockBranches,
-        loading: false,
-        error: null,
-        worktrees: [],
-        refresh: mockRefresh,
-      }));
-
-      const onExit = vi.fn();
-      const { container } = render(<App onExit={onExit} />);
-
-      // TODO: verify navigation to BranchCreatorScreen
-      expect(container).toBeDefined();
-    });
-  });
-});

package/src/cli/ui/__tests__/components/ModelSelectorScreen.initial.test.tsx

@@ -1,91 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import React from "react";
-import { describe, it, expect, beforeEach, vi } from "vitest";
-import { render, waitFor } from "@testing-library/react";
-import { ModelSelectorScreen } from "../../components/screens/ModelSelectorScreen.js";
-import type { ModelSelectionResult } from "../../components/screens/ModelSelectorScreen.js";
-import { Window } from "happy-dom";
-import type {
-  SelectProps,
-  SelectItem,
-} from "../../components/common/Select.js";
-
-const selectMocks: SelectProps<SelectItem>[] = [];
-
-vi.mock("../../components/common/Select.js", () => {
-  return {
-    Select: (props: SelectProps<SelectItem>) => {
-      selectMocks.push(props);
-      return React.createElement("div", {
-        "data-testid": "select-mock",
-        onClick: () =>
-          props.onSelect &&
-          props.onSelect(props.items[props.initialIndex ?? 0]),
-      });
-    },
-  };
-});
-
-describe("ModelSelectorScreen initial selection", () => {
-  beforeEach(() => {
-    selectMocks.length = 0;
-    const window = new Window();
-    globalThis.window = window as unknown as typeof globalThis.window;
-    globalThis.document =
-      window.document as unknown as typeof globalThis.document;
-  });
-
-  it("sets model list initialIndex based on previous selection", async () => {
-    const initial: ModelSelectionResult = {
-      model: "gpt-5.1-codex-max",
-      inferenceLevel: "high",
-    };
-
-    render(
-      <ModelSelectorScreen
-        tool="codex-cli"
-        onBack={() => {}}
-        onSelect={() => {}}
-        initialSelection={initial}
-      />,
-    );
-
-    await waitFor(() => expect(selectMocks.length).toBeGreaterThan(0));
-    const modelSelect = selectMocks.at(-1);
-    expect(modelSelect).toBeDefined();
-    const index = modelSelect.initialIndex as number;
-    // codex-cli models: ["", gpt-5.2-codex, gpt-5.1-codex-max, gpt-5.1-codex-mini, gpt-5.2]
-    // (index 0 = Default/Auto, index 2 = gpt-5.1-codex-max)
-    expect(index).toBe(2);
-  });
-
-  it("includes gpt-5.2-codex in model options and preserves ordering", async () => {
-    const initial: ModelSelectionResult = {
-      model: "gpt-5.1-codex-max",
-      inferenceLevel: "high",
-    };
-
-    render(
-      <ModelSelectorScreen
-        tool="codex-cli"
-        onBack={() => {}}
-        onSelect={() => {}}
-        initialSelection={initial}
-      />,
-    );
-
-    await waitFor(() => expect(selectMocks.length).toBeGreaterThan(1));
-    const modelSelect = selectMocks.at(-1);
-    expect(modelSelect).toBeDefined();
-    const ids = modelSelect?.items.map((item) => item.value);
-    expect(ids).toEqual([
-      "", // Default (Auto)
-      "gpt-5.2-codex",
-      "gpt-5.1-codex-max",
-      "gpt-5.1-codex-mini",
-      "gpt-5.2",
-    ]);
-  });
-});

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

@@ -1,80 +0,0 @@
-/**
- * @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";
-
-describe("Confirm", () => {
-  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;
-  });
-
-  it("should render the message", () => {
-    const onConfirm = vi.fn();
-    const { getByText } = render(
-      <Confirm message="Are you sure?" onConfirm={onConfirm} />,
-    );
-
-    expect(getByText("Are you sure?")).toBeDefined();
-  });
-
-  it("should render Yes and No options", () => {
-    const onConfirm = vi.fn();
-    const { getByText } = render(
-      <Confirm message="Continue?" onConfirm={onConfirm} />,
-    );
-
-    expect(getByText("Yes")).toBeDefined();
-    expect(getByText("No")).toBeDefined();
-  });
-
-  it("should render custom Yes and No labels", () => {
-    const onConfirm = vi.fn();
-    const { getByText } = render(
-      <Confirm
-        message="Delete?"
-        onConfirm={onConfirm}
-        yesLabel="Confirm"
-        noLabel="Cancel"
-      />,
-    );
-
-    expect(getByText("Confirm")).toBeDefined();
-    expect(getByText("Cancel")).toBeDefined();
-  });
-
-  it("should default to Yes option", () => {
-    const onConfirm = vi.fn();
-    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", () => {
-    const onConfirm = vi.fn();
-    const { container } = render(
-      <Confirm message="Continue?" onConfirm={onConfirm} defaultNo />,
-    );
-
-    expect(container).toBeDefined();
-  });
-
-  it("should call onConfirm with false by default when rendered", () => {
-    const onConfirm = vi.fn();
-    render(<Confirm message="Continue?" onConfirm={onConfirm} />);
-
-    // Note: Simulating selection requires ink-testing-library
-    // For now, we just verify the component structure
-    expect(onConfirm).not.toHaveBeenCalled();
-  });
-});

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

@@ -1,104 +0,0 @@
-/**
- * @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";
-
-// Component that throws an error
-const ThrowError = ({ shouldThrow }: { shouldThrow: boolean }) => {
-  if (shouldThrow) {
-    throw new Error("Test error message");
-  }
-  return <Text>No error</Text>;
-};
-
-describe("ErrorBoundary", () => {
-  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;
-
-    // Suppress console.error for expected errors in tests
-    vi.spyOn(console, "error").mockImplementation(() => {});
-  });
-
-  it("should render children when no error occurs", () => {
-    const { getByText } = render(
-      <ErrorBoundary>
-        <ThrowError shouldThrow={false} />
-      </ErrorBoundary>,
-    );
-
-    expect(getByText("No error")).toBeDefined();
-  });
-
-  it("should catch errors and display error message", () => {
-    const { getByText } = render(
-      <ErrorBoundary>
-        <ThrowError shouldThrow={true} />
-      </ErrorBoundary>,
-    );
-
-    expect(getByText(/Error:/)).toBeDefined();
-    expect(getByText(/Test error message/)).toBeDefined();
-  });
-
-  it("should display custom fallback when provided", () => {
-    const CustomFallback = ({ error }: { error: Error }) => (
-      <Box>
-        <Text color="red">Custom Error: {error.message}</Text>
-      </Box>
-    );
-
-    const { getByText } = render(
-      <ErrorBoundary fallback={CustomFallback}>
-        <ThrowError shouldThrow={true} />
-      </ErrorBoundary>,
-    );
-
-    expect(getByText(/Custom Error:/)).toBeDefined();
-    expect(getByText(/Test error message/)).toBeDefined();
-  });
-
-  it("should reset error state when children change", () => {
-    const { rerender, getByText } = render(
-      <ErrorBoundary>
-        <ThrowError shouldThrow={true} />
-      </ErrorBoundary>,
-    );
-
-    // Error is shown
-    expect(getByText(/Error:/)).toBeDefined();
-
-    // Rerender with non-throwing component
-    rerender(
-      <ErrorBoundary>
-        <ThrowError shouldThrow={false} />
-      </ErrorBoundary>,
-    );
-
-    // Original children should be rendered
-    expect(getByText("No error")).toBeDefined();
-  });
-
-  it("should handle errors with no message", () => {
-    const ThrowNoMessage = () => {
-      throw new Error();
-    };
-
-    const { getByText } = render(
-      <ErrorBoundary>
-        <ThrowNoMessage />
-      </ErrorBoundary>,
-    );
-
-    expect(getByText(/Error:/)).toBeDefined();
-    expect(getByText(/Unknown error/)).toBeDefined();
-  });
-});