@elisra-devops/docgen-data-provider 1.136.3 → 1.136.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elisra-devops/docgen-data-provider",
-  "version": "1.136.3",
+  "version": "1.136.4",
   "description": "A document generator data provider, aimed to retrive data from azure devops",
   "repository": {
     "type": "git",

package/src/modules/ResultDataProvider.ts

@@ -3747,6 +3747,11 @@ export default class ResultDataProvider {
         const testCaseId = Number(
           point?.testCaseId || suiteTestCaseItem?.workItem?.id || suiteTestCaseItem?.testCaseId || 0
         );
+        if (useLatestTestCaseProperties) {
+          logger.debug(
+            `[TestReporter][LatestProperties] Fetching latest test case work item for testCaseId=${testCaseId}, runId=${runId}, resultId=${resultId}`
+          );
+        }
         const suiteTestCaseRevision = this.resolveSuiteTestCaseRevision(suiteTestCaseItem);
         const pointAsOfTimestamp = useRunlessAsOf
           ? String(point?.pointAsOfTimestamp || '').trim()
@@ -5826,7 +5831,7 @@ export default class ResultDataProvider {
           return null;
         }
       },
-      [selectedFields, isQueryMode, point, includeAllHistory, useRunlessAsOf]
+      [selectedFields, isQueryMode, point, includeAllHistory, useRunlessAsOf, useLatestTestCaseProperties]
     );
   }
 

package/src/tests/modules/ResultDataProvider.test.ts

@@ -4300,6 +4300,32 @@ describe('ResultDataProvider', () => {
 
       expect(res.testCaseResult).toEqual(expect.objectContaining({ url: '' }));
     });
+
+    it('should pass latest test case properties flag to the point result fetch', async () => {
+      const fetchResultDataBaseSpy = jest
+        .spyOn(resultDataProvider as any, 'fetchResultDataBase')
+        .mockResolvedValue({});
+
+      await (resultDataProvider as any).fetchResultDataForTestReporter(
+        mockProjectName,
+        'suite1',
+        { lastRunId: 456, lastResultId: 789, testCaseId: 123 },
+        ['Custom.SortIndex@testCaseWorkItemField'],
+        false,
+        false,
+        false,
+        true
+      );
+
+      expect(fetchResultDataBaseSpy.mock.calls[0][5]).toEqual([
+        ['Custom.SortIndex@testCaseWorkItemField'],
+        false,
+        { lastRunId: 456, lastResultId: 789, testCaseId: 123 },
+        false,
+        false,
+        true,
+      ]);
+    });
   });
 
   describe('fetchResultDataBasedOnWiBase', () => {
@@ -4352,6 +4378,95 @@ describe('ResultDataProvider', () => {
       );
     });
 
+    it('should use latest test case fields for executed points when latest properties mode is enabled', async () => {
+      const point = {
+        testCaseId: '123',
+        testCaseName: 'TC 123',
+        outcome: 'passed',
+        lastRunId: 456,
+        lastResultId: 789,
+        testSuite: { id: '1', name: 'Suite' },
+      };
+
+      (TFSServices.getItemContent as jest.Mock).mockResolvedValueOnce({
+        id: 123,
+        rev: 10,
+        fields: {
+          'System.State': 'Active',
+          'System.CreatedDate': '2024-01-01T00:00:00',
+          'Microsoft.VSTS.TCM.Priority': 1,
+          'Custom.SortIndex': 3,
+          'Microsoft.VSTS.TCM.Steps': '<steps></steps>',
+        },
+        relations: null,
+      });
+
+      const selectedFields = ['Custom.SortIndex@testCaseWorkItemField'];
+      const res = await (resultDataProvider as any).fetchResultDataBasedOnWiBase(
+        mockProjectName,
+        '456',
+        '789',
+        true,
+        selectedFields,
+        false,
+        point,
+        false,
+        false,
+        true
+      );
+
+      const calledUrls = (TFSServices.getItemContent as jest.Mock).mock.calls.map((args: any[]) => String(args[0]));
+      expect(calledUrls.some((url: string) => url.includes('/_apis/test/runs/456/results/789'))).toBe(false);
+      expect(calledUrls.some((url: string) => url.includes('/_apis/wit/workItems/123?$expand=all'))).toBe(true);
+      expect(res).toEqual(
+        expect.objectContaining({
+          testCaseRevision: 10,
+          filteredFields: { 'Custom.SortIndex': 3 },
+        })
+      );
+    });
+
+    it('should log latest properties mode when bypassing run revision data', async () => {
+      const point = {
+        testCaseId: '123',
+        testCaseName: 'TC 123',
+        outcome: 'passed',
+        lastRunId: 456,
+        lastResultId: 789,
+        testSuite: { id: '1', name: 'Suite' },
+      };
+
+      (TFSServices.getItemContent as jest.Mock).mockResolvedValueOnce({
+        id: 123,
+        rev: 10,
+        fields: {
+          'System.State': 'Active',
+          'System.CreatedDate': '2024-01-01T00:00:00',
+          'Microsoft.VSTS.TCM.Priority': 1,
+          'Custom.SortIndex': 3,
+          'Microsoft.VSTS.TCM.Steps': '<steps></steps>',
+        },
+        relations: null,
+      });
+
+      await (resultDataProvider as any).fetchResultDataBasedOnWiBase(
+        mockProjectName,
+        '456',
+        '789',
+        true,
+        ['Custom.SortIndex@testCaseWorkItemField'],
+        false,
+        point,
+        false,
+        false,
+        true
+      );
+
+      expect(logger.debug).toHaveBeenCalledWith(
+        expect.stringContaining('[TestReporter][LatestProperties] Fetching latest test case work item')
+      );
+    });
+
     it('should fetch no-run test case by suite test-case revision when provided', async () => {
       const point = {
         testCaseId: '123',