@elisra-devops/docgen-data-provider 1.80.0 → 1.81.0

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

@@ -1078,7 +1078,7 @@ describe('ResultDataProvider', () => {
             expect(esuk).toBe('ESUK');
             expect(il).toBe('IL');
         });
-        it('should emit additive rows for bugs and L3/L4 links without bug duplication', () => {
+        it('should zip bug rows with L3/L4 pairs and avoid cross-product duplication', () => {
             const requirements = [
                 {
                     requirementId: 'SR5303',
@@ -1130,21 +1130,20 @@ describe('ResultDataProvider', () => {
                 [
                     'SR5303',
                     [
-                        { id: '9003', title: 'L3 9003', level: 'L3' },
-                        { id: '9103', title: 'L4 9103', level: 'L4' },
+                        { l3Id: '9003', l3Title: 'L3 9003', l4Id: '9103', l4Title: 'L4 9103' },
                     ],
                 ],
             ]);
             const rows = resultDataProvider.buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, l3l4ByBaseKey, externalBugsByTestCase);
-            expect(rows).toHaveLength(4);
-            expect(rows.map((row) => row['Bug ID'])).toEqual([10003, 20003, '', '']);
-            expect(rows[2]).toEqual(expect.objectContaining({
+            expect(rows).toHaveLength(2);
+            expect(rows.map((row) => row['Bug ID'])).toEqual([10003, 20003]);
+            expect(rows[0]).toEqual(expect.objectContaining({
                 'L3 REQ ID': '9003',
-                'L4 REQ ID': '',
+                'L4 REQ ID': '9103',
             }));
-            expect(rows[3]).toEqual(expect.objectContaining({
+            expect(rows[1]).toEqual(expect.objectContaining({
                 'L3 REQ ID': '',
-                'L4 REQ ID': '9103',
+                'L4 REQ ID': '',
             }));
         });
         it('should not emit bug rows from ADO-linked bug ids when external bugs source is empty', () => {
@@ -2142,8 +2141,27 @@ describe('ResultDataProvider', () => {
             ]);
             const map = await resultDataProvider.loadExternalL3L4ByBaseKey(validL3L4Source);
             expect(map.get('SR0001')).toEqual([
-                { id: '7002', title: 'L3 Requirement', level: 'L3' },
-                { id: '7001', title: 'L4 From Level3 Column', level: 'L4' },
+                { l3Id: '', l3Title: '', l4Id: '7001', l4Title: 'L4 From Level3 Column' },
+                { l3Id: '7002', l3Title: 'L3 Requirement', l4Id: '', l4Title: '' },
+            ]);
+        });
+        it('should emit paired L3+L4 when AREA 34 is Level 4 and both level columns are present', async () => {
+            const mewpExternalTableUtils = resultDataProvider.mewpExternalTableUtils;
+            jest.spyOn(mewpExternalTableUtils, 'loadExternalTableRows').mockResolvedValueOnce([
+                {
+                    SR: 'SR0001',
+                    'AREA 34': 'Level 4',
+                    'TargetWorkItemId Level 3': '7401',
+                    TargetTitleLevel3: 'L3 In Level4 Row',
+                    'TargetStateLevel 3': 'Active',
+                    'TargetWorkItemIdLevel 4': '8401',
+                    TargetTitleLevel4: 'L4 In Level4 Row',
+                    'TargetStateLevel 4': 'Active',
+                },
+            ]);
+            const map = await resultDataProvider.loadExternalL3L4ByBaseKey(validL3L4Source);
+            expect(map.get('SR0001')).toEqual([
+                { l3Id: '7401', l3Title: 'L3 In Level4 Row', l4Id: '8401', l4Title: 'L4 In Level4 Row' },
             ]);
         });
         it('should exclude external open L3/L4 rows when SAPWBS resolves to ESUK', async () => {
@@ -2172,8 +2190,8 @@ describe('ResultDataProvider', () => {
             ]);
             const map = await resultDataProvider.loadExternalL3L4ByBaseKey(validL3L4Source);
             expect(map.get('SR0001')).toEqual([
-                { id: '7102', title: 'L3 IL', level: 'L3' },
-                { id: '7201', title: 'L4 IL', level: 'L4' },
+                { l3Id: '', l3Title: '', l4Id: '7201', l4Title: 'L4 IL' },
+                { l3Id: '7102', l3Title: 'L3 IL', l4Id: '', l4Title: '' },
             ]);
         });
         it('should fallback L3/L4 SAPWBS exclusion from SR-mapped requirement when row SAPWBS is empty', async () => {
@@ -2201,7 +2219,9 @@ describe('ResultDataProvider', () => {
                 ['SR0002', 'IL'],
             ]));
             expect(map.has('SR0001')).toBe(false);
-            expect(map.get('SR0002')).toEqual([{ id: '7302', title: 'L3 From IL Requirement', level: 'L3' }]);
+            expect(map.get('SR0002')).toEqual([
+                { l3Id: '7302', l3Title: 'L3 From IL Requirement', l4Id: '', l4Title: '' },
+            ]);
         });
         it('should resolve bug responsibility from AreaPath when SAPWBS is empty', () => {
             const fromEsukAreaPath = resultDataProvider.resolveBugResponsibility({