test: add common.mustNotCall()

This commit adds a mustNotCall() helper for testing. This provides
an alternative to using common.fail() as a callback, or creating
a callback function for the sole purpose of calling common.fail().

PR-URL: https://github.com/nodejs/node/pull/11152
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

1 files changed, 1 insertions, 3 deletions

diff --git a/test/parallel/test-timers-same-timeout-wrong-list-deleted.js b/test/parallel/test-timers-same-timeout-wrong-list-deleted.js
index 8a622b32e4..c66ba0a57e 100644
--- a/test/parallel/test-timers-same-timeout-wrong-list-deleted.js
+++ b/test/parallel/test-timers-same-timeout-wrong-list-deleted.js
@@ -22,9 +22,7 @@ const handle1 = setTimeout(common.mustCall(function() {
   clearTimeout(handle1);
 
   // Cause a new list with the same key (TIMEOUT) to be created for this timer
-  const handle2 = setTimeout(function() {
-    common.fail('Inner callback is not called');
-  }, TIMEOUT);
+  const handle2 = setTimeout(common.mustNotCall(), TIMEOUT);
 
   setTimeout(common.mustCall(function() {
     // Attempt to cancel the second timer. Fix for this bug will keep the