From 33994d896af437badbe93d176115f7fb9ac1317a Mon Sep 17 00:00:00 2001 From: Dzmitry_Prudnikau Date: Fri, 17 Aug 2018 12:41:26 +0300 Subject: [PATCH] test: remove third argument from assert.strictEqual() `test/parallel/test-util-inspect.js` has a call to `assert.strictEqual()` that receives three arguments. The third argument is a string literal. Unfortunately, calling assert.strictEqual() this way means that if there is an AssertionError, the value of the variables pos and npos are not reported. This PR removes this argument. PR-URL: https://github.com/nodejs/node/pull/22371 Reviewed-By: James M Snell Reviewed-By: Luigi Pinca Reviewed-By: Trivikram Kamat Reviewed-By: Jon Moss Reviewed-By: Rich Trott Reviewed-By: George Adams --- test/parallel/test-util-inspect.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js index 78a632a0d55d21..d9d1f20f06f265 100644 --- a/test/parallel/test-util-inspect.js +++ b/test/parallel/test-util-inspect.js @@ -1031,7 +1031,7 @@ if (typeof Symbol !== 'undefined') { const npos = line.search(numRE); if (npos !== -1) { if (pos !== undefined) { - assert.strictEqual(pos, npos, 'container items not aligned'); + assert.strictEqual(pos, npos); } pos = npos; }