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

package/bin/modules/ResultDataProvider.d.ts

@@ -88,6 +88,7 @@ export default class ResultDataProvider {
     private buildMewpCoverageL3L4Rows;
     private buildMewpCoverageRows;
     private resolveCoverageBugResponsibility;
+    private buildMewpTestCaseResponsibilityMap;
     private resolveMewpL2RunStatus;
     private fetchMewpScopedTestData;
     private extractRelNumberFromSuite;
@@ -134,6 +135,7 @@ export default class ResultDataProvider {
     private extractLinkedWorkItemIdsFromRelations;
     private extractMewpRequirementIdentifier;
     private deriveMewpResponsibility;
+    private deriveMewpTestCaseResponsibility;
     private deriveMewpSubSystem;
     private resolveBugResponsibility;
     private resolveMewpResponsibility;

package/bin/modules/ResultDataProvider.js

@@ -306,6 +306,7 @@ class ResultDataProvider {
             const requirements = this.collapseMewpRequirementFamilies(allRequirements, (scopedRequirementKeys === null || scopedRequirementKeys === void 0 ? void 0 : scopedRequirementKeys.size) ? scopedRequirementKeys : undefined);
             const l2ToLinkedL1BaseKeys = await this.buildMewpL2ToLinkedL1BaseKeys(allRequirements, projectName, testData);
             const requirementSapWbsByBaseKey = this.buildRequirementSapWbsByBaseKey(allRequirements);
+            const testCaseResponsibilityById = await this.buildMewpTestCaseResponsibilityMap(testData, projectName);
             const externalBugsByTestCase = await this.loadExternalBugsByTestCase(options === null || options === void 0 ? void 0 : options.externalBugsFile);
             const externalL3L4ByBaseKey = await this.loadExternalL3L4ByBaseKey(options === null || options === void 0 ? void 0 : options.externalL3L4File, requirementSapWbsByBaseKey);
             const hasExternalBugsFile = !!String(((_a = options === null || options === void 0 ? void 0 : options.externalBugsFile) === null || _a === void 0 ? void 0 : _a.name) ||
@@ -442,7 +443,7 @@ class ResultDataProvider {
                 logger_1.default.warn(`MEWP coverage join diagnostics: no overlap between external bug test cases and failed test cases. ` +
                     `Bug rows remain empty because bugs are shown only for failed L2s.`);
             }
-            const rows = this.buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, externalL3L4ByBaseKey, externalBugsByTestCase, externalJoinKeysByL2);
+            const rows = this.buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, externalL3L4ByBaseKey, externalBugsByTestCase, externalJoinKeysByL2, testCaseResponsibilityById);
             const coverageRowStats = rows.reduce((acc, row) => {
                 const hasBug = Number((row === null || row === void 0 ? void 0 : row['Bug ID']) || 0) > 0;
                 const hasL3 = String((row === null || row === void 0 ? void 0 : row['L3 REQ ID']) || '').trim() !== '';
@@ -557,34 +558,56 @@ class ResultDataProvider {
                     diagnostics.testCasesWithoutMentionedCustomerIds += 1;
                 }
                 const mentionedBaseKeys = new Set([...mentionedL2Only].map((code) => this.toRequirementKey(code)).filter((code) => !!code));
-                const expectedFamilyCodes = new Set();
-                for (const baseKey of mentionedBaseKeys) {
-                    const familyCodes = requirementFamilies.get(baseKey);
-                    if (familyCodes === null || familyCodes === void 0 ? void 0 : familyCodes.size) {
-                        familyCodes.forEach((code) => expectedFamilyCodes.add(code));
-                    }
-                    else {
-                        for (const code of mentionedL2Only) {
-                            if (this.toRequirementKey(code) === baseKey)
-                                expectedFamilyCodes.add(code);
-                        }
-                    }
-                }
                 const linkedFullCodesRaw = ((_c = linkedRequirementsByTestCase.get(testCaseId)) === null || _c === void 0 ? void 0 : _c.fullCodes) || new Set();
                 const linkedFullCodes = (scopedRequirementKeys === null || scopedRequirementKeys === void 0 ? void 0 : scopedRequirementKeys.size) && linkedFullCodesRaw.size > 0
                     ? new Set([...linkedFullCodesRaw].filter((code) => scopedRequirementKeys.has(this.toRequirementKey(code))))
                     : linkedFullCodesRaw;
                 const linkedBaseKeys = new Set([...linkedFullCodes].map((code) => this.toRequirementKey(code)).filter((code) => !!code));
-                const missingMentioned = [...mentionedL2Only].filter((code) => {
+                const mentionedCodesByBase = new Map();
+                for (const code of mentionedL2Only) {
                     const baseKey = this.toRequirementKey(code);
                     if (!baseKey)
-                        return false;
-                    const hasSpecificSuffix = /-\d+$/.test(code);
-                    if (hasSpecificSuffix)
-                        return !linkedFullCodes.has(code);
-                    return !linkedBaseKeys.has(baseKey);
-                });
-                const missingFamily = [...expectedFamilyCodes].filter((code) => !linkedFullCodes.has(code));
+                        continue;
+                    if (!mentionedCodesByBase.has(baseKey))
+                        mentionedCodesByBase.set(baseKey, new Set());
+                    mentionedCodesByBase.get(baseKey).add(code);
+                }
+                // Context-based direction A logic:
+                // 1) If no member of family is linked -> report only base SR (Step X: SR0054).
+                // 2) If family is partially linked -> report only specific missing members.
+                // 3) If family fully linked -> report nothing for that family.
+                const missingBaseWhenFamilyUncovered = new Set();
+                const missingSpecificMentionedNoFamily = new Set();
+                const missingFamilyMembers = new Set();
+                for (const [baseKey, mentionedCodes] of mentionedCodesByBase.entries()) {
+                    const familyCodes = requirementFamilies.get(baseKey);
+                    if (familyCodes === null || familyCodes === void 0 ? void 0 : familyCodes.size) {
+                        const missingInFamily = [...familyCodes].filter((code) => !linkedFullCodes.has(code));
+                        if (missingInFamily.length === 0)
+                            continue;
+                        const linkedInFamilyCount = familyCodes.size - missingInFamily.length;
+                        if (linkedInFamilyCount === 0) {
+                            missingBaseWhenFamilyUncovered.add(baseKey);
+                        }
+                        else {
+                            for (const code of missingInFamily) {
+                                missingFamilyMembers.add(code);
+                            }
+                        }
+                        continue;
+                    }
+                    // Fallback path when family data is unavailable for this base key.
+                    for (const code of mentionedCodes) {
+                        const hasSpecificSuffix = /-\d+$/.test(code);
+                        if (hasSpecificSuffix) {
+                            if (!linkedFullCodes.has(code))
+                                missingSpecificMentionedNoFamily.add(code);
+                        }
+                        else if (!linkedBaseKeys.has(baseKey)) {
+                            missingBaseWhenFamilyUncovered.add(baseKey);
+                        }
+                    }
+                }
                 // 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) => {
@@ -604,13 +627,18 @@ class ResultDataProvider {
                     }
                     mentionedButNotLinkedByStep.get(normalizedStepRef).add(normalizedRequirementId);
                 };
-                const sortedMissingMentioned = [...new Set(missingMentioned)].sort((a, b) => a.localeCompare(b));
-                const sortedMissingFamily = [...new Set(missingFamily)].sort((a, b) => a.localeCompare(b));
-                for (const code of sortedMissingMentioned) {
+                const sortedMissingSpecificMentionedNoFamily = [...missingSpecificMentionedNoFamily].sort((a, b) => a.localeCompare(b));
+                const sortedMissingBaseWhenFamilyUncovered = [...missingBaseWhenFamilyUncovered].sort((a, b) => a.localeCompare(b));
+                const sortedMissingFamilyMembers = [...missingFamilyMembers].sort((a, b) => a.localeCompare(b));
+                for (const code of sortedMissingSpecificMentionedNoFamily) {
                     const stepRef = mentionedCodeFirstStep.get(code) || 'Step ?';
                     appendMentionedButNotLinked(code, stepRef);
                 }
-                for (const code of sortedMissingFamily) {
+                for (const baseKey of sortedMissingBaseWhenFamilyUncovered) {
+                    const stepRef = mentionedBaseFirstStep.get(baseKey) || 'Step ?';
+                    appendMentionedButNotLinked(baseKey, stepRef);
+                }
+                for (const code of sortedMissingFamilyMembers) {
                     const baseKey = this.toRequirementKey(code);
                     const stepRef = mentionedBaseFirstStep.get(baseKey) || 'Step ?';
                     appendMentionedButNotLinked(code, stepRef);
@@ -644,7 +672,9 @@ class ResultDataProvider {
                 logger_1.default.debug(`MEWP internal validation parse diagnostics: ` +
                     `testCaseId=${testCaseId} parsedSteps=${executableSteps.length} ` +
                     `stepsWithMentions=${mentionEntries.length} customerIdsFound=${mentionedL2Only.size} ` +
-                    `linkedRequirements=${linkedFullCodes.size} mentionedButNotLinked=${sortedMissingMentioned.length + sortedMissingFamily.length} ` +
+                    `linkedRequirements=${linkedFullCodes.size} mentionedButNotLinked=${sortedMissingSpecificMentionedNoFamily.length +
+                        sortedMissingBaseWhenFamilyUncovered.length +
+                        sortedMissingFamilyMembers.length} ` +
                     `linkedButNotMentioned=${sortedExtraLinked.length} status=${validationStatus} ` +
                     `customerIdSample='${[...mentionedL2Only].slice(0, 5).join(', ')}'`);
                 rows.push({
@@ -813,7 +843,7 @@ class ResultDataProvider {
             return String((a === null || a === void 0 ? void 0 : a.l4Id) || '').localeCompare(String((b === null || b === void 0 ? void 0 : b.l4Id) || ''));
         });
     }
-    buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, l3l4ByBaseKey, externalBugsByTestCase, externalJoinKeysByL2) {
+    buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, l3l4ByBaseKey, externalBugsByTestCase, externalJoinKeysByL2, testCaseResponsibilityById) {
         var _a;
         const rows = [];
         const linkedByRequirement = this.invertBaseRequirementLinks(linkedRequirementsByTestCase);
@@ -847,7 +877,7 @@ class ResultDataProvider {
                         const bugId = Number((bug === null || bug === void 0 ? void 0 : bug.id) || 0);
                         if (!Number.isFinite(bugId) || bugId <= 0)
                             continue;
-                        aggregatedBugs.set(bugId, Object.assign(Object.assign({}, bug), { responsibility: this.resolveCoverageBugResponsibility(String((bug === null || bug === void 0 ? void 0 : bug.responsibility) || ''), requirement) }));
+                        aggregatedBugs.set(bugId, Object.assign(Object.assign({}, bug), { responsibility: this.resolveCoverageBugResponsibility(String((bug === null || bug === void 0 ? void 0 : bug.responsibility) || ''), requirement, String((testCaseResponsibilityById === null || testCaseResponsibilityById === void 0 ? void 0 : testCaseResponsibilityById.get(testCaseId)) || '')) }));
                     }
                 }
             }
@@ -874,18 +904,114 @@ class ResultDataProvider {
         }
         return rows;
     }
-    resolveCoverageBugResponsibility(rawResponsibility, requirement) {
-        const direct = String(rawResponsibility || '').trim();
+    resolveCoverageBugResponsibility(rawResponsibility, requirement, testCaseResponsibility = '') {
+        const normalizeDisplay = (value) => {
+            const direct = String(value || '').trim();
+            if (!direct)
+                return '';
+            const resolved = this.resolveMewpResponsibility(this.toMewpComparableText(direct));
+            if (resolved === 'ESUK')
+                return 'ESUK';
+            if (resolved === 'IL')
+                return 'Elisra';
+            return direct;
+        };
+        const direct = normalizeDisplay(rawResponsibility);
         if (direct && direct.toLowerCase() !== 'unknown')
             return direct;
-        const requirementResponsibility = String((requirement === null || requirement === void 0 ? void 0 : requirement.responsibility) || '')
-            .trim()
-            .toUpperCase();
-        if (requirementResponsibility === 'ESUK')
-            return 'ESUK';
-        if (requirementResponsibility === 'IL' || requirementResponsibility === 'ELISRA')
-            return 'Elisra';
-        return direct || 'Unknown';
+        const fromTestCase = normalizeDisplay(testCaseResponsibility);
+        if (fromTestCase && fromTestCase.toLowerCase() !== 'unknown')
+            return fromTestCase;
+        const fromRequirement = normalizeDisplay(String((requirement === null || requirement === void 0 ? void 0 : requirement.responsibility) || ''));
+        if (fromRequirement && fromRequirement.toLowerCase() !== 'unknown')
+            return fromRequirement;
+        return direct || fromTestCase || fromRequirement || 'Unknown';
+    }
+    async buildMewpTestCaseResponsibilityMap(testData, projectName) {
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m;
+        const out = new Map();
+        const unresolved = new Set();
+        const extractFromWorkItemFields = (workItemFields) => {
+            const fields = {};
+            if (!Array.isArray(workItemFields))
+                return fields;
+            for (const field of workItemFields) {
+                const keyCandidates = [field === null || field === void 0 ? void 0 : field.key, field === null || field === void 0 ? void 0 : field.name, field === null || field === void 0 ? void 0 : field.referenceName]
+                    .map((item) => String(item || '').trim())
+                    .filter((item) => !!item);
+                for (const key of keyCandidates) {
+                    fields[key] = field === null || field === void 0 ? void 0 : field.value;
+                }
+            }
+            return fields;
+        };
+        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 testCaseId = 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(testCaseId) || testCaseId <= 0 || out.has(testCaseId))
+                    continue;
+                const workItemFieldsMap = ((_b = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _b === void 0 ? void 0 : _b.fields) || {};
+                const workItemFieldsList = extractFromWorkItemFields((_c = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _c === void 0 ? void 0 : _c.workItemFields);
+                const directFieldAliases = Object.fromEntries(Object.entries({
+                    'System.AreaPath': ((_d = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _d === void 0 ? void 0 : _d.areaPath) ||
+                        ((_e = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _e === void 0 ? void 0 : _e.AreaPath) ||
+                        (testCase === null || testCase === void 0 ? void 0 : testCase.areaPath) ||
+                        (testCase === null || testCase === void 0 ? void 0 : testCase.AreaPath) ||
+                        ((_f = testCase === null || testCase === void 0 ? void 0 : testCase.testCase) === null || _f === void 0 ? void 0 : _f.areaPath) ||
+                        ((_g = testCase === null || testCase === void 0 ? void 0 : testCase.testCase) === null || _g === void 0 ? void 0 : _g.AreaPath),
+                    'Area Path': ((_h = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _h === void 0 ? void 0 : _h['Area Path']) ||
+                        (testCase === null || testCase === void 0 ? void 0 : testCase['Area Path']) ||
+                        ((_j = testCase === null || testCase === void 0 ? void 0 : testCase.testCase) === null || _j === void 0 ? void 0 : _j['Area Path']),
+                    AreaPath: ((_k = testCase === null || testCase === void 0 ? void 0 : testCase.workItem) === null || _k === void 0 ? void 0 : _k.AreaPath) ||
+                        (testCase === null || testCase === void 0 ? void 0 : testCase.AreaPath) ||
+                        ((_l = testCase === null || testCase === void 0 ? void 0 : testCase.testCase) === null || _l === void 0 ? void 0 : _l.AreaPath),
+                }).filter(([, value]) => value !== undefined && value !== null && String(value).trim() !== ''));
+                const fromWorkItem = this.deriveMewpTestCaseResponsibility(Object.assign(Object.assign(Object.assign({}, directFieldAliases), workItemFieldsList), workItemFieldsMap));
+                if (fromWorkItem) {
+                    out.set(testCaseId, fromWorkItem);
+                }
+                else {
+                    unresolved.add(testCaseId);
+                }
+            }
+            const testPointsItems = Array.isArray(suite === null || suite === void 0 ? void 0 : suite.testPointsItems) ? suite.testPointsItems : [];
+            for (const testPoint of testPointsItems) {
+                const testCaseId = Number((testPoint === null || testPoint === void 0 ? void 0 : testPoint.testCaseId) || ((_m = testPoint === null || testPoint === void 0 ? void 0 : testPoint.testCase) === null || _m === void 0 ? void 0 : _m.id) || 0);
+                if (!Number.isFinite(testCaseId) || testCaseId <= 0 || out.has(testCaseId))
+                    continue;
+                const fromPoint = this.deriveMewpTestCaseResponsibility({
+                    'System.AreaPath': (testPoint === null || testPoint === void 0 ? void 0 : testPoint.areaPath) || (testPoint === null || testPoint === void 0 ? void 0 : testPoint.AreaPath),
+                    'Area Path': testPoint === null || testPoint === void 0 ? void 0 : testPoint['Area Path'],
+                    AreaPath: testPoint === null || testPoint === void 0 ? void 0 : testPoint.AreaPath,
+                });
+                if (fromPoint) {
+                    out.set(testCaseId, fromPoint);
+                    unresolved.delete(testCaseId);
+                }
+                else {
+                    unresolved.add(testCaseId);
+                }
+            }
+        }
+        if (unresolved.size > 0) {
+            try {
+                const workItems = await this.fetchWorkItemsByIds(projectName, [...unresolved], false);
+                for (const workItem of workItems || []) {
+                    const testCaseId = Number((workItem === null || workItem === void 0 ? void 0 : workItem.id) || 0);
+                    if (!Number.isFinite(testCaseId) || testCaseId <= 0 || out.has(testCaseId))
+                        continue;
+                    const resolved = this.deriveMewpTestCaseResponsibility((workItem === null || workItem === void 0 ? void 0 : workItem.fields) || {});
+                    if (!resolved)
+                        continue;
+                    out.set(testCaseId, resolved);
+                }
+            }
+            catch (error) {
+                logger_1.default.warn(`MEWP coverage: failed to enrich test-case responsibility fallback: ${(error === null || error === void 0 ? void 0 : error.message) || error}`);
+            }
+        }
+        return out;
     }
     resolveMewpL2RunStatus(input) {
         if (((input === null || input === void 0 ? void 0 : input.failed) || 0) > 0)
@@ -1998,11 +2124,42 @@ class ResultDataProvider {
             return fromExplicitLabel;
         if (explicitSapWbsByLabel)
             return explicitSapWbsByLabel;
-        const areaPath = this.toMewpComparableText(fields === null || fields === void 0 ? void 0 : fields['System.AreaPath']);
-        const fromAreaPath = this.resolveMewpResponsibility(areaPath);
-        if (fromAreaPath)
-            return fromAreaPath;
-        const keyHints = ['sapwbs', 'responsibility', 'owner'];
+        const areaPathCandidates = [
+            fields === null || fields === void 0 ? void 0 : fields['System.AreaPath'],
+            fields === null || fields === void 0 ? void 0 : fields['Area Path'],
+            fields === null || fields === void 0 ? void 0 : fields['AreaPath'],
+        ];
+        for (const candidate of areaPathCandidates) {
+            const normalized = this.toMewpComparableText(candidate);
+            const resolved = this.resolveMewpResponsibility(normalized);
+            if (resolved)
+                return resolved;
+        }
+        const keyHints = ['sapwbs', 'responsibility', 'owner', 'areapath', 'area path'];
+        for (const [key, value] of Object.entries(fields || {})) {
+            const normalizedKey = String(key || '').toLowerCase();
+            if (!keyHints.some((hint) => normalizedKey.includes(hint)))
+                continue;
+            const resolved = this.resolveMewpResponsibility(this.toMewpComparableText(value));
+            if (resolved)
+                return resolved;
+        }
+        return '';
+    }
+    // Test-case responsibility must come from test-case path context (not SAPWBS).
+    deriveMewpTestCaseResponsibility(fields) {
+        const areaPathCandidates = [
+            fields === null || fields === void 0 ? void 0 : fields['System.AreaPath'],
+            fields === null || fields === void 0 ? void 0 : fields['Area Path'],
+            fields === null || fields === void 0 ? void 0 : fields['AreaPath'],
+        ];
+        for (const candidate of areaPathCandidates) {
+            const normalized = this.toMewpComparableText(candidate);
+            const resolved = this.resolveMewpResponsibility(normalized);
+            if (resolved)
+                return resolved;
+        }
+        const keyHints = ['areapath', 'area path', 'responsibility', 'owner'];
         for (const [key, value] of Object.entries(fields || {})) {
             const normalizedKey = String(key || '').toLowerCase();
             if (!keyHints.some((hint) => normalizedKey.includes(hint)))
@@ -2404,7 +2561,6 @@ class ResultDataProvider {
                     logger_1.default.warn(`Invalid run result ${runId} or result ${resultId}`);
                     return null;
                 }
-                logger_1.default.warn(`Current Test point for Test case ${point.testCaseId} is in Active state`);
                 const url = `${this.orgUrl}${projectName}/_apis/wit/workItems/${point.testCaseId}?$expand=all`;
                 const testCaseData = await tfs_1.TFSServices.getItemContent(url, this.token);
                 const newResultData = {