marketing-shibata50/multica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(test): update mockListIssues for two-phase fetch (open_only + closed)

Browse source 
issueListOptions now makes 2 api.listIssues calls (open_only + closed page).
Tests that mock the response must return data only for the open_only call,
otherwise issues appear twice in the merged result.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Naiyuan Qing 2026-04-08 10:41:29 +08:00
parent 8cf78b7a47
commit eed8e36a69
1 changed files with 10 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

14
apps/web/app/(dashboard)/issues/page.test.tsx
View file

@ -295,7 +295,10 @@ describe("IssuesPage", () => {
});
it("renders issues in board view after loading", async () => {
mockListIssues.mockResolvedValue({ issues: mockIssues, total: mockIssues.length });
// issueListOptions makes 2 calls: open_only + closed page. Return issues for open, empty for closed.
mockListIssues.mockImplementation((params: any) =>
Promise.resolve(params?.open_only ? { issues: mockIssues, total: mockIssues.length } : { issues: [], total: 0 }),
);
renderWithQuery(<IssuesPage />);
@ -305,7 +308,9 @@ describe("IssuesPage", () => {
});
it("renders board columns", async () => {
mockListIssues.mockResolvedValue({ issues: mockIssues, total: mockIssues.length });
mockListIssues.mockImplementation((params: any) =>
Promise.resolve(params?.open_only ? { issues: mockIssues, total: mockIssues.length } : { issues: [], total: 0 }),
);
renderWithQuery(<IssuesPage />);
@ -331,11 +336,12 @@ describe("IssuesPage", () => {
});
it("shows filter and display icon buttons", async () => {
mockListIssues.mockResolvedValue({ issues: mockIssues, total: mockIssues.length });
mockListIssues.mockImplementation((params: any) =>
Promise.resolve(params?.open_only ? { issues: mockIssues, total: mockIssues.length } : { issues: [], total: 0 }),
);
renderWithQuery(<IssuesPage />);
// Wait for query to resolve and component to render past loading state
await screen.findByText("Implement auth");
const buttons = screen.getAllByRole("button");
expect(buttons.length).toBeGreaterThan(0);