From f1d3f97c3bc813528e85b9c3e6506fc75b931d92 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 433b16c39e7..4240ff6d923 100644 --- a/test/parallel/test-util-inspect.js +++ b/test/parallel/test-util-inspect.js @@ -979,7 +979,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; }