Skip to content

test: fix args passed to strictEqual #21584

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
aitchkhan wants to merge 1 commit into from
Closed

test: fix args passed to strictEqual #21584

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
test: fix args passed to strictEqual 
The third argument passed to asert.strictEqual() displays the message
passed as third argument and does not report the difference between
actual and expected if the test is failing.
Hence, the third argument has been removed.
  • Loading branch information
@aitchkhan
aitchkhan committed Jun 29, 2018
commit b50a6e6e8248346bbc8a16705049a42f0a0c5e06
2 changes: 1 addition & 1 deletion test/parallel/test-timers-immediate-queue.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,5 +50,5 @@ for (let i = 0; i < QUEUE; i++)

process.on('exit', function() {
console.log('hit', hit);
assert.strictEqual(hit, QUEUE, 'We ticked between the immediate queue');
assert.strictEqual(hit, QUEUE);
});