fix(reporters): testsuite name should include project root in Junit output (#5116)

Author: fenghan34

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

@@ -268,7 +268,7 @@ export class JUnitReporter implements Reporter {
     await this.writeElement('testsuites', stats, async () => {
       for (const file of transformed) {
         await this.writeElement('testsuite', {
-          name: file.name,
+          name: relative(this.ctx.config.root, file.filepath),
           timestamp: (new Date()).toISOString(),
           hostname: hostname(),
           tests: file.tasks.length,

test/reporters/fixtures/better-testsuite-name/space-1/test/base.test.ts

@@ -0,0 +1,5 @@
+import { describe, test } from "vitest";
+
+describe('space 1', () => {
+  test('base', () => {})
+})

test/reporters/fixtures/better-testsuite-name/space-2/test/base.test.ts

@@ -0,0 +1,5 @@
+import { describe, test } from "vitest";
+
+describe('space 2', () => {
+  test('base', () => {})
+})

test/reporters/fixtures/better-testsuite-name/vitest.config.ts

@@ -0,0 +1,3 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({})

test/reporters/fixtures/better-testsuite-name/vitest.workspace.ts

@@ -0,0 +1,3 @@
+import { defineWorkspace } from "vitest/config";
+
+export default defineWorkspace(['space-1', 'space-2'])

test/reporters/tests/junit.test.ts

@@ -59,3 +59,13 @@ test('emits <failure> when beforeAll/afterAll failed', async () => {
   stdout = stdout.replaceAll(/(timestamp|hostname|time)=".*?"/g, '$1="..."')
   expect(stdout).toMatchSnapshot()
 })
+
+test('write testsuite name relative to root config', async () => {
+  let { stdout } = await runVitest({ reporters: 'junit', root: './fixtures/better-testsuite-name' })
+  stdout = stdout.replaceAll(/(timestamp|hostname|time)=".*?"/g, '$1="..."')
+
+  const space1 = '<testsuite name="space-1/test/base.test.ts" timestamp="..." hostname="..." tests="1" failures="0" errors="0" skipped="0" time="...">'
+  const space2 = '<testsuite name="space-2/test/base.test.ts" timestamp="..." hostname="..." tests="1" failures="0" errors="0" skipped="0" time="...">'
+  expect(stdout).toContain(space1)
+  expect(stdout).toContain(space2)
+})