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

package/bin/modules/ResultDataProvider.d.ts

@@ -194,6 +194,15 @@ export default class ResultDataProvider {
      * Maps raw test point data to a simplified object.
      */
     private mapTestPointForCrossPlans;
+    /**
+     * Resolves a point timestamp that can be used as WIT `asOf` anchor.
+     * Returns an ISO string when a valid date exists, otherwise an empty string.
+     */
+    private resolvePointAsOfTimestamp;
+    /**
+     * Builds a normalized test point shape used by downstream result fetch flows.
+     */
+    private buildMappedTestPoint;
     getTestPointsForTestCases(projectName: string, testCaseId: string[]): Promise<any>;
     /**
      * Fetches test cases by suite ID.
@@ -201,10 +210,24 @@ export default class ResultDataProvider {
     private fetchTestCasesBySuiteId;
     private attachSuiteTestCaseContextToPoints;
     private extractWorkItemFieldsMap;
+    /**
+     * Reads a field value by reference name with case-insensitive key matching.
+     */
+    private getFieldValueByName;
     private resolveSuiteTestCaseRevision;
     private buildWorkItemSnapshotFromSuiteTestCase;
     private fetchWorkItemByRevision;
+    private fetchWorkItemByAsOf;
     private fetchWorkItemLatest;
+    /**
+     * Executes a work-item GET by URL and applies consistent warning/error handling.
+     */
+    private fetchWorkItemByUrl;
+    /**
+     * Resolves runless test case data using ordered fallbacks:
+     * 1) point-based `asOf` snapshot, 2) explicit revision, 3) suite payload snapshot, 4) latest WI.
+     */
+    private resolveRunlessTestCaseData;
     /**
      * Fetches result data based on the Work Item Test Reporter.
      *

package/bin/modules/ResultDataProvider.js

@@ -274,7 +274,7 @@ class ResultDataProvider {
             const planName = await this.fetchTestPlanName(testPlanId, projectName);
             const suites = await this.fetchTestSuites(testPlanId, projectName, selectedSuiteIds, true);
             const testData = await this.fetchTestData(suites, projectName, testPlanId, false);
-            const runResults = await this.fetchAllResultDataTestReporter(testData, projectName, selectedFields, false, includeAllHistory);
+            const runResults = await this.fetchAllResultDataTestReporter(testData, projectName, selectedFields, false, includeAllHistory, true);
             const rows = this.alignStepsWithIterationsFlatReport(testData, runResults, true, testPlanId, planName);
             return { planId: testPlanId, planName, rows: rows || [] };
         }
@@ -2516,7 +2516,7 @@ class ResultDataProvider {
             }
             // Fetch detailed information for each test point and map to required format
             const detailedPoints = await Promise.all(latestPoints.map(async (point) => {
-                const url = `${point.url}?witFields=Microsoft.VSTS.TCM.Steps&includePointDetails=true`;
+                const url = `${point.url}?witFields=Microsoft.VSTS.TCM.Steps,System.Rev&includePointDetails=true`;
                 const detailedPoint = await tfs_1.TFSServices.getItemContent(url, this.token);
                 return this.mapTestPointForCrossPlans(detailedPoint, projectName);
                 // return this.mapTestPointForCrossPlans(detailedPoint, projectName);
@@ -2548,32 +2548,32 @@ class ResultDataProvider {
      */
     mapTestPoint(testPoint, projectName) {
         var _a, _b, _c, _d, _e;
-        return {
+        const pointAsOfTimestamp = this.resolvePointAsOfTimestamp(testPoint);
+        return this.buildMappedTestPoint({
             testPointId: testPoint.id,
             testCaseId: testPoint.testCaseReference.id,
             testCaseName: testPoint.testCaseReference.name,
-            testCaseUrl: `${this.orgUrl}${projectName}/_workitems/edit/${testPoint.testCaseReference.id}`,
             configurationName: (_a = testPoint.configuration) === null || _a === void 0 ? void 0 : _a.name,
-            outcome: ((_b = testPoint.results) === null || _b === void 0 ? void 0 : _b.outcome) || 'Not Run',
+            outcome: (_b = testPoint.results) === null || _b === void 0 ? void 0 : _b.outcome,
             testSuite: testPoint.testSuite,
             lastRunId: (_c = testPoint.results) === null || _c === void 0 ? void 0 : _c.lastTestRunId,
             lastResultId: (_d = testPoint.results) === null || _d === void 0 ? void 0 : _d.lastResultId,
             lastResultDetails: (_e = testPoint.results) === null || _e === void 0 ? void 0 : _e.lastResultDetails,
-        };
+        }, projectName, pointAsOfTimestamp);
     }
     /**
      * Maps raw test point data to a simplified object.
      */
     mapTestPointForCrossPlans(testPoint, projectName) {
         var _a, _b, _c;
-        return {
+        const pointAsOfTimestamp = this.resolvePointAsOfTimestamp(testPoint);
+        return this.buildMappedTestPoint({
             testPointId: testPoint.id,
             testCaseId: testPoint.testCase.id,
             testCaseName: testPoint.testCase.name,
-            testCaseUrl: `${this.orgUrl}${projectName}/_workitems/edit/${testPoint.testCase.id}`,
-            testSuite: testPoint.testSuite,
             configurationName: (_a = testPoint.configuration) === null || _a === void 0 ? void 0 : _a.name,
-            outcome: testPoint.outcome || 'Not Run',
+            outcome: testPoint.outcome,
+            testSuite: testPoint.testSuite,
             lastRunId: (_b = testPoint.lastTestRun) === null || _b === void 0 ? void 0 : _b.id,
             lastResultId: (_c = testPoint.lastResult) === null || _c === void 0 ? void 0 : _c.id,
             lastResultDetails: testPoint.lastResultDetails || {
@@ -2581,7 +2581,50 @@ class ResultDataProvider {
                 dateCompleted: '0000-00-00T00:00:00.000Z',
                 runBy: { displayName: 'No tester', id: '00000000-0000-0000-0000-000000000000' },
             },
+        }, projectName, pointAsOfTimestamp);
+    }
+    /**
+     * Resolves a point timestamp that can be used as WIT `asOf` anchor.
+     * Returns an ISO string when a valid date exists, otherwise an empty string.
+     */
+    resolvePointAsOfTimestamp(testPoint) {
+        const candidates = [
+            testPoint === null || testPoint === void 0 ? void 0 : testPoint.lastUpdatedDate,
+            testPoint === null || testPoint === void 0 ? void 0 : testPoint.lastUpdatedOn,
+            testPoint === null || testPoint === void 0 ? void 0 : testPoint.updatedDate,
+            testPoint === null || testPoint === void 0 ? void 0 : testPoint.updatedOn,
+        ];
+        for (const candidate of candidates) {
+            const raw = String(candidate || '').trim();
+            if (!raw)
+                continue;
+            const parsed = new Date(raw);
+            if (!Number.isNaN(parsed.getTime())) {
+                return parsed.toISOString();
+            }
+        }
+        return '';
+    }
+    /**
+     * Builds a normalized test point shape used by downstream result fetch flows.
+     */
+    buildMappedTestPoint(pointData, projectName, pointAsOfTimestamp) {
+        const mappedPoint = {
+            testPointId: pointData.testPointId,
+            testCaseId: pointData.testCaseId,
+            testCaseName: pointData.testCaseName,
+            testCaseUrl: `${this.orgUrl}${projectName}/_workitems/edit/${pointData.testCaseId}`,
+            configurationName: pointData.configurationName,
+            outcome: pointData.outcome || 'Not Run',
+            testSuite: pointData.testSuite,
+            lastRunId: pointData.lastRunId,
+            lastResultId: pointData.lastResultId,
+            lastResultDetails: pointData.lastResultDetails,
         };
+        if (pointAsOfTimestamp) {
+            mappedPoint.pointAsOfTimestamp = pointAsOfTimestamp;
+        }
+        return mappedPoint;
     }
     // Helper method to get all test points for a test case
     async getTestPointsForTestCases(projectName, testCaseId) {
@@ -2597,7 +2640,7 @@ class ResultDataProvider {
      * Fetches test cases by suite ID.
      */
     async fetchTestCasesBySuiteId(projectName, testPlanId, suiteId) {
-        const url = `${this.orgUrl}${projectName}/_apis/testplan/Plans/${testPlanId}/Suites/${suiteId}/TestCase?witFields=Microsoft.VSTS.TCM.Steps`;
+        const url = `${this.orgUrl}${projectName}/_apis/testplan/Plans/${testPlanId}/Suites/${suiteId}/TestCase?witFields=Microsoft.VSTS.TCM.Steps,System.Rev`;
         const { value: testCases } = await tfs_1.TFSServices.getItemContent(url, this.token);
         return testCases;
     }
@@ -2639,14 +2682,35 @@ class ResultDataProvider {
         }
         return fields;
     }
+    /**
+     * Reads a field value by reference name with case-insensitive key matching.
+     */
+    getFieldValueByName(fields, fieldName) {
+        if (!fields || typeof fields !== 'object')
+            return undefined;
+        if (Object.prototype.hasOwnProperty.call(fields, fieldName)) {
+            return fields[fieldName];
+        }
+        const lookupName = String(fieldName || '').toLowerCase().trim();
+        if (!lookupName)
+            return undefined;
+        const matchedKey = Object.keys(fields).find((key) => String(key || '').toLowerCase().trim() === lookupName);
+        return matchedKey ? fields[matchedKey] : undefined;
+    }
     resolveSuiteTestCaseRevision(testCaseItem) {
-        var _a, _b, _c, _d, _e;
+        var _a, _b, _c, _d, _e, _f, _g;
+        const fieldsFromList = this.extractWorkItemFieldsMap((_a = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _a === void 0 ? void 0 : _a.workItemFields);
+        const fieldsFromMap = ((_b = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _b === void 0 ? void 0 : _b.fields) || {};
+        const systemRevFromList = this.getFieldValueByName(fieldsFromList, 'System.Rev');
+        const systemRevFromMap = this.getFieldValueByName(fieldsFromMap, 'System.Rev');
         const revisionCandidates = [
-            (_a = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _a === void 0 ? void 0 : _a.rev,
-            (_b = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _b === void 0 ? void 0 : _b.revision,
-            (_c = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _c === void 0 ? void 0 : _c.version,
-            (_d = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _d === void 0 ? void 0 : _d.workItemRevision,
-            (_e = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _e === void 0 ? void 0 : _e.workItemVersion,
+            systemRevFromList,
+            systemRevFromMap,
+            (_c = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _c === void 0 ? void 0 : _c.rev,
+            (_d = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _d === void 0 ? void 0 : _d.revision,
+            (_e = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _e === void 0 ? void 0 : _e.version,
+            (_f = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _f === void 0 ? void 0 : _f.workItemRevision,
+            (_g = testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItem) === null || _g === void 0 ? void 0 : _g.workItemVersion,
             testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.revision,
             testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItemRevision,
             testCaseItem === null || testCaseItem === void 0 ? void 0 : testCaseItem.workItemVersion,
@@ -2693,13 +2757,19 @@ class ResultDataProvider {
             return null;
         const expandParam = expandAll ? '?$expand=all' : '';
         const url = `${this.orgUrl}${projectName}/_apis/wit/workItems/${id}/revisions/${rev}${expandParam}`;
-        try {
-            return await tfs_1.TFSServices.getItemContent(url, this.token);
-        }
-        catch (error) {
-            logger_1.default.warn(`Failed to fetch work item ${id} by revision ${rev}: ${(error === null || error === void 0 ? void 0 : error.message) || error}`);
+        return this.fetchWorkItemByUrl(url, `work item ${id} by revision ${rev}`);
+    }
+    async fetchWorkItemByAsOf(projectName, workItemId, asOfTimestamp, expandAll = false) {
+        const id = Number(workItemId || 0);
+        const parsedAsOf = new Date(String(asOfTimestamp || '').trim());
+        if (!Number.isFinite(id) || id <= 0 || Number.isNaN(parsedAsOf.getTime()))
             return null;
+        const query = [`asOf=${encodeURIComponent(parsedAsOf.toISOString())}`];
+        if (expandAll) {
+            query.push(`$expand=all`);
         }
+        const url = `${this.orgUrl}${projectName}/_apis/wit/workItems/${id}?${query.join('&')}`;
+        return this.fetchWorkItemByUrl(url, `work item ${id} by asOf ${parsedAsOf.toISOString()}`);
     }
     async fetchWorkItemLatest(projectName, workItemId, expandAll = false) {
         const id = Number(workItemId || 0);
@@ -2707,14 +2777,37 @@ class ResultDataProvider {
             return null;
         const expandParam = expandAll ? '?$expand=all' : '';
         const url = `${this.orgUrl}${projectName}/_apis/wit/workItems/${id}${expandParam}`;
+        return this.fetchWorkItemByUrl(url, `latest work item ${id}`);
+    }
+    /**
+     * Executes a work-item GET by URL and applies consistent warning/error handling.
+     */
+    async fetchWorkItemByUrl(url, failureContext) {
         try {
             return await tfs_1.TFSServices.getItemContent(url, this.token);
         }
         catch (error) {
-            logger_1.default.warn(`Failed to fetch latest work item ${id}: ${(error === null || error === void 0 ? void 0 : error.message) || error}`);
+            logger_1.default.warn(`Failed to fetch ${failureContext}: ${(error === null || error === void 0 ? void 0 : error.message) || error}`);
             return null;
         }
     }
+    /**
+     * 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) {
+        if (pointAsOfTimestamp) {
+            const asOfSnapshot = await this.fetchWorkItemByAsOf(projectName, testCaseId, pointAsOfTimestamp, expandAll);
+            if (asOfSnapshot)
+                return asOfSnapshot;
+        }
+        const revisionSnapshot = await this.fetchWorkItemByRevision(projectName, testCaseId, suiteTestCaseRevision, expandAll);
+        if (revisionSnapshot)
+            return revisionSnapshot;
+        if (fallbackSnapshot)
+            return fallbackSnapshot;
+        return this.fetchWorkItemLatest(projectName, testCaseId, expandAll);
+    }
     /**
      * Fetches result data based on the Work Item Test Reporter.
      *
@@ -2729,7 +2822,7 @@ class ResultDataProvider {
      * @param isQueryMode - (Optional) A flag indicating whether the result data is being fetched in query mode.
      * @returns A promise that resolves to the fetched result data.
      */
-    async fetchResultDataBasedOnWiBase(projectName, runId, resultId, isTestReporter = false, selectedFields, isQueryMode, point, includeAllHistory = false) {
+    async fetchResultDataBasedOnWiBase(projectName, runId, resultId, isTestReporter = false, selectedFields, isQueryMode, point, includeAllHistory = false, useRunlessAsOf = false) {
         var _a, _b, _c, _d, _e, _f, _g, _h, _j;
         try {
             let filteredFields = {};
@@ -2744,14 +2837,11 @@ class ResultDataProvider {
                 const suiteTestCaseItem = point === null || point === void 0 ? void 0 : point.suiteTestCase;
                 const testCaseId = Number((point === null || point === void 0 ? void 0 : point.testCaseId) || ((_a = suiteTestCaseItem === null || suiteTestCaseItem === void 0 ? void 0 : suiteTestCaseItem.workItem) === null || _a === void 0 ? void 0 : _a.id) || (suiteTestCaseItem === null || suiteTestCaseItem === void 0 ? void 0 : suiteTestCaseItem.testCaseId) || 0);
                 const suiteTestCaseRevision = this.resolveSuiteTestCaseRevision(suiteTestCaseItem);
+                const pointAsOfTimestamp = useRunlessAsOf
+                    ? String((point === null || point === void 0 ? void 0 : point.pointAsOfTimestamp) || '').trim()
+                    : '';
                 const fallbackSnapshot = this.buildWorkItemSnapshotFromSuiteTestCase(suiteTestCaseItem, testCaseId, String((point === null || point === void 0 ? void 0 : point.testCaseName) || ''));
-                let testCaseData = await this.fetchWorkItemByRevision(projectName, testCaseId, suiteTestCaseRevision, isTestReporter);
-                if (!testCaseData) {
-                    testCaseData = fallbackSnapshot;
-                }
-                if (!testCaseData) {
-                    testCaseData = await this.fetchWorkItemLatest(projectName, testCaseId, isTestReporter);
-                }
+                const testCaseData = await this.resolveRunlessTestCaseData(projectName, testCaseId, suiteTestCaseRevision, pointAsOfTimestamp, fallbackSnapshot, isTestReporter);
                 if (!testCaseData) {
                     logger_1.default.warn(`Could not resolve test case ${point.testCaseId} for runless point fallback.`);
                     return null;
@@ -4027,8 +4117,8 @@ class ResultDataProvider {
      * @param selectedFields - (Optional) An array of field names to include in the result data.
      * @returns A promise that resolves to the fetched result data.
      */
-    async fetchResultDataBasedOnWiTestReporter(projectName, runId, resultId, selectedFields, isQueryMode, point, includeAllHistory = false) {
-        return this.fetchResultDataBasedOnWiBase(projectName, runId, resultId, true, selectedFields, isQueryMode, point, includeAllHistory);
+    async fetchResultDataBasedOnWiTestReporter(projectName, runId, resultId, selectedFields, isQueryMode, point, includeAllHistory = false, useRunlessAsOf = false) {
+        return this.fetchResultDataBasedOnWiBase(projectName, runId, resultId, true, selectedFields, isQueryMode, point, includeAllHistory, useRunlessAsOf);
     }
     /**
      * Fetches all result data for the test reporter by processing the provided test data.
@@ -4042,8 +4132,8 @@ class ResultDataProvider {
      * @param selectedFields - An optional array of field names to include in the result data.
      * @returns A promise that resolves to an array of processed result data.
      */
-    async fetchAllResultDataTestReporter(testData, projectName, selectedFields, isQueryMode, includeAllHistory = false) {
-        return this.fetchAllResultDataBase(testData, projectName, true, (projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory) => this.fetchResultDataForTestReporter(projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory), [selectedFields, isQueryMode, includeAllHistory]);
+    async fetchAllResultDataTestReporter(testData, projectName, selectedFields, isQueryMode, includeAllHistory = false, useRunlessAsOf = false) {
+        return this.fetchAllResultDataBase(testData, projectName, true, (projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory, useRunlessAsOf) => this.fetchResultDataForTestReporter(projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory, useRunlessAsOf), [selectedFields, isQueryMode, includeAllHistory, useRunlessAsOf]);
     }
     /**
      * Aligns test steps with iterations for the test reporter by processing test data and iterations
@@ -4151,8 +4241,8 @@ class ResultDataProvider {
      * @returns A promise that resolves to the formatted result data object containing details about the test case,
      *          test suite, last run, iteration, and other selected fields.
      */
-    async fetchResultDataForTestReporter(projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory = false) {
-        return this.fetchResultDataBase(projectName, testSuiteId, point, (project, runId, resultId, fields, isQueryMode, point, includeAllHistory) => this.fetchResultDataBasedOnWiTestReporter(project, runId, resultId, fields, isQueryMode, point, includeAllHistory), (resultData, testSuiteId, point, selectedFields) => {
+    async fetchResultDataForTestReporter(projectName, testSuiteId, point, selectedFields, isQueryMode, includeAllHistory = false, useRunlessAsOf = false) {
+        return this.fetchResultDataBase(projectName, testSuiteId, point, (project, runId, resultId, fields, isQueryMode, point, includeAllHistory, useRunlessAsOf) => this.fetchResultDataBasedOnWiTestReporter(project, runId, resultId, fields, isQueryMode, point, includeAllHistory, useRunlessAsOf), (resultData, testSuiteId, point, selectedFields) => {
             var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u, _v, _w;
             const { lastRunId, lastResultId, configurationName, lastResultDetails } = point;
             try {
@@ -4245,7 +4335,7 @@ class ResultDataProvider {
                 logger_1.default.error(`Error stack: ${err.stack}`);
                 return null;
             }
-        }, [selectedFields, isQueryMode, point, includeAllHistory]);
+        }, [selectedFields, isQueryMode, point, includeAllHistory, useRunlessAsOf]);
     }
     getTestOutcome(resultData) {
         // Default outcome if nothing else is available