@akiojin/gwt 4.10.0 → 4.11.6

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

@@ -1,216 +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 { PRCleanupScreen } from "../../../components/screens/PRCleanupScreen.js";
-import { Window } from "happy-dom";
-import type { CleanupTarget } from "../../../types.js";
-
-describe("PRCleanupScreen", () => {
-  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;
-  });
-
-  const mockTargets: CleanupTarget[] = [
-    {
-      branch: "feature/add-new-feature",
-      cleanupType: "worktree-and-branch",
-      pullRequest: {
-        number: 123,
-        title: "Add new feature",
-        branch: "feature/add-new-feature",
-        mergedAt: "2025-01-20T10:00:00Z",
-        author: "user1",
-      },
-      worktreePath: "/workspace/feature-add-new-feature",
-      hasUncommittedChanges: false,
-      hasUnpushedCommits: false,
-      hasRemoteBranch: true,
-      isAccessible: true,
-    },
-    {
-      branch: "hotfix/fix-bug",
-      cleanupType: "branch-only",
-      pullRequest: {
-        number: 124,
-        title: "Fix bug",
-        branch: "hotfix/fix-bug",
-        mergedAt: "2025-01-21T15:30:00Z",
-        author: "user2",
-      },
-      worktreePath: null,
-      hasUncommittedChanges: false,
-      hasUnpushedCommits: false,
-      hasRemoteBranch: false,
-    },
-  ];
-
-  it("should render header with title", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/Branch Cleanup/i)).toBeDefined();
-  });
-
-  it("should render PR list", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/feature\/add-new-feature/i)).toBeDefined();
-    expect(getByText(/hotfix\/fix-bug/i)).toBeDefined();
-  });
-
-  it("should render footer with actions", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getAllByText } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
-    expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
-  });
-
-  it("should handle empty PR list", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText } = render(
-      <PRCleanupScreen
-        targets={[]}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/No cleanup targets found/i)).toBeDefined();
-  });
-
-  it("should display PR count in stats", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText, getAllByText } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/Total:/i)).toBeDefined();
-    expect(getAllByText(/^2$/).length).toBeGreaterThan(0);
-  });
-
-  it("should use terminal height for layout calculation", () => {
-    const originalRows = process.stdout.rows;
-    process.stdout.rows = 30;
-
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { container } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(container).toBeDefined();
-
-    process.stdout.rows = originalRows;
-  });
-
-  it("should handle back navigation with ESC key", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { container } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    // Test will verify onBack is called when ESC is pressed
-    expect(container).toBeDefined();
-  });
-
-  it("should render status message when provided", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText } = render(
-      <PRCleanupScreen
-        targets={mockTargets}
-        loading={false}
-        error={null}
-        statusMessage="Cleanup completed"
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/Cleanup completed/i)).toBeDefined();
-  });
-
-  it("should render loading message when loading", () => {
-    const onBack = vi.fn();
-    const onCleanup = vi.fn();
-    const { getByText } = render(
-      <PRCleanupScreen
-        targets={[]}
-        loading
-        error={null}
-        onBack={onBack}
-        onRefresh={vi.fn()}
-        onCleanup={onCleanup}
-      />,
-    );
-
-    expect(getByText(/Loading cleanup targets/i)).toBeDefined();
-  });
-});

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

