chore(ci): Use relative paths in junit XML file attribute (#7642)

cbeauchesne · web-flow · commit 02c42bc14eae · 2026-03-03T10:45:11.000+01:00
* Use relative names in XML exports

* Fix mocha
diff --git a/.mochamultireporterrc.js b/.mochamultireporterrc.js
@@ -4,12 +4,12 @@ const reporterEnabled = ['spec']
 
 // eslint-disable-next-line eslint-rules/eslint-process-env
 if (process.env.CI) {
-  reporterEnabled.push('mocha-junit-reporter')
+  reporterEnabled.push('./scripts/junit-reporter.js')
 }
 
 module.exports = {
   reporterEnabled,
-  mochaJunitReporterReporterOptions: {
+  scriptsJunitReporterJsReporterOptions: {
     mochaFile: `./node-${process.versions.node}-junit.xml`,
   },
 }
diff --git a/scripts/junit-reporter.js b/scripts/junit-reporter.js
@@ -0,0 +1,23 @@
+'use strict'
+
+const path = require('node:path')
+const MochaJUnitReporter = require('mocha-junit-reporter')
+
+const root = path.resolve(__dirname, '..')
+
+/**
+ * Thin wrapper around mocha-junit-reporter that rewrites absolute `file`
+ * attributes to paths relative to the repository root.
+ */
+class JUnitReporter extends MochaJUnitReporter {
+  getTestsuiteData (suite) {
+    const testSuite = super.getTestsuiteData(suite)
+    const attrs = testSuite.testsuite[0]._attr
+    if (attrs.file) {
+      attrs.file = path.relative(root, attrs.file)
+    }
+    return testSuite
+  }
+}
+
+module.exports = JUnitReporter
diff --git a/scripts/mocha-parallel-files.js b/scripts/mocha-parallel-files.js
@@ -216,7 +216,7 @@ async function main () {
   const runFileScript = path.join(repoRoot, 'scripts', 'mocha-run-file.js')
 
   const junitTmpDir = path.join(repoRoot, '.junit-tmp')
-  const junitOutFile = path.join(repoRoot, multiMochaRc.mochaJunitReporterReporterOptions.mochaFile)
+  const junitOutFile = path.join(repoRoot, multiMochaRc.scriptsJunitReporterJsReporterOptions.mochaFile)
   const junitTmpFiles = []
   const emitJunit = Boolean(process.env.CI)
 
@@ -348,7 +348,7 @@ async function main () {
         const reporterOptions = emitJunit
           ? {
               reporterEnabled: ['spec', 'xunit'],
-              xunitReporterOptions: { output: /** @type {string} */ (junitShard) },
+              xunitReporterOptions: { output: /** @type {string} */ (junitShard), showRelativePaths: true },
             }
           : {
               reporterEnabled: ['spec'],

Original file line number	Diff line number	Diff line change
`@@ -4,12 +4,12 @@ const reporterEnabled = ['spec']`
`4`	`4`
`5`	`5`	`// eslint-disable-next-line eslint-rules/eslint-process-env`
`6`	`6`	`if (process.env.CI) {`
`7`		`- reporterEnabled.push('mocha-junit-reporter')`
	`7`	`+ reporterEnabled.push('./scripts/junit-reporter.js')`
`8`	`8`	`}`
`9`	`9`
`10`	`10`	`module.exports = {`
`11`	`11`	`reporterEnabled,`
`12`		`- mochaJunitReporterReporterOptions: {`
	`12`	`+ scriptsJunitReporterJsReporterOptions: {`
`13`	`13`	mochaFile: `./node-${process.versions.node}-junit.xml`,
`14`	`14`	`},`
`15`	`15`	`}`