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

package/bin/modules/ResultDataProvider.d.ts

@@ -101,7 +101,10 @@ export default class ResultDataProvider {
     private buildMewpTestCaseTitleMap;
     private extractMewpTestCaseId;
     private buildTestCaseStepsXmlMap;
+    private extractStepsXmlFromTestCaseItem;
+    private extractStepsXmlFromFieldsMap;
     private extractStepsXmlFromWorkItemFields;
+    private enrichMewpStepsXmlMapFromWorkItems;
     private classifyRequirementStepOutcome;
     private accumulateRequirementCountsFromActionResults;
     private resolveRequirementStatusForWindow;

package/bin/modules/ResultDataProvider.js

@@ -506,6 +506,11 @@ class ResultDataProvider {
                         allTestCaseIds.add(id);
                 }
             }
+            const preloadedStepXmlCount = stepsXmlByTestCase.size;
+            const fallbackStepLoadStats = await this.enrichMewpStepsXmlMapFromWorkItems(projectName, [...allTestCaseIds], stepsXmlByTestCase);
+            logger_1.default.info(`MEWP internal validation steps source summary: testCases=${allTestCaseIds.size} ` +
+                `fromSuitePayload=${preloadedStepXmlCount} fromWorkItemFallback=${fallbackStepLoadStats.loadedFromFallback} ` +
+                `stepsXmlAvailable=${stepsXmlByTestCase.size} unresolved=${fallbackStepLoadStats.unresolvedCount}`);
             const validL2BaseKeys = new Set([...requirementFamilies.keys()]);
             const diagnostics = {
                 totalTestCases: 0,
@@ -580,7 +585,14 @@ 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();
                 const appendMentionedButNotLinked = (requirementId, stepRef) => {
                     const normalizedRequirementId = this.normalizeMewpRequirementCodeWithSuffix(requirementId);
@@ -1134,18 +1146,17 @@ class ResultDataProvider {
         return Number.isFinite(testCaseId) ? testCaseId : 0;
     }
     buildTestCaseStepsXmlMap(testData) {
-        var _a, _b;
+        var _a;
         const map = new Map();
         for (const suite of testData || []) {
             const testCasesItems = Array.isArray(suite === null || suite === void 0 ? void 0 : suite.testCasesItems) ? suite.testCasesItems : [];
             for (const testCase of testCasesItems) {
-                const id = Number((_a = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _a === void 0 ? void 0 : _a.id);
+                const id = Number(((_a = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _a === void 0 ? void 0 : _a.id) || (testCase === null || testCase === void 0 ? void 0 : testCase.testCaseId) || (testCase === null || testCase === void 0 ? void 0 : testCase.id) || 0);
                 if (!Number.isFinite(id))
                     continue;
                 if (map.has(id))
                     continue;
-                const fields = (_b = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _b === void 0 ? void 0 : _b.workItemFields;
-                const stepsXml = this.extractStepsXmlFromWorkItemFields(fields);
+                const stepsXml = this.extractStepsXmlFromTestCaseItem(testCase);
                 if (stepsXml) {
                     map.set(id, stepsXml);
                 }
@@ -1153,6 +1164,43 @@ class ResultDataProvider {
         }
         return map;
     }
+    extractStepsXmlFromTestCaseItem(testCase) {
+        var _a, _b, _c, _d, _e;
+        const fieldsList = (_a = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _a === void 0 ? void 0 : _a.workItemFields;
+        const fromFieldList = this.extractStepsXmlFromWorkItemFields(fieldsList);
+        if (fromFieldList)
+            return fromFieldList;
+        const workItemFields = ((_b = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _b === void 0 ? void 0 : _b.fields) || {};
+        const fromFieldsMap = this.extractStepsXmlFromFieldsMap(workItemFields);
+        if (fromFieldsMap)
+            return fromFieldsMap;
+        const directCandidates = [
+            testCase === null || testCase === void 0 ? void 0 : testCase.stepsResultXml,
+            (_c = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _c === void 0 ? void 0 : _c.stepsResultXml,
+            (_d = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _d === void 0 ? void 0 : _d['Microsoft.VSTS.TCM.Steps'],
+            (_e = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _e === void 0 ? void 0 : _e.steps,
+        ];
+        for (const candidate of directCandidates) {
+            const value = String(candidate || '').trim();
+            if (value)
+                return value;
+        }
+        return '';
+    }
+    extractStepsXmlFromFieldsMap(fields) {
+        const directCandidates = [
+            fields === null || fields === void 0 ? void 0 : fields['Microsoft.VSTS.TCM.Steps'],
+            fields === null || fields === void 0 ? void 0 : fields['microsoft.vsts.tcm.steps'],
+            fields === null || fields === void 0 ? void 0 : fields['Steps'],
+            fields === null || fields === void 0 ? void 0 : fields['steps'],
+        ];
+        for (const candidate of directCandidates) {
+            const value = String(candidate || '').trim();
+            if (value)
+                return value;
+        }
+        return '';
+    }
     extractStepsXmlFromWorkItemFields(workItemFields) {
         if (!Array.isArray(workItemFields))
             return '';
@@ -1171,6 +1219,36 @@ class ResultDataProvider {
         }
         return '';
     }
+    async enrichMewpStepsXmlMapFromWorkItems(projectName, testCaseIds, stepsXmlByTestCase) {
+        const missingIds = [...new Set(testCaseIds)]
+            .map((id) => Number(id))
+            .filter((id) => Number.isFinite(id) && id > 0)
+            .filter((id) => !stepsXmlByTestCase.has(id));
+        if (missingIds.length === 0) {
+            return { loadedFromFallback: 0, unresolvedCount: 0 };
+        }
+        let loadedFromFallback = 0;
+        try {
+            const workItems = await this.fetchWorkItemsByIds(projectName, missingIds, false);
+            for (const workItem of workItems || []) {
+                const id = Number((workItem === null || workItem === void 0 ? void 0 : workItem.id) || 0);
+                if (!Number.isFinite(id) || id <= 0 || stepsXmlByTestCase.has(id))
+                    continue;
+                const stepsXml = this.extractStepsXmlFromFieldsMap((workItem === null || workItem === void 0 ? void 0 : workItem.fields) || {});
+                if (!stepsXml)
+                    continue;
+                stepsXmlByTestCase.set(id, stepsXml);
+                loadedFromFallback += 1;
+            }
+        }
+        catch (error) {
+            logger_1.default.warn(`MEWP internal validation: failed loading missing steps via work-item fallback: ${(error === null || error === void 0 ? void 0 : error.message) || error}`);
+        }
+        return {
+            loadedFromFallback,
+            unresolvedCount: Math.max(0, missingIds.length - loadedFromFallback),
+        };
+    }
     classifyRequirementStepOutcome(outcome) {
         const normalized = String(outcome || '')
             .trim()