feat: change respect workflow-level totalTimeMs calculation (#2322)

Author: DmitryAnansky

.changeset/twelve-birds-wonder.md

@@ -0,0 +1,6 @@
+---
+"@redocly/respect-core": minor
+"@redocly/cli": minor
+---
+
+Adjusted the calculation of Respect's workflow-level `totalTimeMs` to sum the network request times of all steps.

packages/respect-core/src/modules/__tests__/flow-runner/runner/create-workflow-runner.test.ts

@@ -119,6 +119,7 @@ describe('runWorkflow', () => {
         filePath: fileName,
         logger,
       },
+      executedSteps: [],
     } as unknown as TestContext;
 
     await runWorkflow({ workflowInput: 'test', ctx, executedStepsCount: { value: 0 } });
@@ -298,6 +299,7 @@ describe('runWorkflow', () => {
         logger,
       },
       $outputs: {},
+      executedSteps: [],
     } as unknown as TestContext;
 
     await runWorkflow({ workflowInput: 'test', ctx, executedStepsCount: { value: 0 } });
@@ -352,6 +354,7 @@ describe('runWorkflow', () => {
         logger,
       },
       $outputs: {},
+      executedSteps: [],
     } as unknown as TestContext;
 
     await runWorkflow({

packages/respect-core/src/modules/flow-runner/runner.ts

@@ -28,6 +28,7 @@ import type {
   RunWorkflowInput,
   WorkflowExecutionResult,
   ExecutedStepsCount,
+  Step,
 } from '../../types.js';
 
 export async function runTestFile({
@@ -237,7 +238,7 @@ export async function runWorkflow({
     stepId: parentStepId,
     startTime: workflowStartTime,
     endTime,
-    totalTimeMs: Math.ceil(endTime - workflowStartTime),
+    totalTimeMs: calculateWorkflowTotalTimeMs(ctx.executedSteps),
     executedSteps: ctx.executedSteps,
     ctx,
     globalTimeoutError: hasFailedTimeoutSteps,
@@ -339,3 +340,23 @@ function findSourceDescriptionUrl(
     );
   }
 }
+
+function isWorkflowExecutionResult(
+  step: Step | WorkflowExecutionResult
+): step is WorkflowExecutionResult {
+  return 'type' in step && step.type === 'workflow';
+}
+
+function calculateWorkflowTotalTimeMs(executedSteps: (Step | WorkflowExecutionResult)[]) {
+  let totalTime = 0;
+
+  for (const step of executedSteps) {
+    if (isWorkflowExecutionResult(step)) {
+      totalTime += step.totalTimeMs;
+    } else {
+      totalTime += step.response?.time || 0;
+    }
+  }
+
+  return totalTime;
+}