@elisra-devops/docgen-data-provider 1.102.0 → 1.104.0

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

@@ -894,6 +894,44 @@ describe('ResultDataProvider', () => {
         });
     });
     describe('getMewpL2CoverageFlatResults', () => {
+        it('should split SR prefix into SR # + L2 REQ Title while preserving full title', () => {
+            const row = resultDataProvider.createMewpCoverageRow({
+                workItemId: 5054,
+                requirementId: 'SR0054',
+                title: 'SR0054 - Engine startup coverage',
+                owner: 'ESUK',
+                subSystem: 'Propulsion',
+                responsibility: 'ESUK',
+            }, 'Pass', { id: '', title: '', responsibility: '' }, { l3Id: '', l3Title: '', l4Id: '', l4Title: '' });
+            expect(row).toEqual(expect.objectContaining({
+                'L2 REQ ID': '5054',
+                'SR #': 'SR0054',
+                'L2 REQ Title': 'Engine startup coverage',
+                'L2 REQ Full Title': 'SR0054 - Engine startup coverage',
+                'L2 Owner': 'ESUK',
+                'L2 SubSystem': 'Propulsion',
+            }));
+        });
+        it('should keep full title when title is only the SR token', () => {
+            const row = resultDataProvider.createMewpCoverageRow({
+                workItemId: 9999,
+                requirementId: 'SR9999',
+                title: 'SR9999',
+                owner: 'IL',
+                subSystem: '',
+                responsibility: 'IL',
+            }, 'Not Run', { id: '', title: '', responsibility: '' }, { l3Id: '', l3Title: '', l4Id: '', l4Title: '' });
+            expect(row['L2 REQ Title']).toBe('SR9999');
+            expect(row['L2 REQ Full Title']).toBe('SR9999');
+            expect(row['L2 Owner']).toBe('IL');
+        });
+        it('should resolve L2 run status precedence: Fail > Not Run > Pass', () => {
+            const resolve = resultDataProvider.resolveMewpL2RunStatus.bind(resultDataProvider);
+            expect(resolve({ passed: 2, failed: 1, notRun: 3, hasAnyTestCase: true })).toBe('Fail');
+            expect(resolve({ passed: 5, failed: 0, notRun: 1, hasAnyTestCase: true })).toBe('Not Run');
+            expect(resolve({ passed: 1, failed: 0, notRun: 0, hasAnyTestCase: true })).toBe('Pass');
+            expect(resolve({ passed: 0, failed: 0, notRun: 0, hasAnyTestCase: true })).toBe('Not Run');
+        });
         it('should fetch MEWP scoped test data from selected suites', async () => {
             jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
             const scopedSpy = jest
@@ -987,7 +1025,7 @@ describe('ResultDataProvider', () => {
             const result = await resultDataProvider.getMewpL2CoverageFlatResults('123', mockProjectName, [1]);
             expect(result).toEqual(expect.objectContaining({
                 sheetName: expect.stringContaining('MEWP L2 Coverage'),
-                columnOrder: expect.arrayContaining(['L2 REQ ID', 'L2 REQ Title', 'L2 Run Status']),
+                columnOrder: expect.arrayContaining(['L2 REQ ID', 'SR #', 'L2 REQ Title', 'L2 Owner', 'L2 Run Status']),
             }));
             const covered = result.rows.find((row) => row['L2 REQ ID'] === '5001');
             const inferredByStepText = result.rows.find((row) => row['L2 REQ ID'] === '5002');
@@ -2883,6 +2921,81 @@ describe('ResultDataProvider', () => {
                 'Validation Status': 'Pass',
             }));
         });
+        it('should prefer latest test-case revision fields (title/description/steps) in internal validation flow', async () => {
+            jest.spyOn(resultDataProvider, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
+            jest.spyOn(resultDataProvider, 'fetchMewpScopedTestData').mockResolvedValueOnce([
+                {
+                    testPointsItems: [{ testCaseId: 777, testCaseName: 'TC 777 (snapshot)' }],
+                    testCasesItems: [
+                        {
+                            workItem: {
+                                id: 777,
+                                workItemFields: [
+                                    { key: 'System.Title', value: 'TC 777 (snapshot title)' },
+                                    {
+                                        key: 'Microsoft.VSTS.TCM.Steps',
+                                        value: '<steps><step id="2" type="ActionStep"><parameterizedString isformatted="true">Action</parameterizedString><parameterizedString isformatted="true">SR0001</parameterizedString></step></steps>',
+                                    },
+                                    { key: 'System.Description', value: '<p>Snapshot description</p>' },
+                                ],
+                            },
+                        },
+                    ],
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'fetchMewpL2Requirements').mockResolvedValueOnce([
+                {
+                    workItemId: 9777,
+                    requirementId: 'SR7777',
+                    baseKey: 'SR7777',
+                    title: 'Req 7777',
+                    responsibility: 'ESUK',
+                    linkedTestCaseIds: [777],
+                    areaPath: 'MEWP\\Customer Requirements\\Level 2',
+                },
+            ]);
+            jest.spyOn(resultDataProvider, 'buildLinkedRequirementsByTestCase').mockResolvedValueOnce(new Map([
+                [
+                    777,
+                    {
+                        baseKeys: new Set(['SR7777']),
+                        fullCodes: new Set(['SR7777']),
+                    },
+                ],
+            ]));
+            const fetchWorkItemsByIdsSpy = jest
+                .spyOn(resultDataProvider, 'fetchWorkItemsByIds')
+                .mockResolvedValueOnce([
+                {
+                    id: 777,
+                    fields: {
+                        'System.Title': 'TC 777 (latest title)',
+                        'System.Description': '<p><b><u>Trial specific assumptions, constraints, dependencies and requirements</u></b></p><p>SR7777</p>',
+                        'Microsoft.VSTS.TCM.Steps': '<steps><step id="2" type="ActionStep"><parameterizedString isformatted="true">Action</parameterizedString><parameterizedString isformatted="true">SR7777</parameterizedString></step></steps>',
+                    },
+                },
+            ]);
+            jest.spyOn(resultDataProvider.testStepParserHelper, 'parseTestSteps').mockImplementation(async (...args) => [
+                {
+                    stepId: '1',
+                    stepPosition: '1',
+                    action: 'Action',
+                    expected: String((args === null || args === void 0 ? void 0 : args[0]) || '').includes('SR7777') ? 'SR7777' : 'SR0001',
+                    isSharedStepTitle: false,
+                },
+            ]);
+            const result = await resultDataProvider.getMewpInternalValidationFlatResults('123', mockProjectName, [1]);
+            expect(fetchWorkItemsByIdsSpy).toHaveBeenCalledWith(mockProjectName, [777], false);
+            expect(resultDataProvider.testStepParserHelper.parseTestSteps).toHaveBeenCalledWith(expect.stringContaining('SR7777'), expect.any(Map));
+            expect(result.rows).toHaveLength(1);
+            expect(result.rows[0]).toEqual(expect.objectContaining({
+                'Test Case ID': 777,
+                'Test Case Title': 'TC 777 (latest title)',
+                'Mentioned but Not Linked': '',
+                'Linked but Not Mentioned': '',
+                'Validation Status': 'Pass',
+            }));
+        });
     });
     describe('buildLinkedRequirementsByTestCase', () => {
         it('should map linked requirements only for supported test-case requirement relation types', async () => {