@@ -1,147 +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 { SessionSelectorScreen } from "../../../components/screens/SessionSelectorScreen.js";
-import { Window } from "happy-dom";
-
-describe("SessionSelectorScreen", () => {
-  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;
-  });
-
-  const sessionItems = [
-    {
-      sessionId: "session-1",
-      branch: "feature/foo",
-      toolLabel: "Codex",
-      timestamp: 1700000000000,
-    },
-    {
-      sessionId: "session-2",
-      branch: "feature/bar",
-      toolLabel: "Claude",
-      timestamp: 1700000100000,
-    },
-    {
-      sessionId: "session-3",
-      branch: "feature/baz",
-      toolLabel: "Codex",
-      timestamp: 1700000200000,
-    },
-  ];
-
-  it("should render header with title", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { getByText } = render(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(getByText(/Session Selection/i)).toBeDefined();
-  });
-
-  it("should render session list", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { getByText } = render(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(getByText(/feature\/foo/i)).toBeDefined();
-    expect(getByText(/feature\/bar/i)).toBeDefined();
-    expect(getByText(/feature\/baz/i)).toBeDefined();
-  });
-
-  it("should render footer with actions", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { getAllByText } = render(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(getAllByText(/enter/i).length).toBeGreaterThan(0);
-    expect(getAllByText(/esc/i).length).toBeGreaterThan(0);
-  });
-
-  it("should handle empty session list", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { getByText } = render(
-      <SessionSelectorScreen
-        sessions={[]}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(getByText(/No sessions found/i)).toBeDefined();
-  });
-
-  it("should display session count in stats", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { getByText, getAllByText } = render(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(getByText(/Total:/i)).toBeDefined();
-    expect(getAllByText(/3/).length).toBeGreaterThan(0);
-  });
-
-  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(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    expect(container).toBeDefined();
-
-    process.stdout.rows = originalRows;
-  });
-
-  it("should handle back navigation with ESC key", () => {
-    const onBack = vi.fn();
-    const onSelect = vi.fn();
-    const { container } = render(
-      <SessionSelectorScreen
-        sessions={sessionItems}
-        onBack={onBack}
-        onSelect={onSelect}
-      />,
-    );
-
-    // Test will verify onBack is called when ESC is pressed
-    expect(container).toBeDefined();
-  });
-});

package/src/cli/ui/__tests__/hooks/useGitData.nonblocking.test.tsx

