fix to replace some more escape chars

nodejs · Dec 7, 2022 · bbba3c7 · bbba3c7
1 parent 1fc39d5
commit bbba3c7
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/lib/internal/test_runner/tap_stream.js b/lib/internal/test_runner/tap_stream.js
@@ -131,9 +131,18 @@ class TapStream extends Readable {
 
 // In certain places, # and \ need to be escaped as \# and \\.
 function tapEscape(input) {
-  return StringPrototypeReplaceAll(StringPrototypeReplaceAll(
-    StringPrototypeReplaceAll(input, '\\', '\\\\'), '#', '\\#'
-  ), '\n', '\\n');
+  [{ escapeChar: '\\', tappedEscape: '\\\\' },
+   { escapeChar: '#', tappedEscape: '\\#' },
+   { escapeChar: '\n', tappedEscape: '\\n' },
+   { escapeChar: '\t', tappedEscape: '\\t' },
+   { escapeChar: '\r', tappedEscape: '\\r' },
+   { escapeChar: '\f', tappedEscape: '\\f' },
+   { escapeChar: '\b', tappedEscape: '\\b' },
+   { escapeChar: '\v', tappedEscape: '\\v' },
+  ].forEach(({ escapeChar, tappedEscape }) => {
+    input = StringPrototypeReplaceAll(input, escapeChar, tappedEscape);
+  });
+  return input;
 }
 
 function jsToYaml(indent, name, value) {