test: remove unneeded escaping in template strings

Trott · Myles Borins · commit 39a53a0f29d9 · 2016-11-11T15:46:24.000-05:00
The no-useless-escape rule in ESLint did not previously flag certain unnecessary escaping in template strings. These will be flagged in ESLint 3.8.0. PR-URL: #9112 Reviewed-By: Teddy Katz <teddy.katz@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
diff --git a/test/parallel/test-debug-port-numbers.js b/test/parallel/test-debug-port-numbers.js
@@ -53,7 +53,7 @@ function kill(child) {
 process.on('exit', function() {
   for (const child of children) {
     const port = child.test.port;
-    const one = RegExp(`Debugger listening on (\\[::\\]|0\.0\.0\.0):${port}`);
+    const one = RegExp(`Debugger listening on (\\[::\\]|0.0.0.0):${port}`);
     const two = RegExp(`connecting to 127.0.0.1:${port}`);
     assert(one.test(child.test.stdout));
     assert(two.test(child.test.stdout));
diff --git a/test/parallel/test-debugger-pid.js b/test/parallel/test-debugger-pid.js
@@ -43,7 +43,7 @@ interfacer.on('line', function(line) {
     }
   } else {
     line = line.replace(/^(debug> *)+/, '');
-    expected = `(node:${pid}) Target process: 655555 doesn\'t exist.`;
+    expected = `(node:${pid}) Target process: 655555 doesn't exist.`;
   }
 
   assert.strictEqual(expected, line);
diff --git a/test/parallel/test-tick-processor-cpp-core.js b/test/parallel/test-tick-processor-cpp-core.js
@@ -20,7 +20,7 @@ const base = require('./tick-processor-base.js');
 base.runTest({
   pattern: /RunInDebugContext/,
   code: `function f() {
-           require(\'vm\').runInDebugContext(\'Debug\');
+           require('vm').runInDebugContext('Debug');
            setImmediate(function() { f(); });
          };
          f();`
diff --git a/test/parallel/test-vm-cached-data.js b/test/parallel/test-vm-cached-data.js
@@ -6,7 +6,7 @@ const spawnSync = require('child_process').spawnSync;
 const Buffer = require('buffer').Buffer;
 
 function getSource(tag) {
-  return `(function ${tag}() { return \'${tag}\'; })`;
+  return `(function ${tag}() { return '${tag}'; })`;
 }
 
 function produce(source, count) {
diff --git a/test/sequential/test-child-process-execsync.js b/test/sequential/test-child-process-execsync.js
@@ -48,7 +48,7 @@ var msgBuf = Buffer.from(msg + '\n');
 
 // console.log ends every line with just '\n', even on Windows.
 
-cmd = `"${process.execPath}" -e "console.log(\'${msg}\');"`;
+cmd = `"${process.execPath}" -e "console.log('${msg}');"`;
 
 ret = execSync(cmd);
 

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ interfacer.on('line', function(line) {`
`43`	`43`	`}`
`44`	`44`	`} else {`
`45`	`45`	`line = line.replace(/^(debug> *)+/, '');`
`46`		- expected = `(node:${pid}) Target process: 655555 doesn\'t exist.`;
	`46`	+ expected = `(node:${pid}) Target process: 655555 doesn't exist.`;
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`assert.strictEqual(expected, line);`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ const spawnSync = require('child_process').spawnSync;`
`6`	`6`	`const Buffer = require('buffer').Buffer;`
`7`	`7`
`8`	`8`	`function getSource(tag) {`
`9`		- return `(function ${tag}() { return \'${tag}\'; })`;
	`9`	+ return `(function ${tag}() { return '${tag}'; })`;
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`function produce(source, count) {`