From 3554d4b049f34065c6a643c5cda3d88c70a4908e Mon Sep 17 00:00:00 2001 From: Jordan Harband Date: Mon, 25 May 2020 13:46:44 -0700 Subject: [PATCH] [eslint] remove useless regex escapes --- .eslintrc | 1 + lib/results.js | 2 +- lib/test.js | 4 ++-- test/common.js | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.eslintrc b/.eslintrc index 6e67d8ee..8ca1d024 100644 --- a/.eslintrc +++ b/.eslintrc @@ -19,6 +19,7 @@ "named": "never", }], "no-undef": "error", + "no-useless-escape": "error", "strict": "error", }, "overrides": [ diff --git a/lib/results.js b/lib/results.js index 8fbeb763..2be5b2f8 100644 --- a/lib/results.js +++ b/lib/results.js @@ -9,7 +9,7 @@ var inspect = require('object-inspect'); var bind = require('function-bind'); var has = require('has'); var regexpTest = bind.call(Function.call, RegExp.prototype.test); -var yamlIndicators = /\:|\-|\?/; +var yamlIndicators = /:|-|\?/; var nextTick = typeof setImmediate !== 'undefined' ? setImmediate : process.nextTick diff --git a/lib/test.js b/lib/test.js index 9012dc84..2ea55df8 100644 --- a/lib/test.js +++ b/lib/test.js @@ -301,8 +301,8 @@ Test.prototype._assert = function assert(ok, opts) { /((?:\/|[a-zA-Z]:\\)[^:\)]+:(\d+)(?::(\d+))?)\)?/ */ - var re = /^(?:[^\s]*\s*\bat\s+)(?:(.*)\s+\()?((?:\/|[a-zA-Z]:\\)[^:\)]+:(\d+)(?::(\d+))?)\)?$/; - var lineWithTokens = err[i].replace(process.cwd(), '/\$CWD').replace(__dirname, '/\$TEST'); + var re = /^(?:[^\s]*\s*\bat\s+)(?:(.*)\s+\()?((?:\/|[a-zA-Z]:\\)[^:)]+:(\d+)(?::(\d+))?)\)?$/; + var lineWithTokens = err[i].replace(process.cwd(), '/$CWD').replace(__dirname, '/$TEST'); var m = re.exec(lineWithTokens); if (!m) { diff --git a/test/common.js b/test/common.js index f960ee29..201b3df7 100644 --- a/test/common.js +++ b/test/common.js @@ -41,7 +41,7 @@ var stripChangingData = function (line) { var withoutPackageDir = withoutTestDir.replace(path.dirname(__dirname), '$TAPE'); var withoutPathSep = withoutPackageDir.replace(new RegExp('\\' + path.sep, 'g'), '/'); var withoutLineNumbers = withoutPathSep.replace(/:\d+:\d+/g, ':$LINE:$COL'); - var withoutNestedLineNumbers = withoutLineNumbers.replace(/, \:\$LINE:\$COL\)$/, ')'); + var withoutNestedLineNumbers = withoutLineNumbers.replace(/, :\$LINE:\$COL\)$/, ')'); return withoutNestedLineNumbers; };