test: confirm callback is invoked in fs test

Trott · MylesBorins · commit 4a3e08998412 · 2017-08-15T20:57:22.000-07:00
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>
diff --git a/test/parallel/test-fs-makeStatsCallback.js b/test/parallel/test-fs-makeStatsCallback.js
@@ -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());

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ function testMakeStatsCallback(cb) {`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`// Verify the case where a callback function is provided`
`16`		`-assert.doesNotThrow(testMakeStatsCallback(common.noop));`
	`16`	`+assert.doesNotThrow(testMakeStatsCallback(common.mustCall()));`
`17`	`17`
`18`	`18`	`// Passing undefined/nothing calls rethrow() internally`
`19`	`19`	`assert.doesNotThrow(testMakeStatsCallback());`