root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(FN-4521): complete Step 3 — unify active diff data source

Browse Source 
Fusion-Task-Id: FN-4521
Fusion-Task-Lineage: 675472a2-480d-4b9e-8861-98d2d82680db
This commit is contained in:
Fusion
2026-05-14 13:31:59 -07:00
committed by gsxdsm
parent c72dc6f3fc
commit a5678527dd
5 changed files with 57 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
packages/dashboard/app/hooks/__tests__/useTaskDiffStats.test.ts
View File

@@ -53,25 +53,25 @@ describe("useTaskDiffStats", () => {
expect(mockFetchTaskDiff).toHaveBeenCalledWith("FN-123", undefined, "proj-1");
});
it("does not fetch for active columns without a worktree", async () => {
it("fetches for active columns without a worktree", async () => {
mockFetchTaskDiff.mockResolvedValue({
files: [],
stats: { filesChanged: 2, additions: 4, deletions: 1 },
});
const { result: inProgress } = renderHook(() =>
useTaskDiffStats("FN-123", "in-progress", "abc1234", undefined),
);
const { result: todo } = renderHook(() =>
useTaskDiffStats("FN-123", "todo", "abc1234", undefined),
);
const { result: inReview } = renderHook(() =>
useTaskDiffStats("FN-123", "in-review", "abc1234", undefined),
);
await waitFor(() => expect(inProgress.current.loading).toBe(false));
await waitFor(() => expect(todo.current.loading).toBe(false));
await waitFor(() => expect(inReview.current.loading).toBe(false));
expect(inProgress.current.stats).toBeNull();
expect(todo.current.stats).toBeNull();
expect(inReview.current.stats).toBeNull();
expect(mockFetchTaskDiff).not.toHaveBeenCalled();
expect(inProgress.current.stats).toEqual({ filesChanged: 2, additions: 4, deletions: 1 });
expect(inReview.current.stats).toEqual({ filesChanged: 2, additions: 4, deletions: 1 });
expect(mockFetchTaskDiff).toHaveBeenCalledWith("FN-123", undefined, undefined);
});
it("fetches diff stats for active tasks with a worktree", async () => {
@@ -169,6 +169,16 @@ describe("useTaskDiffStats", () => {
expect(mockFetchTaskDiff).toHaveBeenCalledTimes(2);
});
it("does not fetch for inactive columns", async () => {
const { result: todo } = renderHook(() =>
useTaskDiffStats("FN-123", "todo", "abc1234", undefined),
);
await waitFor(() => expect(todo.current.loading).toBe(false));
expect(todo.current.stats).toBeNull();
expect(mockFetchTaskDiff).not.toHaveBeenCalled();
});
it("resets stats when column changes from done to non-done", async () => {
mockFetchTaskDiff.mockResolvedValueOnce({
files: [],