test: migrate dashboard and run detail v2 fixtures

Author: Optic00

frontend/src/lib/features/testing/testRunDetailData.test.js

@@ -7,16 +7,23 @@ describe('test run detail request graph', () => {
       tests: {
         testRuns: {
           getDetail: vi.fn().mockResolvedValue({
-            run: { id: 9, set_id: 4 },
-            test_cases: [{ id: 1, test_steps: [{ id: 11 }] }, { id: 2 }],
+            run: { id: 9, plan_id: 4 },
+            test_cases: [
+              { id: 1, test_steps: [{ id: 11 }] },
+              { id: 2, test_steps: [{ id: 12 }] },
+              { id: 3 },
+            ],
             results: [{ id: 21, test_case_id: 1 }],
-            step_results: { '1_11': { step_id: 11, status: 'passed' } },
+            step_results: [
+              { test_case_id: 1, step_id: 11, status: 'passed' },
+              { test_case_id: 2, step_id: 12, status: 'failed' },
+            ],
           }),
           get: vi.fn(),
           getResults: vi.fn(),
           getStepResults: vi.fn(),
         },
-        testSets: {
+        testPlans: {
           get: vi.fn(),
           getTestCases: vi.fn(),
         },
@@ -31,17 +38,21 @@ describe('test run detail request graph', () => {
     expect(apiClient.tests.testRuns.get).not.toHaveBeenCalled();
     expect(apiClient.tests.testRuns.getResults).not.toHaveBeenCalled();
     expect(apiClient.tests.testRuns.getStepResults).not.toHaveBeenCalled();
-    expect(apiClient.tests.testSets.get).not.toHaveBeenCalled();
-    expect(apiClient.tests.testSets.getTestCases).not.toHaveBeenCalled();
+    expect(apiClient.tests.testPlans.get).not.toHaveBeenCalled();
+    expect(apiClient.tests.testPlans.getTestCases).not.toHaveBeenCalled();
     expect(apiClient.tests.testCases.steps.getAll).not.toHaveBeenCalled();
     expect(detail).toEqual({
-      run: { id: 9, set_id: 4 },
+      run: { id: 9, plan_id: 4 },
       testCases: [
         { id: 1, test_steps: [{ id: 11 }] },
-        { id: 2, test_steps: [] },
+        { id: 2, test_steps: [{ id: 12 }] },
+        { id: 3, test_steps: [] },
       ],
       results: [{ id: 21, test_case_id: 1 }],
-      stepResults: { '1_11': { step_id: 11, status: 'passed' } },
+      stepResults: {
+        '1_11': { test_case_id: 1, step_id: 11, status: 'passed' },
+        '2_12': { test_case_id: 2, step_id: 12, status: 'failed' },
+      },
     });
   });
 
@@ -52,4 +63,24 @@ describe('test run detail request graph', () => {
 
     await expect(loadTestRunDetail(apiClient, 3, 9)).rejects.toThrow('Test run not found');
   });
+
+  it('normalizes missing optional graph lists without additional requests', async () => {
+    const getDetail = vi.fn().mockResolvedValue({ run: { id: 9, plan_id: 4 } });
+    const apiClient = { tests: { testRuns: { getDetail } } };
+    await expect(loadTestRunDetail(apiClient, 3, 9)).resolves.toEqual({
+      run: { id: 9, plan_id: 4 },
+      testCases: [],
+      results: [],
+      stepResults: {},
+    });
+    expect(getDetail).toHaveBeenCalledExactlyOnceWith(3, 9);
+  });
+
+  it('propagates a failed aggregate request', async () => {
+    const failure = new Error('Run unavailable');
+    const getDetail = vi.fn().mockRejectedValue(failure);
+    const apiClient = { tests: { testRuns: { getDetail } } };
+    await expect(loadTestRunDetail(apiClient, 3, 9)).rejects.toBe(failure);
+    expect(getDetail).toHaveBeenCalledExactlyOnceWith(3, 9);
+  });
 });

frontend/src/lib/widgets/dashboard/taskWidgetState.test.js

@@ -85,28 +85,50 @@ describe('normalizeTaskResponse', () => {
   ];
 
   test('sorts by due date (earliest first, nulls last)', () => {
-    const result = normalizeTaskResponse(items);
+    const result = normalizeTaskResponse({ data: items });
     expect(result.map((t) => t.id)).toEqual([3, 1, 4, 2]);
   });
 
   test('respects numeric maxItems cap', () => {
-    const result = normalizeTaskResponse(items, 2);
+    const result = normalizeTaskResponse({ data: items }, 2);
     expect(result).toHaveLength(2);
     expect(result[0].id).toBe(3); // earliest due date
   });
 
   test("'all' returns every item", () => {
-    const result = normalizeTaskResponse(items, 'all');
+    const result = normalizeTaskResponse({ data: items }, 'all');
     expect(result).toHaveLength(4);
   });
 
-  test('handles wrapped { items: [...] } responses', () => {
-    const result = normalizeTaskResponse({ items });
+  test('handles v2 data and pagination without rendering metadata', () => {
+    const result = normalizeTaskResponse({
+      data: items,
+      pagination: { page: 1, page_size: 50, total_items: 4, total_pages: 1 },
+    });
     expect(result).toHaveLength(4);
   });
 
   test('filters out entries without an id', () => {
-    const result = normalizeTaskResponse([{ title: 'no id' }, ...items]);
+    const result = normalizeTaskResponse({ data: [null, { title: 'no id' }, ...items] });
     expect(result).toHaveLength(4);
   });
+
+  test('normalizes due dates without mutating the source objects or ordering', () => {
+    const source = Object.freeze(items.map((item) => Object.freeze({ ...item })));
+    const result = normalizeTaskResponse({ data: source });
+    expect(result.map((item) => item.dueDate)).toEqual([
+      '2026-06-01T00:00:00Z',
+      '2026-06-05T00:00:00Z',
+      '2026-06-10T00:00:00Z',
+      null,
+    ]);
+    expect(source.map((item) => item.id)).toEqual([1, 2, 3, 4]);
+    expect(source.every((item) => !Object.hasOwn(item, 'dueDate'))).toBe(true);
+  });
+
+  test('returns no rows for empty or absent v2 data', () => {
+    for (const response of [null, undefined, {}, { data: [] }]) {
+      expect(normalizeTaskResponse(response)).toEqual([]);
+    }
+  });
 });