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.
  • Loading branch information
diprudnikov authored and Trott committed Aug 18, 2018
1 parent 1744205 commit 9b2fbe2
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 9b2fbe2

Please sign in to comment.