gitforest 0.1.0 → 1.1.0

package/test/unit/components/ProgressBar.test.tsx

@@ -1,138 +0,0 @@
-/**
- * Tests for ProgressBar component
- */
-
-import { describe, test, expect } from "bun:test";
-import React from "react";
-import { render } from "ink-testing-library";
-import { ProgressBar, Spinner } from "../../../src/components/ProgressBar.tsx";
-
-describe("ProgressBar", () => {
-  describe("rendering", () => {
-    test("renders progress bar with current/total", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Processing" current={5} total={10} />
-      );
-
-      expect(lastFrame()).toContain("Processing");
-      expect(lastFrame()).toContain("5/10");
-    });
-
-    test("shows percentage correctly at 50%", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Loading" current={5} total={10} />
-      );
-
-      expect(lastFrame()).toContain("(50%)");
-      // Check that the bar has correct filled/empty ratio
-      expect(lastFrame()).toContain("[██████████░░░░░░░░░░]");
-    });
-
-    test("handles 0% progress", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Starting" current={0} total={10} />
-      );
-
-      expect(lastFrame()).toContain("0/10");
-      expect(lastFrame()).toContain("(0%)");
-      // All empty bars (24 characters total)
-      expect(lastFrame()).toContain("[░░░░░░░░░░░░░░░░░░░░]");
-    });
-
-    test("handles 100% progress", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Complete" current={10} total={10} />
-      );
-
-      expect(lastFrame()).toContain("10/10");
-      expect(lastFrame()).toContain("(100%)");
-      // All filled bars
-      expect(lastFrame()).toContain("[████████████████████]");
-    });
-
-    test.skip("handles over 100% progress (BUG: component fails with negative empty value)", () => {
-      // This test is skipped because the ProgressBar component has a bug
-      // when current > total, causing empty to be negative
-      // TODO: Fix ProgressBar component to handle over 100% progress
-      const { lastFrame } = render(
-        <ProgressBar label="Overcomplete" current={15} total={10} />
-      );
-
-      const output = lastFrame();
-      expect(output).toContain("15/10");
-      expect(output).toContain("(150%)");
-      // When over 100%, it should cap at 100% filled
-      expect(output).toContain("[████████████████████]");
-    });
-
-    test("handles zero total gracefully", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Empty" current={0} total={0} />
-      );
-
-      expect(lastFrame()).toContain("0/0");
-      expect(lastFrame()).toContain("(0%)");
-      expect(lastFrame()).toContain("[░░░░░░░░░░░░░░░░░░░░]");
-    });
-
-    test("displays label if provided", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Downloading files" current={25} total={100} />
-      );
-
-      expect(lastFrame()).toContain("Downloading files");
-      expect(lastFrame()).toContain("25/100");
-      expect(lastFrame()).toContain("(25%)");
-    });
-
-    test("uses custom width when provided", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Test" current={5} total={10} width={10} />
-      );
-
-      // With width=10, 50% should show 5 filled and 5 empty
-      expect(lastFrame()).toContain("[█████░░░░░]");
-    });
-
-    test("has correct colors", () => {
-      const { lastFrame } = render(
-        <ProgressBar label="Colored" current={3} total={10} />
-      );
-
-      const frame = lastFrame();
-      // Just verify the output contains expected elements
-      expect(frame).toContain("Colored");
-      expect(frame).toContain("[██████░░░░░░░░░░░░░░]");
-      expect(frame).toContain("3/10 (30%)");
-    });
-  });
-});
-
-describe("Spinner", () => {
-  test("renders spinner with label", () => {
-    const { lastFrame } = render(<Spinner label="Loading..." />);
-
-    expect(lastFrame()).toContain("Loading...");
-    // Should show one of the spinner frames
-    const frame = lastFrame();
-    expect(frame).toBeDefined();
-    const hasSpinnerFrame = frame && ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"].some(
-      (spin) => frame.includes(spin)
-    );
-    expect(hasSpinnerFrame).toBe(true);
-  });
-
-  test("has correct colors", () => {
-    const { lastFrame } = render(<Spinner label="Working" />);
-
-    const output = lastFrame();
-    expect(output).toBeDefined();
-    // Verify spinner and label are present
-    expect(output).toContain("Working");
-    // Should show one of the spinner frames
-    const hasSpinnerFrame = output && ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"].some(
-      (spin) => output.includes(spin)
-    );
-    expect(hasSpinnerFrame).toBe(true);
-  });
-});

package/test/unit/components/ProjectItem.test.tsx

