@elisra-devops/docgen-data-provider 1.90.0 → 1.92.0

package/bin/tests/modules/ResultDataProvider.test.js

@@ -1557,7 +1557,7 @@ describe('ResultDataProvider', () => {
                 'Validation Status': 'Fail',
             }));
         });
-        it('should emit Direction A rows with step context for missing sibling links', async () => {
+        it('should emit Direction A rows only for specifically mentioned child requirements', async () => {
             jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
             jest.spyOn(resultDataProvider, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
             jest.spyOn(resultDataProvider, 'fetchTestData').mockResolvedValueOnce([
@@ -1599,7 +1599,7 @@ describe('ResultDataProvider', () => {
                     stepId: '2',
                     stepPosition: '2',
                     action: '',
-                    expected: 'SR0001; SR0002',
+                    expected: 'SR0001-1',
                     isSharedStepTitle: false,
                 },
             ]);
@@ -1612,6 +1612,63 @@ describe('ResultDataProvider', () => {
                 }),
             ]));
         });
+        it('should keep explicit child IDs when multiple specifically mentioned children are missing', async () => {
+            jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
+            jest.spyOn(resultDataProvider, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
+            jest.spyOn(resultDataProvider, 'fetchTestData').mockResolvedValueOnce([
+                {
+                    testPointsItems: [{ testCaseId: 111, testCaseName: 'TC 111' }],
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 111,
+                                workItemFields: [{ key: 'Steps', value: '<steps></steps>' }],
+                            },
+                        },
+                    ],
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'fetchMewpL2Requirements').mockResolvedValueOnce([
+                {
+                    workItemId: 5101,
+                    requirementId: 'SR0054-1',
+                    baseKey: 'SR0054',
+                    title: 'Req child 1',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+                {
+                    workItemId: 5102,
+                    requirementId: 'SR0054-2',
+                    baseKey: 'SR0054',
+                    title: 'Req child 2',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+            ]);
+            jest
+                .spyOn(resultDataProvider, 'buildLinkedRequirementsByTestCase')
+                .mockResolvedValueOnce(new Map([[111, { baseKeys: new Set(), fullCodes: new Set() }]]));
+            jest.spyOn(resultDataProvider.testStepParserHelper, 'parseTestSteps').mockResolvedValueOnce([
+                {
+                    stepId: '4',
+                    stepPosition: '4',
+                    action: '',
+                    expected: 'SR0054-1; SR0054-2',
+                    isSharedStepTitle: false,
+                },
+            ]);
+            const result = await resultDataProvider.getMewpInternalValidationFlatResults('123', mockProjectName, [1]);
+            expect(result.rows).toHaveLength(1);
+            expect(result.rows[0]).toEqual(expect.objectContaining({
+                'Test Case ID': 111,
+                'Mentioned but Not Linked': 'Step 4: SR0054-1; SR0054-2',
+                'Linked but Not Mentioned': '',
+                'Validation Status': 'Fail',
+            }));
+        });
         it('should pass when a base SR mention is fully covered by its only linked child', async () => {
             jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
             jest.spyOn(resultDataProvider, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
@@ -1675,6 +1732,100 @@ describe('ResultDataProvider', () => {
                 'Validation Status': 'Pass',
             }));
         });
+        it('should support cross-test-case family coverage when siblings are linked on different test cases', async () => {
+            jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
+            jest.spyOn(resultDataProvider, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
+            jest.spyOn(resultDataProvider, 'fetchTestData').mockResolvedValueOnce([
+                {
+                    testPointsItems: [
+                        { testCaseId: 501, testCaseName: 'TC 501 - sibling 1' },
+                        { testCaseId: 502, testCaseName: 'TC 502 - sibling 2' },
+                    ],
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 501,
+                                workItemFields: [{ key: 'Steps', value: '<steps id=\"mock-steps-tc-501\"></steps>' }],
+                            },
+                        },
+                        {
+                            workItem: {
+                                id: 502,
+                                workItemFields: [{ key: 'Steps', value: '<steps id=\"mock-steps-tc-502\"></steps>' }],
+                            },
+                        },
+                    ],
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'fetchMewpL2Requirements').mockResolvedValueOnce([
+                {
+                    workItemId: 9301,
+                    requirementId: 'SR0054-1',
+                    baseKey: 'SR0054',
+                    title: 'SR0054 child 1',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+                {
+                    workItemId: 9302,
+                    requirementId: 'SR0054-2',
+                    baseKey: 'SR0054',
+                    title: 'SR0054 child 2',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'buildLinkedRequirementsByTestCase').mockResolvedValueOnce(new Map([
+                [
+                    501,
+                    {
+                        baseKeys: new Set(['SR0054']),
+                        fullCodes: new Set(['SR0054-1']),
+                    },
+                ],
+                [
+                    502,
+                    {
+                        baseKeys: new Set(['SR0054']),
+                        fullCodes: new Set(['SR0054-2']),
+                    },
+                ],
+            ]));
+            jest
+                .spyOn(resultDataProvider.testStepParserHelper, 'parseTestSteps')
+                .mockResolvedValueOnce([
+                {
+                    stepId: '1',
+                    stepPosition: '1',
+                    action: 'Parent mention on first test case',
+                    expected: 'SR0054',
+                    isSharedStepTitle: false,
+                },
+            ])
+                .mockResolvedValueOnce([
+                {
+                    stepId: '1',
+                    stepPosition: '1',
+                    action: 'Parent mention on second test case',
+                    expected: 'SR0054',
+                    isSharedStepTitle: false,
+                },
+            ]);
+            const result = await resultDataProvider.getMewpInternalValidationFlatResults('123', mockProjectName, [1]);
+            const byTestCase = new Map(result.rows.map((row) => [Number(row['Test Case ID']), row]));
+            expect(byTestCase.get(501)).toEqual(expect.objectContaining({
+                'Mentioned but Not Linked': '',
+                'Linked but Not Mentioned': '',
+                'Validation Status': 'Pass',
+            }));
+            expect(byTestCase.get(502)).toEqual(expect.objectContaining({
+                'Mentioned but Not Linked': '',
+                'Linked but Not Mentioned': '',
+                'Validation Status': 'Pass',
+            }));
+        });
         it('should group linked-but-not-mentioned requirements by SR family', async () => {
             jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
             jest.spyOn(resultDataProvider, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
@@ -1741,7 +1892,9 @@ describe('ResultDataProvider', () => {
             const result = await resultDataProvider.getMewpInternalValidationFlatResults('123', mockProjectName, [1]);
             expect(result.rows).toHaveLength(1);
             expect(String(result.rows[0]['Mentioned but Not Linked'] || '')).toBe('');
-            expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).toContain('SR0054: SR0054-1, SR0054-2');
+            expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).toContain('SR0054');
+            expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).not.toContain('SR0054-1');
+            expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).not.toContain('SR0054-2');
             expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).toContain('SR0100-1');
         });
         it('should report only base SR when an entire mentioned family is uncovered', async () => {
@@ -2111,7 +2264,7 @@ describe('ResultDataProvider', () => {
             expect(new Set(result.rows.map((row) => row['Test Case ID']))).toEqual(new Set([201, 202, 203]));
             expect(byTestCase.get(201)).toEqual(expect.objectContaining({
                 'Test Case Title': 'TC 201 - Mixed discrepancies',
-                'Mentioned but Not Linked': 'Step 1: SR0095-3\nSR0511: SR0511-1, SR0511-2',
+                'Mentioned but Not Linked': 'Step 1: SR0095-3; SR0511',
                 'Linked but Not Mentioned': 'SR8888',
                 'Validation Status': 'Fail',
             }));