Commit 39a53a0
test: remove unneeded escaping in template strings
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>1 parent 127ed73 commit 39a53a0
5 files changed
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -53,7 +53,7 @@ function kill(child) { | |||
| 53 | 53 | process.on('exit', function() { | |
| 54 | 54 | for (const child of children) { | |
| 55 | 55 | const port = child.test.port; | |
| 56 | - const one = RegExp(`Debugger listening on (\\[::\\]|0\.0\.0\.0):${port}`); | ||
| 56 | + const one = RegExp(`Debugger listening on (\\[::\\]|0.0.0.0):${port}`); | ||
| 57 | 57 | const two = RegExp(`connecting to 127.0.0.1:${port}`); | |
| 58 | 58 | assert(one.test(child.test.stdout)); | |
| 59 | 59 | assert(two.test(child.test.stdout)); | |
| 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 | |
|---|---|---|---|
@@ -20,7 +20,7 @@ const base = require('./tick-processor-base.js'); | |||
| 20 | 20 | base.runTest({ | |
| 21 | 21 | pattern: /RunInDebugContext/, | |
| 22 | 22 | code: `function f() { | |
| 23 | - require(\'vm\').runInDebugContext(\'Debug\'); | ||
| 23 | + require('vm').runInDebugContext('Debug'); | ||
| 24 | 24 | setImmediate(function() { f(); }); | |
| 25 | 25 | }; | |
| 26 | 26 | f();` | |
| 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) { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -48,7 +48,7 @@ var msgBuf = Buffer.from(msg + '\n'); | |||
| 48 | 48 | ||
| 49 | 49 | // console.log ends every line with just '\n', even on Windows. | |
| 50 | 50 | ||
| 51 | - cmd = `"${process.execPath}" -e "console.log(\'${msg}\');"`; | ||
| 51 | + cmd = `"${process.execPath}" -e "console.log('${msg}');"`; | ||
| 52 | 52 | ||
| 53 | 53 | ret = execSync(cmd); | |
| 54 | 54 | ||
0 commit comments