Skip to content

Commit

Permalink
test: remove third argument from assert.strictEqual()
Browse files Browse the repository at this point in the history
`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: #22371
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Jon Moss <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: George Adams <[email protected]>
  • Loading branch information
diprudnikov authored and targos committed Sep 3, 2018
1 parent 660c515 commit 33994d8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-util-inspect.js
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit 33994d8

Please sign in to comment.