@elisra-devops/docgen-data-provider 1.77.0 → 1.79.0

package/bin/models/mewp-reporting.d.ts

@@ -36,6 +36,7 @@ export interface MewpL2RequirementWorkItem {
     areaPath: string;
 }
 export interface MewpL2RequirementFamily {
+    workItemId?: number;
     requirementId: string;
     baseKey: string;
     title: string;

package/bin/modules/ResultDataProvider.d.ts

@@ -86,7 +86,6 @@ export default class ResultDataProvider {
     private createEmptyMewpCoverageBugCell;
     private createEmptyMewpCoverageL3L4Cell;
     private buildMewpCoverageL3L4Rows;
-    private formatMewpCustomerId;
     private buildMewpCoverageRows;
     private resolveCoverageBugResponsibility;
     private resolveMewpL2RunStatus;

package/bin/modules/ResultDataProvider.js

@@ -381,6 +381,53 @@ class ResultDataProvider {
                 }));
                 this.accumulateRequirementCountsFromActionResults(fallbackActionResults, testCaseId, requirementKeys, requirementIndex, observedTestCaseIdsByRequirement);
             }
+            const requirementBaseKeys = new Set(requirements.map((item) => String((item === null || item === void 0 ? void 0 : item.baseKey) || '').trim()).filter((item) => !!item));
+            const externalL3L4BaseKeys = new Set([...externalL3L4ByBaseKey.keys()]);
+            const externalL3L4OverlapKeys = [...externalL3L4BaseKeys].filter((key) => requirementBaseKeys.has(key));
+            const failedRequirementBaseKeys = new Set();
+            const failedTestCaseIds = new Set();
+            for (const [requirementBaseKey, byTestCase] of requirementIndex.entries()) {
+                for (const [testCaseId, counts] of byTestCase.entries()) {
+                    if (Number((counts === null || counts === void 0 ? void 0 : counts.failed) || 0) > 0) {
+                        failedRequirementBaseKeys.add(requirementBaseKey);
+                        failedTestCaseIds.add(testCaseId);
+                    }
+                }
+            }
+            const externalBugTestCaseIds = new Set([...externalBugsByTestCase.keys()]);
+            const externalBugFailedTestCaseOverlap = [...externalBugTestCaseIds].filter((id) => failedTestCaseIds.has(id));
+            const externalBugBaseKeys = new Set();
+            for (const bugs of externalBugsByTestCase.values()) {
+                for (const bug of bugs || []) {
+                    const key = String((bug === null || bug === void 0 ? void 0 : bug.requirementBaseKey) || '').trim();
+                    if (key)
+                        externalBugBaseKeys.add(key);
+                }
+            }
+            const externalBugRequirementOverlap = [...externalBugBaseKeys].filter((key) => requirementBaseKeys.has(key));
+            const externalBugFailedRequirementOverlap = [...externalBugBaseKeys].filter((key) => failedRequirementBaseKeys.has(key));
+            logger_1.default.info(`MEWP coverage join diagnostics: requirementBaseKeys=${requirementBaseKeys.size} ` +
+                `failedRequirementBaseKeys=${failedRequirementBaseKeys.size} failedTestCases=${failedTestCaseIds.size}; ` +
+                `externalL3L4BaseKeys=${externalL3L4BaseKeys.size} externalL3L4Overlap=${externalL3L4OverlapKeys.length}; ` +
+                `externalBugTestCases=${externalBugTestCaseIds.size} externalBugFailedTestCaseOverlap=${externalBugFailedTestCaseOverlap.length}; ` +
+                `externalBugBaseKeys=${externalBugBaseKeys.size} externalBugRequirementOverlap=${externalBugRequirementOverlap.length} ` +
+                `externalBugFailedRequirementOverlap=${externalBugFailedRequirementOverlap.length}`);
+            if (externalL3L4BaseKeys.size > 0 && externalL3L4OverlapKeys.length === 0) {
+                const sampleReq = [...requirementBaseKeys].slice(0, 5);
+                const sampleExt = [...externalL3L4BaseKeys].slice(0, 5);
+                logger_1.default.warn(`MEWP coverage join diagnostics: no L3/L4 key overlap found. ` +
+                    `sampleRequirementKeys=${sampleReq.join(', ')} sampleExternalL3L4Keys=${sampleExt.join(', ')}`);
+            }
+            if (externalBugBaseKeys.size > 0 && externalBugRequirementOverlap.length === 0) {
+                const sampleReq = [...requirementBaseKeys].slice(0, 5);
+                const sampleExt = [...externalBugBaseKeys].slice(0, 5);
+                logger_1.default.warn(`MEWP coverage join diagnostics: no bug requirement-key overlap found. ` +
+                    `sampleRequirementKeys=${sampleReq.join(', ')} sampleExternalBugKeys=${sampleExt.join(', ')}`);
+            }
+            if (externalBugTestCaseIds.size > 0 && externalBugFailedTestCaseOverlap.length === 0) {
+                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);
             const coverageRowStats = rows.reduce((acc, row) => {
                 const hasBug = Number((row === null || row === void 0 ? void 0 : row['Bug ID']) || 0) > 0;
@@ -647,7 +694,8 @@ class ResultDataProvider {
         return `MEWP Internal Validation - ${suffix}`;
     }
     createMewpCoverageRow(requirement, runStatus, bug, linkedL3L4) {
-        const l2ReqId = this.formatMewpCustomerId(requirement.requirementId);
+        const l2ReqIdNumeric = Number((requirement === null || requirement === void 0 ? void 0 : requirement.workItemId) || 0);
+        const l2ReqId = l2ReqIdNumeric > 0 ? String(l2ReqIdNumeric) : '';
         const l2ReqTitle = this.toMewpComparableText(requirement.title);
         const l2SubSystem = this.toMewpComparableText(requirement.subSystem);
         return {
@@ -688,15 +736,6 @@ class ResultDataProvider {
         }
         return rows;
     }
-    formatMewpCustomerId(rawValue) {
-        const normalized = this.normalizeMewpRequirementCode(this.toMewpComparableText(rawValue));
-        if (normalized)
-            return normalized;
-        const onlyDigits = String(rawValue || '').replace(/\D/g, '');
-        if (onlyDigits)
-            return `SR${onlyDigits}`;
-        return '';
-    }
     buildMewpCoverageRows(requirements, requirementIndex, observedTestCaseIdsByRequirement, linkedRequirementsByTestCase, l3l4ByBaseKey, externalBugsByTestCase) {
         var _a;
         const rows = [];
@@ -1322,7 +1361,7 @@ class ResultDataProvider {
         const workItems = await this.fetchWorkItemsByIds(projectName, requirementIds, true);
         const requirements = workItems.map((wi) => {
             const fields = (wi === null || wi === void 0 ? void 0 : wi.fields) || {};
-            const requirementId = this.extractMewpRequirementIdentifier(fields, Number((wi === null || wi === void 0 ? void 0 : wi.id) || 0));
+            const requirementId = this.extractMewpRequirementIdentifier(fields);
             const areaPath = this.toMewpComparableText(fields === null || fields === void 0 ? void 0 : fields['System.AreaPath']);
             return {
                 workItemId: Number((wi === null || wi === void 0 ? void 0 : wi.id) || 0),
@@ -1404,13 +1443,14 @@ class ResultDataProvider {
         }
         return [...families.entries()]
             .map(([baseKey, family]) => {
-            var _a, _b, _c, _d;
+            var _a, _b, _c, _d, _e;
             return ({
-                requirementId: String(((_a = family === null || family === void 0 ? void 0 : family.representative) === null || _a === void 0 ? void 0 : _a.requirementId) || baseKey),
+                workItemId: Number(((_a = family === null || family === void 0 ? void 0 : family.representative) === null || _a === void 0 ? void 0 : _a.workItemId) || 0),
+                requirementId: String(((_b = family === null || family === void 0 ? void 0 : family.representative) === null || _b === void 0 ? void 0 : _b.requirementId) || baseKey),
                 baseKey,
-                title: String(((_b = family === null || family === void 0 ? void 0 : family.representative) === null || _b === void 0 ? void 0 : _b.title) || ''),
-                subSystem: String(((_c = family === null || family === void 0 ? void 0 : family.representative) === null || _c === void 0 ? void 0 : _c.subSystem) || ''),
-                responsibility: String(((_d = family === null || family === void 0 ? void 0 : family.representative) === null || _d === void 0 ? void 0 : _d.responsibility) || ''),
+                title: String(((_c = family === null || family === void 0 ? void 0 : family.representative) === null || _c === void 0 ? void 0 : _c.title) || ''),
+                subSystem: String(((_d = family === null || family === void 0 ? void 0 : family.representative) === null || _d === void 0 ? void 0 : _d.subSystem) || ''),
+                responsibility: String(((_e = family === null || family === void 0 ? void 0 : family.representative) === null || _e === void 0 ? void 0 : _e.responsibility) || ''),
                 linkedTestCaseIds: [...family.linkedTestCaseIds].sort((a, b) => a - b),
             });
         })
@@ -1666,48 +1706,30 @@ class ResultDataProvider {
         }
         return [...out].sort((a, b) => a - b);
     }
-    extractMewpRequirementIdentifier(fields, fallbackWorkItemId) {
+    extractMewpRequirementIdentifier(fields) {
         const entries = Object.entries(fields || {});
-        // First pass: only trusted identifier-like fields.
-        const strictHints = [
+        const normalizeFieldKey = (value) => String(value || '')
+            .toLowerCase()
+            .replace(/[^a-z0-9]/g, '');
+        // Strict MEWP mode: only explicit MEWP customer-id fields are accepted.
+        // API display name: "Customer ID"
+        // API reference name: "Custom.CustomerID"
+        const customerIdFieldKeys = new Set([
             'customerid',
-            'customer id',
-            'customerrequirementid',
-            'requirementid',
-            'externalid',
-            'srid',
-            'sapwbsid',
-        ];
+            'customcustomerid',
+        ]);
         for (const [key, value] of entries) {
-            const normalizedKey = String(key || '').toLowerCase();
-            if (!strictHints.some((hint) => normalizedKey.includes(hint)))
+            const normalizedKey = normalizeFieldKey(key);
+            if (!customerIdFieldKeys.has(normalizedKey))
                 continue;
             const valueAsString = this.toMewpComparableText(value);
             if (!valueAsString)
                 continue;
-            const normalized = this.normalizeMewpRequirementCodeWithSuffix(valueAsString);
-            if (normalized)
-                return normalized;
+            const normalizedRequirementId = this.normalizeMewpRequirementCodeWithSuffix(valueAsString);
+            if (normalizedRequirementId)
+                return normalizedRequirementId;
         }
-        // Second pass: weaker hints, but still key-based only.
-        const looseHints = ['customer', 'requirement', 'external', 'sapwbs', 'sr'];
-        for (const [key, value] of entries) {
-            const normalizedKey = String(key || '').toLowerCase();
-            if (!looseHints.some((hint) => normalizedKey.includes(hint)))
-                continue;
-            const valueAsString = this.toMewpComparableText(value);
-            if (!valueAsString)
-                continue;
-            const normalized = this.normalizeMewpRequirementCodeWithSuffix(valueAsString);
-            if (normalized)
-                return normalized;
-        }
-        // Optional fallback from title only (avoid scanning all fields and accidental SR matches).
-        const title = this.toMewpComparableText(fields === null || fields === void 0 ? void 0 : fields['System.Title']);
-        const titleCode = this.normalizeMewpRequirementCodeWithSuffix(title);
-        if (titleCode)
-            return titleCode;
-        return fallbackWorkItemId ? `SR${fallbackWorkItemId}` : '';
+        return '';
     }
     deriveMewpResponsibility(fields) {
         const explicitSapWbs = this.toMewpComparableText(fields === null || fields === void 0 ? void 0 : fields['Custom.SAPWBS']);