fix: improve test name escaping in TAP reporters (#1367)

vitest-dev · May 24, 2022 · f8db7d7 · f8db7d7
1 parent cc17448
commit f8db7d7
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/vitest/src/node/reporters/tap.ts b/packages/vitest/src/node/reporters/tap.ts
@@ -8,9 +8,10 @@ function yamlString(str: string): string {
 }
 
 function tapString(str: string): string {
-  // Test name cannot contain #
-  // Test name cannot start with number
-  return str.replace(/#/g, '?').replace(/^[0-9]+/, '?')
+  return str
+    .replace(/\\/g, '\\\\') // escape slashes
+    .replace(/#/g, '\\#') // escape #
+    .replace(/\n/g, ' ') // remove newlines
 }
 
 export class TapReporter implements Reporter {