@@ -1,182 +0,0 @@
-/**
- * Tests for ProjectItem component
- */
-
-import { describe, test, expect } from "bun:test";
-import React from "react";
-import { render } from "ink-testing-library";
-import { ProjectItem } from "../../../src/components/ProjectItem.tsx";
-import {
-  createMockProject,
-  createDirtyProject,
-  createAheadProject,
-  createNonGitProject,
-  createSubmoduleProject,
-} from "../mocks/index.ts";
-import { createNoRemoteStatus } from "../mocks/git-service.ts";
-
-describe("ProjectItem", () => {
-  describe("rendering", () => {
-    test("renders clean git project with checkmark", () => {
-      const project = createMockProject({ name: "test-project" });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("test-project");
-      expect(lastFrame()).toContain("✓");
-      expect(lastFrame()).toContain("[ ]");
-    });
-
-    test("renders dirty project with yellow dot", () => {
-      const project = createDirtyProject("dirty-project");
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("dirty-project");
-      expect(lastFrame()).toContain("●");
-    });
-
-    test("renders project with unpushed commits", () => {
-      const project = createAheadProject("ahead-project", 5);
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("ahead-project");
-      expect(lastFrame()).toContain("↑5");
-    });
-
-    test("renders project with no remote", () => {
-      const project = createMockProject({
-        name: "no-remote-project",
-        status: createNoRemoteStatus(),
-      });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("no-remote-project");
-      expect(lastFrame()).toContain("no-remote");
-    });
-
-    test("renders non-git project with marker", () => {
-      const project = createNonGitProject("npm-project", "package.json");
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("npm-project");
-      expect(lastFrame()).toContain("[package.json]");
-      expect(lastFrame()).toContain("-"); // Non-git status icon
-    });
-
-    test("renders submodule with indicator", () => {
-      const project = createSubmoduleProject("my-submodule");
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("my-submodule");
-      expect(lastFrame()).toContain("(sub)");
-      expect(lastFrame()).toContain("○"); // Submodule status icon
-    });
-  });
-
-  describe("selection state", () => {
-    test("shows selection checkbox when selected", () => {
-      const project = createMockProject({ name: "selected-project" });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={true} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("[x]");
-    });
-
-    test("shows empty checkbox when not selected", () => {
-      const project = createMockProject({ name: "unselected-project" });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("[ ]");
-    });
-  });
-
-  describe("cursor state", () => {
-    test("shows cursor indicator when at cursor", () => {
-      const project = createMockProject({ name: "cursor-project" });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={true} />
-      );
-
-      expect(lastFrame()).toContain(">");
-      expect(lastFrame()).toContain("cursor-project");
-    });
-
-    test("shows no cursor indicator when not at cursor", () => {
-      const project = createMockProject({ name: "non-cursor-project" });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      // Should not have cursor indicator at position 0
-      const frame = lastFrame()!;
-      expect(frame.startsWith(">")).toBe(false);
-    });
-  });
-
-  describe("last activity formatting", () => {
-    test("shows 'today' for recent commits", () => {
-      const project = createMockProject({
-        name: "recent-project",
-        status: {
-          ...createMockProject().status!,
-          lastLocalCommit: new Date(),
-        },
-      });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("today");
-    });
-
-    test("shows 'yesterday' for day-old commits", () => {
-      const yesterday = new Date();
-      yesterday.setDate(yesterday.getDate() - 1);
-
-      const project = createMockProject({
-        name: "yesterday-project",
-        status: {
-          ...createMockProject().status!,
-          lastLocalCommit: yesterday,
-        },
-      });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("yesterday");
-    });
-
-    test("shows days ago for recent commits", () => {
-      const fiveDaysAgo = new Date();
-      fiveDaysAgo.setDate(fiveDaysAgo.getDate() - 5);
-
-      const project = createMockProject({
-        name: "old-project",
-        status: {
-          ...createMockProject().status!,
-          lastLocalCommit: fiveDaysAgo,
-        },
-      });
-      const { lastFrame } = render(
-        <ProjectItem project={project} isSelected={false} isCursor={false} />
-      );
-
-      expect(lastFrame()).toContain("5d ago");
-    });
-  });
-});

package/test/unit/components/ProjectList.test.tsx

