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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elisra-devops/docgen-data-provider",
-  "version": "1.82.0",
+  "version": "1.83.0",
   "description": "A document generator data provider, aimed to retrive data from azure devops",
   "repository": {
     "type": "git",

package/src/modules/ResultDataProvider.ts

@@ -864,7 +864,13 @@ export default class ResultDataProvider {
           return !linkedBaseKeys.has(baseKey);
         });
         const missingFamily = [...expectedFamilyCodes].filter((code) => !linkedFullCodes.has(code));
-        const extraLinked = [...linkedFullCodes].filter((code) => !expectedFamilyCodes.has(code));
+        // Direction B is family-based: if any member of a family is mentioned in Expected Result,
+        // linked members of that same family are not considered "linked but not mentioned".
+        const extraLinked = [...linkedFullCodes].filter((code) => {
+          const baseKey = this.toRequirementKey(code);
+          if (!baseKey) return false;
+          return !mentionedBaseKeys.has(baseKey);
+        });
         const mentionedButNotLinkedByStep = new Map<string, Set<string>>();
         const appendMentionedButNotLinked = (requirementId: string, stepRef: string) => {
           const normalizedRequirementId = this.normalizeMewpRequirementCodeWithSuffix(requirementId);

package/src/tests/modules/ResultDataProvider.test.ts

@@ -1903,6 +1903,69 @@ describe('ResultDataProvider', () => {
       );
     });
 
+    it('should not flag linked child as Direction B when parent family is mentioned in Expected Result', async () => {
+      jest.spyOn(resultDataProvider as any, 'fetchTestPlanName').mockResolvedValueOnce('Plan A');
+      jest.spyOn(resultDataProvider as any, 'fetchTestSuites').mockResolvedValueOnce([{ testSuiteId: 1 }]);
+      jest.spyOn(resultDataProvider as any, 'fetchTestData').mockResolvedValueOnce([
+        {
+          testPointsItems: [{ testCaseId: 301, testCaseName: 'TC 301' }],
+          testCasesItems: [
+            {
+              workItem: {
+                id: 301,
+                workItemFields: [{ key: 'Steps', value: '<steps></steps>' }],
+              },
+            },
+          ],
+        },
+      ]);
+      jest.spyOn(resultDataProvider as any, 'fetchMewpL2Requirements').mockResolvedValueOnce([
+        {
+          workItemId: 7001,
+          requirementId: 'SR0054',
+          baseKey: 'SR0054',
+          title: 'Parent 0054',
+          responsibility: 'ESUK',
+          linkedTestCaseIds: [],
+          areaPath: 'MEWP\\Customer Requirements\\Level 2',
+        },
+      ]);
+      jest.spyOn(resultDataProvider as any, 'buildLinkedRequirementsByTestCase').mockResolvedValueOnce(
+        new Map([
+          [
+            301,
+            {
+              baseKeys: new Set(['SR0054']),
+              fullCodes: new Set(['SR0054-1']),
+            },
+          ],
+        ])
+      );
+      jest.spyOn((resultDataProvider as any).testStepParserHelper, 'parseTestSteps').mockResolvedValueOnce([
+        {
+          stepId: '1',
+          stepPosition: '1',
+          action: '',
+          expected: 'SR0054',
+          isSharedStepTitle: false,
+        },
+      ]);
+
+      const result = await (resultDataProvider as any).getMewpInternalValidationFlatResults(
+        '123',
+        mockProjectName,
+        [1]
+      );
+
+      expect(result.rows).toHaveLength(1);
+      expect(result.rows[0]).toEqual(
+        expect.objectContaining({
+          'Test Case ID': 301,
+          'Linked but Not Mentioned': '',
+        })
+      );
+    });
+
     it('should produce one detailed row per test case with correct bidirectional discrepancies', async () => {
       const mockDetailedStepsByTestCase = new Map<number, any[]>([
         [