@@ -1,206 +0,0 @@
-/**
- * @vitest-environment happy-dom
- */
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
-import { renderHook, waitFor } from "@testing-library/react";
-import { GIT_DATA_TIMEOUT_MS, useGitData } from "../../hooks/useGitData.js";
-import { Window } from "happy-dom";
-
-vi.mock("../../../git.js", () => ({
-  getAllBranches: vi.fn(),
-  fetchAllRemotes: vi.fn(),
-  getRepositoryRoot: vi.fn(),
-  collectUpstreamMap: vi.fn(),
-  getBranchDivergenceStatuses: vi.fn(),
-  hasUnpushedCommitsInRepo: vi.fn(),
-  hasUncommittedChanges: vi.fn(),
-}));
-
-vi.mock("../../../worktree.js", () => ({
-  listAdditionalWorktrees: vi.fn(),
-}));
-
-vi.mock("../../../github.js", () => ({
-  getPullRequestByBranch: vi.fn(),
-}));
-
-vi.mock("../../../config/index.js", () => ({
-  getLastToolUsageMap: vi.fn(),
-}));
-
-import {
-  getAllBranches,
-  fetchAllRemotes,
-  getRepositoryRoot,
-  collectUpstreamMap,
-  getBranchDivergenceStatuses,
-  hasUnpushedCommitsInRepo,
-  hasUncommittedChanges,
-} from "../../../git.js";
-import { listAdditionalWorktrees } from "../../../worktree.js";
-import { getPullRequestByBranch } from "../../../github.js";
-import { getLastToolUsageMap } from "../../../config/index.js";
-
-const advanceTimersBy = async (ms: number) => {
-  if (typeof vi.advanceTimersByTimeAsync === "function") {
-    await vi.advanceTimersByTimeAsync(ms);
-  } else if (typeof vi.advanceTimersByTime === "function") {
-    vi.advanceTimersByTime(ms);
-  }
-};
-
-describe("useGitData non-blocking fetch", () => {
-  beforeEach(() => {
-    const window = new Window();
-    globalThis.window = window as unknown as typeof globalThis.window;
-    globalThis.document =
-      window.document as unknown as typeof globalThis.document;
-
-    (getAllBranches as ReturnType<typeof vi.fn>).mockReset();
-    (fetchAllRemotes as ReturnType<typeof vi.fn>).mockReset();
-    (getRepositoryRoot as ReturnType<typeof vi.fn>).mockReset();
-    (collectUpstreamMap as ReturnType<typeof vi.fn>).mockReset();
-    (getBranchDivergenceStatuses as ReturnType<typeof vi.fn>).mockReset();
-    (hasUnpushedCommitsInRepo as ReturnType<typeof vi.fn>).mockReset();
-    (hasUncommittedChanges as ReturnType<typeof vi.fn>).mockReset();
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockReset();
-    (getPullRequestByBranch as ReturnType<typeof vi.fn>).mockReset();
-    (getLastToolUsageMap as ReturnType<typeof vi.fn>).mockReset();
-  });
-
-  afterEach(() => {
-    if (typeof vi.useRealTimers === "function") {
-      vi.useRealTimers();
-    }
-  });
-
-  it("does not block loading on fetchAllRemotes", async () => {
-    const pending = new Promise<void>(() => {});
-
-    (getRepositoryRoot as ReturnType<typeof vi.fn>).mockResolvedValue("/repo");
-    (fetchAllRemotes as ReturnType<typeof vi.fn>).mockReturnValue(pending);
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (getLastToolUsageMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (collectUpstreamMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (getBranchDivergenceStatuses as ReturnType<typeof vi.fn>).mockResolvedValue(
-      [],
-    );
-    (hasUnpushedCommitsInRepo as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (hasUncommittedChanges as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (getPullRequestByBranch as ReturnType<typeof vi.fn>).mockResolvedValue(
-      null,
-    );
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(
-      () => {
-        expect(result.current.loading).toBe(false);
-      },
-      { timeout: 1000 },
-    );
-
-    expect(fetchAllRemotes).toHaveBeenCalled();
-  });
-
-  it("releases loading state when branch fetch stalls", async () => {
-    if (typeof vi.useFakeTimers === "function") {
-      vi.useFakeTimers();
-    }
-
-    const pending = new Promise<void>(() => {});
-
-    (getRepositoryRoot as ReturnType<typeof vi.fn>).mockResolvedValue("/repo");
-    (fetchAllRemotes as ReturnType<typeof vi.fn>).mockResolvedValue(undefined);
-    (getAllBranches as ReturnType<typeof vi.fn>).mockReturnValue(
-      pending as unknown as Promise<unknown>,
-    );
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (getLastToolUsageMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (collectUpstreamMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (getBranchDivergenceStatuses as ReturnType<typeof vi.fn>).mockResolvedValue(
-      [],
-    );
-    (hasUnpushedCommitsInRepo as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (hasUncommittedChanges as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (getPullRequestByBranch as ReturnType<typeof vi.fn>).mockResolvedValue(
-      null,
-    );
-
-    const { result } = renderHook(() => useGitData());
-
-    await advanceTimersBy(GIT_DATA_TIMEOUT_MS + 10);
-
-    await waitFor(
-      () => {
-        expect(result.current.loading).toBe(false);
-      },
-      { timeout: 1000 },
-    );
-  });
-
-  it("passes repoRoot to getAllBranches", async () => {
-    const mockRepoRoot = "/custom/repo/root";
-
-    (getRepositoryRoot as ReturnType<typeof vi.fn>).mockResolvedValue(
-      mockRepoRoot,
-    );
-    (fetchAllRemotes as ReturnType<typeof vi.fn>).mockResolvedValue(undefined);
-    (getAllBranches as ReturnType<typeof vi.fn>).mockResolvedValue([
-      {
-        name: "main",
-        type: "local",
-        branchType: "main",
-        isCurrent: true,
-      },
-    ]);
-    (listAdditionalWorktrees as ReturnType<typeof vi.fn>).mockResolvedValue([]);
-    (getLastToolUsageMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (collectUpstreamMap as ReturnType<typeof vi.fn>).mockResolvedValue(
-      new Map(),
-    );
-    (getBranchDivergenceStatuses as ReturnType<typeof vi.fn>).mockResolvedValue(
-      [],
-    );
-    (hasUnpushedCommitsInRepo as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (hasUncommittedChanges as ReturnType<typeof vi.fn>).mockResolvedValue(
-      false,
-    );
-    (getPullRequestByBranch as ReturnType<typeof vi.fn>).mockResolvedValue(
-      null,
-    );
-
-    const { result } = renderHook(() => useGitData());
-
-    await waitFor(
-      () => {
-        expect(result.current.loading).toBe(false);
-      },
-      { timeout: 1000 },
-    );
-
-    // getAllBranches が repoRoot を引数として受け取っていることを検証
-    expect(getAllBranches).toHaveBeenCalledWith(mockRepoRoot);
-  });
-});