fix: correct transform time calculation in merged report (#10570) (#10578)

Co-authored-by: Hiroshi Ogawa <hi.ogawa.zz@gmail.com>

Author: potatomatoooo

packages/vitest/src/node/core.ts

@@ -642,8 +642,9 @@ export class Vitest {
         throw new Error('Cannot merge reports when `--reporter=blob` is used. Remove blob reporter from the config first.')
       }
 
-      const { files, errors, coverages, executionTimes } = await readBlobs(this.version, directory || this.config.mergeReports, this.projects)
-      this.state.blobs = { files, errors, coverages, executionTimes }
+      const { files, errors, coverages, executionTimes, transformTimes } = await readBlobs(this.version, directory || this.config.mergeReports, this.projects)
+      this.state.blobs = { files, errors, coverages, executionTimes, transformTimes }
+      this.state.transformTime = transformTimes.reduce((a, b) => a + b, 0)
 
       await this.report('onInit', this)
 

packages/vitest/src/node/reporters/blob.ts

@@ -82,6 +82,7 @@ export class BlobReporter implements Reporter {
       coverage,
       executionTime,
       environmentModules,
+      this.ctx.state.transformTime,
     ] satisfies MergeReport)
 
     let outputFile = this.options.outputFile ?? getOutputFile(this.ctx.config, 'blob')
@@ -133,15 +134,15 @@ export async function readBlobs(
       )
     }
     const content = await readFile(fullPath, 'utf-8')
-    const [version, files, errors, coverage, executionTime, environmentModules] = parse(
+    const [version, files, errors, coverage, executionTime, environmentModules, transformTime] = parse(
       content,
     ) as MergeReport
     if (!version) {
       throw new TypeError(
         `vitest.mergeReports() expects all paths in "${blobsDirectory}" to be files generated by the blob reporter, but "${filename}" is not a valid blob file`,
       )
     }
-    return { version, files, errors, coverage, file: filename, executionTime, environmentModules }
+    return { version, files, errors, coverage, file: filename, executionTime, environmentModules, transformTime }
   })
   const blobs = await Promise.all(promises)
 
@@ -209,12 +210,14 @@ export async function readBlobs(
   const errors = blobs.flatMap(blob => blob.errors)
   const coverages = blobs.map(blob => blob.coverage)
   const executionTimes = blobs.map(blob => blob.executionTime)
+  const transformTimes = blobs.map(blob => blob.transformTime)
 
   return {
     files,
     errors,
     coverages,
     executionTimes,
+    transformTimes,
   }
 }
 
@@ -223,6 +226,7 @@ export interface MergedBlobs {
   errors: unknown[]
   coverages: unknown[]
   executionTimes: number[]
+  transformTimes: number[]
 }
 
 export type MergeReport = [
@@ -232,6 +236,7 @@ export type MergeReport = [
   coverage: unknown,
   executionTime: number,
   environmentModules: MergeReportEnvironmentModules,
+  transformTime: number,
 ]
 
 interface MergeReportEnvironmentModules {

test/e2e/test/reporters/merge-reports.test.ts

@@ -265,7 +265,7 @@ test('total and merged execution times are shown', async () => {
     file.tasks.push(createTest('some test', file))
 
     await writeBlob(
-      [version, [file], [], undefined, 1500 * index, {}],
+      [version, [file], [], undefined, 1500 * index, {}, 2000 * index],
       resolve(`./fixtures/reporters/merge-reports/.vitest/blob/blob-${index}-2.json`),
     )
   }
@@ -279,7 +279,7 @@ test('total and merged execution times are shown', async () => {
   expect(stdout).toContain('✓ first.test.ts (1 test)')
   expect(stdout).toContain('✓ second.test.ts (1 test)')
 
-  expect(stdout).toContain('Duration  4.50s')
+  expect(stdout).toContain('Duration  4.50s (transform 6.00s')
   expect(stdout).toContain('Per blob  1.50s 3.00s')
 })