@@ -1,311 +0,0 @@
-/**
- * Tests for ProjectList component
- */
-
-import { describe, test, expect, beforeEach, mock } from "bun:test";
-import React from "react";
-import { render } from "ink-testing-library";
-import { ProjectList } from "../../../src/components/ProjectList.tsx";
-import { StoreContext } from "../../../src/state/store.tsx";
-import type { UnifiedAppState, UnifiedAppAction } from "../../../src/types/index.ts";
-import { 
-  createMockUnifiedAppState, 
-  createMockUnifiedRepo,
-  createLocalOnlyUnifiedRepo,
-  createGitHubOnlyUnifiedRepo,
-  createSyncedUnifiedRepo
-} from "../mocks/index.ts";
-
-/**
- * Helper to create a mock store context value
- */
-function createMockStoreContext(state: Partial<UnifiedAppState> = {}) {
-  const mockDispatch = mock((action: UnifiedAppAction) => {});
-  const mockState = createMockUnifiedAppState(state);
-  
-  return {
-    state: mockState,
-    dispatch: mockDispatch,
-  };
-}
-
-/**
- * Helper to render ProjectList with custom store context
- */
-function renderProjectListWithContext(contextValue: { state: UnifiedAppState; dispatch: any }, height?: number) {
-  return render(
-    <StoreContext.Provider value={contextValue}>
-      <ProjectList height={height} />
-    </StoreContext.Provider>
-  );
-}
-
-describe("ProjectList", () => {
-  describe("loading state", () => {
-    test("shows spinner when isLoading is true", () => {
-      const context = createMockStoreContext({ isLoading: true });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("Scanning projects...");
-      // Should not show headers or list during loading
-      expect(lastFrame()).not.toContain("Name");
-      expect(lastFrame()).not.toContain("Status");
-    });
-
-    test("shows spinner centered in specified height", () => {
-      const context = createMockStoreContext({ isLoading: true });
-      const { lastFrame } = renderProjectListWithContext(context, 10);
-
-      // The spinner should be rendered with the specified height
-      const output = lastFrame();
-      expect(output).toContain("Scanning projects...");
-      // Ink's Box component with height creates empty lines for spacing
-      expect(output?.split('\n').length).toBeGreaterThan(5);
-    });
-  });
-
-  describe("empty state", () => {
-    test("shows 'No repositories found' when empty", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [] 
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("No repositories found");
-      // Should still show headers even when empty
-      expect(lastFrame()).toContain("Name");
-      // Headers use nerd font icons
-      expect(lastFrame()).toContain(""); // repo icon
-      expect(lastFrame()).toContain(""); // branch icon
-    });
-
-    test("shows filter hint when filterText is set", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [],
-        filterText: "test-filter"
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("No repositories found");
-      expect(lastFrame()).toContain("Try a different filter");
-    });
-
-    test("shows GitHub token hint when viewMode is 'github' and no repos", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [],
-        viewMode: "github",
-        githubRepos: []
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("No repositories found");
-      expect(lastFrame()).toContain("Set GITHUB_TOKEN to see GitHub repos");
-    });
-
-    test("shows GitHub error when githubError is set", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [],
-        viewMode: "github",
-        githubRepos: [],
-        githubError: "API rate limit exceeded"
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("No repositories found");
-      expect(lastFrame()).toContain("GitHub error: API rate limit exceeded");
-      expect(lastFrame()).not.toContain("Set GITHUB_TOKEN");
-    });
-  });
-
-  describe("list rendering", () => {
-    test("renders correct number of visible repos", () => {
-      const repos = [
-        createMockUnifiedRepo({ name: "repo1" }),
-        createMockUnifiedRepo({ name: "repo2" }),
-        createMockUnifiedRepo({ name: "repo3" }),
-      ];
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos
-      });
-      const { lastFrame } = renderProjectListWithContext(context, 10);
-
-      // Should show all 3 repos (with headers taking 2 lines, 8 lines left for repos)
-      expect(lastFrame()).toContain("repo1");
-      expect(lastFrame()).toContain("repo2");
-      expect(lastFrame()).toContain("repo3");
-    });
-
-    test("highlights cursor position correctly", () => {
-      const repos = [
-        createMockUnifiedRepo({ name: "repo1" }),
-        createMockUnifiedRepo({ name: "repo2" }),
-        createMockUnifiedRepo({ name: "repo3" }),
-      ];
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos,
-        cursorIndex: 1 // Second repo
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      // The cursor should be on the second repo (index 1)
-      // In the actual implementation, this would show with different styling
-      // For now we just verify the repo is rendered
-      expect(lastFrame()).toContain("repo2");
-    });
-
-    test("shows selection state correctly", () => {
-      const repos = [
-        createMockUnifiedRepo({ name: "repo1" }),
-        createMockUnifiedRepo({ name: "repo2" }),
-        createMockUnifiedRepo({ name: "repo3" }),
-      ];
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos,
-        selectedIndices: new Set([0, 2]) // First and third repos selected
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      // All repos should be rendered
-      expect(lastFrame()).toContain("repo1");
-      expect(lastFrame()).toContain("repo2");
-      expect(lastFrame()).toContain("repo3");
-      // The actual selection indicator would be shown by UnifiedProjectItem
-    });
-
-    test("renders UnifiedProjectItem for each visible repo", () => {
-      const repos = [
-        createLocalOnlyUnifiedRepo("local-repo"),
-        createGitHubOnlyUnifiedRepo("github-repo"),
-        createSyncedUnifiedRepo("synced-repo")
-      ];
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      // Should render all three repos with their respective types
-      expect(lastFrame()).toContain("local-repo");
-      expect(lastFrame()).toContain("github-repo");
-      expect(lastFrame()).toContain("synced-repo");
-    });
-
-    test("always renders ColumnHeader", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [createMockUnifiedRepo({ name: "repo1" })]
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      expect(lastFrame()).toContain("Name");
-      // Headers use nerd font icons
-      expect(lastFrame()).toContain(""); // repo icon
-      expect(lastFrame()).toContain(""); // branch icon
-    });
-
-    test("ColumnSeparator returns null (no visual separator)", () => {
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: [createMockUnifiedRepo({ name: "repo1" })]
-      });
-      const { lastFrame } = renderProjectListWithContext(context);
-
-      // ColumnSeparator component returns null, so no separator line is rendered
-      // The separation is handled by spacing in the layout
-      expect(lastFrame()).not.toMatch(/[-─]/);
-    });
-  });
-
-  describe("scrolling", () => {
-    test("calculates visible range correctly with many repos", () => {
-      const repos = Array.from({ length: 50 }, (_, i) => 
-        createMockUnifiedRepo({ name: `repo${i}` })
-      );
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos
-      });
-      // Height of 10 means 8 lines for repos (2 for headers)
-      const { lastFrame } = renderProjectListWithContext(context, 10);
-
-      // Should only show first 8 repos
-      expect(lastFrame()).toContain("repo0");
-      expect(lastFrame()).toContain("repo7");
-      expect(lastFrame()).not.toContain("repo8");
-      // Should show scroll indicator
-      expect(lastFrame()).toContain("more below");
-    });
-
-    test("shows scroll indicator when more items below", () => {
-      const repos = Array.from({ length: 20 }, (_, i) => 
-        createMockUnifiedRepo({ name: `repo${i}` })
-      );
-      
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos
-      });
-      const { lastFrame } = renderProjectListWithContext(context, 8); // 6 visible repos
-
-      // Should show scroll indicator with remaining count
-      expect(lastFrame()).toContain("more below");
-      expect(lastFrame()).toMatch(/\d+ more below/);
-    });
-
-    test("adjusts startIndex near end of list", () => {
-      const repos = Array.from({ length: 10 }, (_, i) => 
-        createMockUnifiedRepo({ name: `repo${i}` })
-      );
-      
-      // Cursor at the end
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos,
-        cursorIndex: 9 // Last repo
-      });
-      const { lastFrame } = renderProjectListWithContext(context, 6); // 4 visible repos
-
-      // Should show the last repos
-      expect(lastFrame()).toContain("repo9");
-      expect(lastFrame()).toContain("repo8");
-      expect(lastFrame()).toContain("repo7");
-      expect(lastFrame()).toContain("repo6");
-      expect(lastFrame()).not.toContain("repo0");
-    });
-
-    test("centers cursor in visible area", () => {
-      const repos = Array.from({ length: 30 }, (_, i) => 
-        createMockUnifiedRepo({ name: `repo${i}` })
-      );
-      
-      // Cursor in the middle
-      const context = createMockStoreContext({ 
-        isLoading: false,
-        unifiedRepos: repos,
-        cursorIndex: 15
-      });
-      const { lastFrame } = renderProjectListWithContext(context, 10); // 8 visible repos
-
-      // With cursor at 15 and 8 visible repos, startIndex should be around 11
-      // So we should see repos 11-18
-      expect(lastFrame()).toContain("repo11");
-      expect(lastFrame()).toContain("repo15"); // Cursor should be visible
-      expect(lastFrame()).toContain("repo18");
-      // Should not see repos much earlier or later
-      expect(lastFrame()).not.toContain("repo10");
-      expect(lastFrame()).not.toContain("repo19");
-    });
-  });
-});