test: add 21 tests for lib/git.ts (run, helpers, fallbacks)

Author: TerminalGravity

tests/lib/git.test.ts

@@ -0,0 +1,178 @@
+import { describe, it, expect, vi, beforeEach } from "vitest";
+import * as childProcess from "child_process";
+import {
+  run,
+  getBranch,
+  getStatus,
+  getRecentCommits,
+  getLastCommit,
+  getLastCommitTime,
+  getDiffFiles,
+  getStagedFiles,
+  getDiffStat,
+} from "../../src/lib/git.js";
+
+vi.mock("child_process", () => ({
+  execFileSync: vi.fn(),
+}));
+
+const mockExec = vi.mocked(childProcess.execFileSync);
+
+beforeEach(() => {
+  mockExec.mockReset();
+});
+
+describe("run()", () => {
+  it("passes array args directly to execFileSync", () => {
+    mockExec.mockReturnValue("ok\n" as any);
+    const result = run(["status", "--short"]);
+    expect(result).toBe("ok");
+    expect(mockExec).toHaveBeenCalledWith(
+      "git",
+      ["status", "--short"],
+      expect.objectContaining({ encoding: "utf-8" }),
+    );
+  });
+
+  it("splits string args on whitespace", () => {
+    mockExec.mockReturnValue("main\n" as any);
+    const result = run("branch --show-current");
+    expect(result).toBe("main");
+    expect(mockExec).toHaveBeenCalledWith(
+      "git",
+      ["branch", "--show-current"],
+      expect.any(Object),
+    );
+  });
+
+  it("trims output", () => {
+    mockExec.mockReturnValue("  abc123 some commit  \n" as any);
+    expect(run(["log", "--oneline", "-1"])).toBe("abc123 some commit");
+  });
+
+  it("returns timeout message when process is killed", () => {
+    const err: any = new Error("timed out");
+    err.killed = true;
+    mockExec.mockImplementation(() => { throw err; });
+    expect(run(["status"])).toBe("[timed out after 10000ms]");
+  });
+
+  it("returns stderr on failure", () => {
+    const err: any = new Error("fail");
+    err.stderr = "fatal: not a git repo\n";
+    err.stdout = "";
+    mockExec.mockImplementation(() => { throw err; });
+    expect(run(["status"])).toBe("fatal: not a git repo");
+  });
+
+  it("returns ENOENT message when git is missing", () => {
+    const err: any = new Error("spawn git ENOENT");
+    err.code = "ENOENT";
+    mockExec.mockImplementation(() => { throw err; });
+    expect(run(["status"])).toBe("[git not found]");
+  });
+
+  it("returns command failed for unknown errors", () => {
+    const err: any = new Error("unknown");
+    err.status = 128;
+    mockExec.mockImplementation(() => { throw err; });
+    expect(run(["push"])).toBe("[command failed: git push (exit 128)]");
+  });
+
+  it("respects custom timeout", () => {
+    const err: any = new Error("timed out");
+    err.killed = true;
+    mockExec.mockImplementation(() => { throw err; });
+    expect(run(["status"], { timeout: 5000 })).toBe("[timed out after 5000ms]");
+  });
+});
+
+describe("convenience helpers", () => {
+  it("getBranch calls branch --show-current", () => {
+    mockExec.mockReturnValue("feature/foo\n" as any);
+    expect(getBranch()).toBe("feature/foo");
+  });
+
+  it("getStatus calls status --short", () => {
+    mockExec.mockReturnValue(" M src/index.ts\n" as any);
+    expect(getStatus()).toBe("M src/index.ts");
+  });
+
+  it("getRecentCommits defaults to 5", () => {
+    mockExec.mockReturnValue("abc one\ndef two\n" as any);
+    const result = getRecentCommits();
+    expect(mockExec).toHaveBeenCalledWith(
+      "git",
+      ["log", "--oneline", "-5"],
+      expect.any(Object),
+    );
+    expect(result).toBe("abc one\ndef two");
+  });
+
+  it("getRecentCommits accepts custom count", () => {
+    mockExec.mockReturnValue("abc one\n" as any);
+    getRecentCommits(3);
+    expect(mockExec).toHaveBeenCalledWith(
+      "git",
+      ["log", "--oneline", "-3"],
+      expect.any(Object),
+    );
+  });
+
+  it("getLastCommit returns single line", () => {
+    mockExec.mockReturnValue("abc123 fix: something\n" as any);
+    expect(getLastCommit()).toBe("abc123 fix: something");
+  });
+
+  it("getLastCommitTime returns timestamp", () => {
+    mockExec.mockReturnValue("2026-03-12 09:00:00 -0700\n" as any);
+    expect(getLastCommitTime()).toBe("2026-03-12 09:00:00 -0700");
+  });
+
+  it("getStagedFiles returns file list", () => {
+    mockExec.mockReturnValue("src/a.ts\nsrc/b.ts\n" as any);
+    expect(getStagedFiles()).toBe("src/a.ts\nsrc/b.ts");
+  });
+});
+
+describe("getDiffFiles", () => {
+  it("returns diff output on success", () => {
+    mockExec.mockReturnValue("src/a.ts\nsrc/b.ts\n" as any);
+    expect(getDiffFiles("HEAD~3")).toBe("src/a.ts\nsrc/b.ts");
+  });
+
+  it("falls back to HEAD~1 when primary ref fails", () => {
+    mockExec
+      .mockReturnValueOnce("[command failed: git diff (exit 128)]" as any)
+      .mockReturnValueOnce("src/c.ts\n" as any);
+    expect(getDiffFiles("origin/main")).toBe("src/c.ts");
+  });
+
+  it("returns 'no commits' when both attempts fail", () => {
+    mockExec
+      .mockReturnValueOnce("[command failed]" as any)
+      .mockReturnValueOnce("[command failed]" as any);
+    expect(getDiffFiles()).toBe("no commits");
+  });
+});
+
+describe("getDiffStat", () => {
+  it("returns stat output on success", () => {
+    mockExec.mockReturnValue(" 2 files changed, 10 insertions(+)\n" as any);
+    expect(getDiffStat("HEAD~5")).toBe("2 files changed, 10 insertions(+)");
+  });
+
+  it("falls back to HEAD~3 when primary ref fails", () => {
+    mockExec
+      .mockReturnValueOnce("[command failed]" as any)
+      .mockReturnValueOnce(" 1 file changed\n" as any);
+    expect(getDiffStat("origin/main")).toBe("1 file changed");
+  });
+
+  it("returns fallback message when both attempts fail", () => {
+    mockExec
+      .mockReturnValueOnce("[timed out]" as any)
+      .mockReturnValueOnce("[timed out]" as any);
+    expect(getDiffStat()).toBe("no diff stats available");
+  });
+});