Skip to content

Commit

Permalink
test: confirm callback is invoked in fs test
Browse files Browse the repository at this point in the history
Use common.mustCall() in test-fs-makeStatsCallback to confirm that the
callback is invoked.

PR-URL: #13132
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <mhdawson@ibm.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
Trott authored and MylesBorins committed Aug 16, 2017
1 parent 8b161e0 commit 4a3e089
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-fs-makeStatsCallback.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ function testMakeStatsCallback(cb) {
}

// Verify the case where a callback function is provided
assert.doesNotThrow(testMakeStatsCallback(common.noop));
assert.doesNotThrow(testMakeStatsCallback(common.mustCall()));

// Passing undefined/nothing calls rethrow() internally
assert.doesNotThrow(testMakeStatsCallback());
Expand Down

0 comments on commit 4a3e089

Please sign in to comment.