@elisra-devops/docgen-data-provider 1.83.0 → 1.85.0

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

@@ -1078,6 +1078,67 @@ describe('ResultDataProvider', () => {
             expect(esuk).toBe('ESUK');
             expect(il).toBe('IL');
         });
+        it('should derive responsibility from Area Path alias when System.AreaPath is missing', () => {
+            const esuk = resultDataProvider.deriveMewpResponsibility({
+                'Custom.SAPWBS': '',
+                'Area Path': 'MEWP\\Customer Requirements\\Level 2\\ATP\\ESUK',
+            });
+            const il = resultDataProvider.deriveMewpResponsibility({
+                'Custom.SAPWBS': '',
+                'Area Path': 'MEWP\\Customer Requirements\\Level 2\\ATP',
+            });
+            expect(esuk).toBe('ESUK');
+            expect(il).toBe('IL');
+        });
+        it('should derive test-case responsibility from testCasesItems area-path fields', async () => {
+            const fetchByIdsSpy = jest
+                .spyOn(resultDataProvider, 'fetchWorkItemsByIds')
+                .mockResolvedValue([]);
+            const map = await resultDataProvider.buildMewpTestCaseResponsibilityMap([
+                {
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 101,
+                                workItemFields: [{ name: 'Area Path', value: 'MEWP\\Customer Requirements\\Level 2\\ATP' }],
+                            },
+                        },
+                        {
+                            workItem: {
+                                id: 102,
+                                workItemFields: [
+                                    { referenceName: 'System.AreaPath', value: 'MEWP\\Customer Requirements\\Level 2\\ATP\\ESUK' },
+                                ],
+                            },
+                        },
+                    ],
+                    testPointsItems: [],
+                },
+            ], mockProjectName);
+            expect(map.get(101)).toBe('IL');
+            expect(map.get(102)).toBe('ESUK');
+            expect(fetchByIdsSpy).not.toHaveBeenCalled();
+        });
+        it('should derive test-case responsibility from AreaPath even when SAPWBS exists on test-case payload', async () => {
+            jest.spyOn(resultDataProvider, 'fetchWorkItemsByIds').mockResolvedValue([]);
+            const map = await resultDataProvider.buildMewpTestCaseResponsibilityMap([
+                {
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 201,
+                                workItemFields: [
+                                    { referenceName: 'Custom.SAPWBS', value: 'ESUK' },
+                                    { referenceName: 'System.AreaPath', value: 'MEWP\\Customer Requirements\\Level 2\\ATP' },
+                                ],
+                            },
+                        },
+                    ],
+                    testPointsItems: [],
+                },
+            ], mockProjectName);
+            expect(map.get(201)).toBe('IL');
+        });
         it('should zip bug rows with L3/L4 pairs and avoid cross-product duplication', () => {
             const requirements = [
                 {
@@ -1309,6 +1370,62 @@ describe('ResultDataProvider', () => {
             expect(rows[0]['Bug ID']).toBe(99001);
             expect(rows[0]['Bug Responsibility']).toBe('Elisra');
         });
+        it('should fallback bug responsibility from test case mapping when external bug row is unknown', () => {
+            const requirements = [
+                {
+                    requirementId: 'SR5310',
+                    baseKey: 'SR5310',
+                    title: 'Req 5310',
+                    subSystem: 'Power',
+                    responsibility: '',
+                    linkedTestCaseIds: [101],
+                },
+            ];
+            const requirementIndex = new Map([
+                [
+                    'SR5310',
+                    new Map([
+                        [
+                            101,
+                            {
+                                passed: 0,
+                                failed: 1,
+                                notRun: 0,
+                            },
+                        ],
+                    ]),
+                ],
+            ]);
+            const observedTestCaseIdsByRequirement = new Map([
+                ['SR5310', new Set([101])],
+            ]);
+            const linkedRequirementsByTestCase = new Map([
+                [
+                    101,
+                    {
+                        baseKeys: new Set(['SR5310']),
+                        fullCodes: new Set(['SR5310']),
+                        bugIds: new Set([10003]),
+                    },
+                ],
+            ]);
+            const externalBugsByTestCase = new Map([
+                [
+                    101,
+                    [
+                        {
+                            id: 10003,
+                            title: 'Bug 10003',
+                            responsibility: 'Unknown',
+                            requirementBaseKey: 'SR5310',
+                        },
+                    ],
+                ],
+            ]);
+            const rows = resultDataProvider.buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, new Map(), externalBugsByTestCase, new Map(), new Map([[101, 'IL']]));
+            expect(rows).toHaveLength(1);
+            expect(rows[0]['Bug Responsibility']).toBe('Elisra');
+        });
     });
     describe('getMewpInternalValidationFlatResults', () => {
         it('should skip test cases with no in-scope expected requirements and no linked requirements', async () => {
@@ -2028,6 +2145,68 @@ describe('ResultDataProvider', () => {
             expect(String(result.rows[0]['Mentioned but Not Linked'] || '')).not.toContain('VVRM');
             expect(String(result.rows[0]['Linked but Not Mentioned'] || '')).not.toContain('VVRM');
         });
+        it('should fallback to work-item fields for steps XML when suite payload has no workItemFields', async () => {
+            jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
+            jest.spyOn(resultDataProvider, 'fetchMewpScopedTestData').mockResolvedValueOnce([
+                {
+                    testPointsItems: [{ testCaseId: 501, testCaseName: 'TC 501' }],
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 501,
+                                workItemFields: [],
+                            },
+                        },
+                    ],
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'fetchMewpL2Requirements').mockResolvedValueOnce([
+                {
+                    workItemId: 9001,
+                    requirementId: 'SR0501',
+                    baseKey: 'SR0501',
+                    title: 'Req 501',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [501],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'buildLinkedRequirementsByTestCase').mockResolvedValueOnce(new Map([
+                [
+                    501,
+                    {
+                        baseKeys: new Set(['SR0501']),
+                        fullCodes: new Set(['SR0501']),
+                    },
+                ],
+            ]));
+            jest.spyOn(resultDataProvider, 'fetchWorkItemsByIds').mockResolvedValueOnce([
+                {
+                    id: 501,
+                    fields: {
+                        'Microsoft.VSTS.TCM.Steps': '<steps><step id="2" type="ActionStep"><parameterizedString isformatted="true">Action</parameterizedString><parameterizedString isformatted="true">SR0501</parameterizedString></step></steps>',
+                    },
+                },
+            ]);
+            jest.spyOn(resultDataProvider.testStepParserHelper, 'parseTestSteps').mockResolvedValueOnce([
+                {
+                    stepId: '1',
+                    stepPosition: '1',
+                    action: 'Action',
+                    expected: 'SR0501',
+                    isSharedStepTitle: false,
+                },
+            ]);
+            const result = await resultDataProvider.getMewpInternalValidationFlatResults('123', mockProjectName, [1]);
+            expect(resultDataProvider.fetchWorkItemsByIds).toHaveBeenCalled();
+            expect(result.rows).toHaveLength(1);
+            expect(result.rows[0]).toEqual(expect.objectContaining({
+                'Test Case ID': 501,
+                'Mentioned but Not Linked': '',
+                'Linked but Not Mentioned': '',
+                'Validation Status': 'Pass',
+            }));
+        });
     });
     describe('MEWP rel fallback scoping', () => {
         it('should fallback to previous Rel run when latest selected Rel has no run for a test case', async () => {
@@ -2216,6 +2395,36 @@ describe('ResultDataProvider', () => {
                 responsibility: 'ESUK',
             }));
         });
+        it('should resolve external bug responsibility from AreaPath columns when SAPWBS is empty', async () => {
+            const mewpExternalTableUtils = resultDataProvider.mewpExternalTableUtils;
+            jest.spyOn(mewpExternalTableUtils, 'loadExternalTableRows').mockResolvedValueOnce([
+                {
+                    Elisra_SortIndex: '101',
+                    SR: 'SR0001',
+                    TargetWorkItemId: '9011',
+                    Title: 'Bug from ATP\\ESUK path',
+                    TargetState: 'Active',
+                    SAPWBS: '',
+                    AreaPath: 'MEWP\\Customer Requirements\\Level 2\\ATP\\ESUK',
+                },
+                {
+                    Elisra_SortIndex: '102',
+                    SR: 'SR0002',
+                    TargetWorkItemId: '9012',
+                    Title: 'Bug from ATP path',
+                    TargetState: 'Active',
+                    SAPWBS: '',
+                    'System.AreaPath': 'MEWP\\Customer Requirements\\Level 2\\ATP',
+                },
+            ]);
+            const map = await resultDataProvider.loadExternalBugsByTestCase(validBugsSource);
+            const bugs101 = map.get(101) || [];
+            const bugs102 = map.get(102) || [];
+            expect(bugs101).toHaveLength(1);
+            expect(bugs102).toHaveLength(1);
+            expect(bugs101[0].responsibility).toBe('ESUK');
+            expect(bugs102[0].responsibility).toBe('Elisra');
+        });
         it('should require Elisra_SortIndex and ignore rows that only provide WorkItemId', async () => {
             const mewpExternalTableUtils = resultDataProvider.mewpExternalTableUtils;
             jest.spyOn(mewpExternalTableUtils, 'loadExternalTableRows').mockResolvedValueOnce([