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 George Adams committed Aug 20, 2018
1 parent 28675b6 commit f1d3f97
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 @@ -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;
}
Expand Down

0 comments on commit f1d3f97

Please sign in to comment.