feat(reporter): Removed the mention of suitename in classnameTemplate

Change-Id: I6d4e936a6cabaacf31a82036acf22f9af401bdb0

Author: Jean-Philippe Leclerc

docs/guide/reporters.md

@@ -254,13 +254,12 @@ The outputted XML contains nested `testsuites` and `testcase` tags. You can use
 The supported placeholders for the `classnameTemplate` option are:
 - filename
 - filepath
-- suitename
 
 ```ts
 export default defineConfig({
   test: {
     reporters: [
-      ['junit', { suiteName: 'custom suite name', classnameTemplate: 'filename:{filename} - filepath:{filepath} - suite:{suitename}' }]
+      ['junit', { suiteName: 'custom suite name', classnameTemplate: 'filename:{filename} - filepath:{filepath}' }]
     ]
   },
 })

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

@@ -14,7 +14,6 @@ import { IndentedLogger } from './renderers/indented-logger'
 interface ClassnameTemplateVariables {
   filename: string
   filepath: string
-  suitename: string
 }
 
 export interface JUnitOptions {
@@ -23,7 +22,7 @@ export interface JUnitOptions {
   classname?: string
 
   /**
-   * Template for the classname attribute. Can be either a string or a function. The string can contain placeholders like {filename}, {filepath}, {suitename}.
+   * Template for the classname attribute. Can be either a string or a function. The string can contain placeholders {filename} and {filepath}.
    */
   classnameTemplate?: string | ((classnameVariables: ClassnameTemplateVariables) => string)
   suiteName?: string
@@ -212,7 +211,6 @@ export class JUnitReporter implements Reporter {
       const templateVars: ClassnameTemplateVariables = {
         filename: task.file.name,
         filepath: task.file.filepath,
-        suitename: this.options.suiteName ?? task.suite?.name ?? '',
       }
 
       if (typeof this.options.classnameTemplate === 'function') {
@@ -222,7 +220,6 @@ export class JUnitReporter implements Reporter {
         classname = this.options.classnameTemplate
           .replace(/\{filename\}/g, templateVars.filename)
           .replace(/\{filepath\}/g, templateVars.filepath)
-          .replace(/\{suitename\}/g, templateVars.suitename)
       }
       else if (typeof this.options.classname === 'string') {
         classname = this.options.classname

test/reporters/tests/__snapshots__/reporters.spec.ts.snap

@@ -18,7 +18,7 @@ exports[`JUnit reporter with custom function classnameTemplate 1`] = `
 "<?xml version="1.0" encoding="UTF-8" ?>
 <testsuites name="vitest tests" tests="1" failures="0" errors="0" time="0">
     <testsuite name="test/core/test/basic.test.ts" timestamp="2022-01-19T10:10:01.759Z" hostname="hostname" tests="1" failures="0" errors="0" skipped="0" time="0.145992842">
-        <testcase classname="filename:basic.test.ts - filepath:/vitest/test/core/test/basic.test.ts - suite:suite" name="Math.sqrt()" time="0.001442286">
+        <testcase classname="filename:basic.test.ts - filepath:/vitest/test/core/test/basic.test.ts" name="Math.sqrt()" time="0.001442286">
         </testcase>
     </testsuite>
 </testsuites>
@@ -40,7 +40,7 @@ exports[`JUnit reporter with custom string classnameTemplate 1`] = `
 "<?xml version="1.0" encoding="UTF-8" ?>
 <testsuites name="vitest tests" tests="1" failures="0" errors="0" time="0">
     <testsuite name="test/core/test/basic.test.ts" timestamp="2022-01-19T10:10:01.759Z" hostname="hostname" tests="1" failures="0" errors="0" skipped="0" time="0.145992842">
-        <testcase classname="filename:basic.test.ts - filepath:/vitest/test/core/test/basic.test.ts - suite:suite" name="Math.sqrt()" time="0.001442286">
+        <testcase classname="filename:basic.test.ts - filepath:/vitest/test/core/test/basic.test.ts" name="Math.sqrt()" time="0.001442286">
         </testcase>
     </testsuite>
 </testsuites>

test/reporters/tests/reporters.spec.ts

@@ -90,7 +90,7 @@ test('JUnit reporter with custom string classname', async () => {
 
 test('JUnit reporter with custom function classnameTemplate', async () => {
   // Arrange
-  const reporter = new JUnitReporter({ classnameTemplate: task => `filename:${task.filename} - filepath:${task.filepath} - suite:${task.suitename}` })
+  const reporter = new JUnitReporter({ classnameTemplate: task => `filename:${task.filename} - filepath:${task.filepath}` })
   const context = getContext()
 
   // Act
@@ -103,7 +103,7 @@ test('JUnit reporter with custom function classnameTemplate', async () => {
 })
 test('JUnit reporter with custom string classnameTemplate', async () => {
   // Arrange
-  const reporter = new JUnitReporter({ classnameTemplate: `filename:{filename} - filepath:{filepath} - suite:{suitename}` })
+  const reporter = new JUnitReporter({ classnameTemplate: `filename:{filename} - filepath:{filepath}` })
   const context = getContext()
 
   // Act