@elisra-devops/docgen-data-provider 1.95.0 → 1.97.0

package/bin/modules/ResultDataProvider.d.ts

@@ -223,6 +223,11 @@ export default class ResultDataProvider {
      * Executes a work-item GET by URL and applies consistent warning/error handling.
      */
     private fetchWorkItemByUrl;
+    /**
+     * Returns true when the snapshot includes a non-empty test steps XML payload.
+     */
+    private hasStepsInWorkItemSnapshot;
+    private logRunlessSnapshotDecision;
     /**
      * Resolves runless test case data using ordered fallbacks:
      * 1) point-based `asOf` snapshot, 2) explicit revision, 3) suite payload snapshot, 4) latest WI.

package/bin/modules/ResultDataProvider.js

@@ -2764,6 +2764,7 @@ class ResultDataProvider {
         const parsedAsOf = new Date(String(asOfTimestamp || '').trim());
         if (!Number.isFinite(id) || id <= 0 || Number.isNaN(parsedAsOf.getTime()))
             return null;
+        logger_1.default.debug(`[RunlessResolver] Fetching work item ${id} by asOf (raw="${String(asOfTimestamp || '')}", normalized="${parsedAsOf.toISOString()}", expandAll=${String(expandAll)})`);
         const query = [`asOf=${encodeURIComponent(parsedAsOf.toISOString())}`];
         if (expandAll) {
             query.push(`$expand=all`);
@@ -2791,22 +2792,62 @@ class ResultDataProvider {
             return null;
         }
     }
+    /**
+     * Returns true when the snapshot includes a non-empty test steps XML payload.
+     */
+    hasStepsInWorkItemSnapshot(workItemData) {
+        const stepsXml = this.extractStepsXmlFromFieldsMap((workItemData === null || workItemData === void 0 ? void 0 : workItemData.fields) || {});
+        return String(stepsXml || '').trim() !== '';
+    }
+    logRunlessSnapshotDecision(testCaseId, source, snapshot) {
+        var _a;
+        if (!snapshot) {
+            logger_1.default.debug(`[RunlessResolver] TC ${testCaseId}: source=${source}, snapshot=none`);
+            return;
+        }
+        const stepsXml = this.extractStepsXmlFromFieldsMap((snapshot === null || snapshot === void 0 ? void 0 : snapshot.fields) || {});
+        const stepsLength = String(stepsXml || '').trim().length;
+        logger_1.default.debug(`[RunlessResolver] TC ${testCaseId}: source=${source}, rev=${String((_a = snapshot === null || snapshot === void 0 ? void 0 : snapshot.rev) !== null && _a !== void 0 ? _a : '')}, hasSteps=${String(stepsLength > 0)}, stepsLength=${String(stepsLength)}`);
+    }
     /**
      * Resolves runless test case data using ordered fallbacks:
      * 1) point-based `asOf` snapshot, 2) explicit revision, 3) suite payload snapshot, 4) latest WI.
      */
     async resolveRunlessTestCaseData(projectName, testCaseId, suiteTestCaseRevision, pointAsOfTimestamp, fallbackSnapshot, expandAll) {
+        let bestSnapshotWithoutSteps = null;
         if (pointAsOfTimestamp) {
             const asOfSnapshot = await this.fetchWorkItemByAsOf(projectName, testCaseId, pointAsOfTimestamp, expandAll);
-            if (asOfSnapshot)
-                return asOfSnapshot;
+            this.logRunlessSnapshotDecision(testCaseId, 'asOf', asOfSnapshot);
+            if (asOfSnapshot) {
+                if (this.hasStepsInWorkItemSnapshot(asOfSnapshot))
+                    return asOfSnapshot;
+                bestSnapshotWithoutSteps = asOfSnapshot;
+            }
+        }
+        else {
+            logger_1.default.debug(`[RunlessResolver] TC ${testCaseId}: asOf timestamp is empty, skipping asOf fetch`);
         }
         const revisionSnapshot = await this.fetchWorkItemByRevision(projectName, testCaseId, suiteTestCaseRevision, expandAll);
-        if (revisionSnapshot)
-            return revisionSnapshot;
-        if (fallbackSnapshot)
-            return fallbackSnapshot;
-        return this.fetchWorkItemLatest(projectName, testCaseId, expandAll);
+        this.logRunlessSnapshotDecision(testCaseId, `revision:${String(suiteTestCaseRevision)}`, revisionSnapshot);
+        if (revisionSnapshot) {
+            if (this.hasStepsInWorkItemSnapshot(revisionSnapshot))
+                return revisionSnapshot;
+            bestSnapshotWithoutSteps = bestSnapshotWithoutSteps || revisionSnapshot;
+        }
+        this.logRunlessSnapshotDecision(testCaseId, 'suiteSnapshot', fallbackSnapshot);
+        if (fallbackSnapshot) {
+            if (this.hasStepsInWorkItemSnapshot(fallbackSnapshot))
+                return fallbackSnapshot;
+            bestSnapshotWithoutSteps = bestSnapshotWithoutSteps || fallbackSnapshot;
+        }
+        const latestSnapshot = await this.fetchWorkItemLatest(projectName, testCaseId, expandAll);
+        this.logRunlessSnapshotDecision(testCaseId, 'latest', latestSnapshot);
+        if (latestSnapshot) {
+            if (this.hasStepsInWorkItemSnapshot(latestSnapshot))
+                return latestSnapshot;
+            bestSnapshotWithoutSteps = bestSnapshotWithoutSteps || latestSnapshot;
+        }
+        return bestSnapshotWithoutSteps;
     }
     /**
      * Fetches result data based on the Work Item Test Reporter.
@@ -2840,6 +2881,7 @@ class ResultDataProvider {
                 const pointAsOfTimestamp = useRunlessAsOf
                     ? String((point === null || point === void 0 ? void 0 : point.pointAsOfTimestamp) || '').trim()
                     : '';
+                logger_1.default.debug(`[RunlessResolver] Start TC ${String(testCaseId)}: useRunlessAsOf=${String(useRunlessAsOf)}, pointAsOfTimestamp="${pointAsOfTimestamp}", suiteRevision=${String(suiteTestCaseRevision)}, pointOutcome="${String((point === null || point === void 0 ? void 0 : point.outcome) || '')}"`);
                 const fallbackSnapshot = this.buildWorkItemSnapshotFromSuiteTestCase(suiteTestCaseItem, testCaseId, String((point === null || point === void 0 ? void 0 : point.testCaseName) || ''));
                 const testCaseData = await this.resolveRunlessTestCaseData(projectName, testCaseId, suiteTestCaseRevision, pointAsOfTimestamp, fallbackSnapshot, isTestReporter);
                 if (!testCaseData) {
@@ -3587,14 +3629,26 @@ class ResultDataProvider {
      */
     createIterationsMap(iterations, isTestReporter, includeNotRunTestCases) {
         return iterations.reduce((map, iterationItem) => {
-            if ((isTestReporter && iterationItem.lastRunId && iterationItem.lastResultId) ||
-                iterationItem.iteration) {
+            const hasRunIdentifiers = (iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastRunId) !== undefined &&
+                (iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastRunId) !== null &&
+                String(iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastRunId).trim() !== '' &&
+                (iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastResultId) !== undefined &&
+                (iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastResultId) !== null &&
+                String(iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.lastResultId).trim() !== '';
+            if (hasRunIdentifiers) {
                 const key = `${iterationItem.lastRunId}-${iterationItem.lastResultId}-${iterationItem.testCaseId}`;
                 map[key] = iterationItem;
             }
             else if (includeNotRunTestCases) {
                 const key = `${iterationItem.testCaseId}`;
                 map[key] = iterationItem;
+                if (isTestReporter && (iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.iteration)) {
+                    logger_1.default.debug(`[RunlessResolver] createIterationsMap: mapped runless testCaseId=${String(iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.testCaseId)} to case-only key`);
+                }
+            }
+            else if ((iterationItem === null || iterationItem === void 0 ? void 0 : iterationItem.iteration) && !isTestReporter) {
+                const key = `${iterationItem.lastRunId}-${iterationItem.lastResultId}-${iterationItem.testCaseId}`;
+                map[key] = iterationItem;
             }
             return map;
         }, {});