test: wrap suppressed callbacks with common.mustCall

Author: DivyanshuX9

test/parallel/test-diagnostics-channel-suppression.js

@@ -12,9 +12,9 @@ const { AsyncLocalStorage } = require('async_hooks');
   const handler = common.mustNotCall();
   ch.subscribe(handler, { subscriberId: key });
 
-  suppressed(key, () => {
+  suppressed(key, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
   ch.unsubscribe(handler);
 }
@@ -28,9 +28,9 @@ const { AsyncLocalStorage } = require('async_hooks');
   ch.subscribe(optedHandler, { subscriberId: key });
   ch.subscribe(regularHandler); // no suppression
 
-  suppressed(key, () => {
+  suppressed(key, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
   ch.unsubscribe(optedHandler);
   ch.unsubscribe(regularHandler);
@@ -48,16 +48,16 @@ const { AsyncLocalStorage } = require('async_hooks');
   ch.subscribe(h1, { subscriberId: k1 });
   ch.subscribe(h2, { subscriberId: k2 });
 
-  suppressed(k1, () => {
+  suppressed(k1, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
   assert.strictEqual(h1Calls, 0);
   assert.strictEqual(h2Calls, 1);
 
-  suppressed(k2, () => {
+  suppressed(k2, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
   assert.strictEqual(h1Calls, 1);
   assert.strictEqual(h2Calls, 1);
@@ -104,10 +104,10 @@ const { AsyncLocalStorage } = require('async_hooks');
   const handler = common.mustNotCall();
   ch.subscribe(handler, { subscriberId: key });
 
-  suppressed(key, async () => {
+  suppressed(key, common.mustCall(async () => {
     await Promise.resolve();
     ch.publish({});
-  }).then(common.mustCall(() => {
+  })).then(common.mustCall(() => {
     ch.unsubscribe(handler);
   }));
 }
@@ -119,18 +119,18 @@ const { AsyncLocalStorage } = require('async_hooks');
   const handler = common.mustNotCall();
   ch.subscribe(handler, { subscriberId: key });
 
-  suppressed(key, async () => {
+  suppressed(key, common.mustCall(async () => {
     await new Promise((resolve) => {
-      setImmediate(() => {
+      setImmediate(common.mustCall(() => {
         ch.publish({});
 
-        queueMicrotask(() => {
+        queueMicrotask(common.mustCall(() => {
           ch.publish({});
           resolve();
-        });
-      });
+        }));
+      }));
     });
-  }).then(common.mustCall(() => {
+  })).then(common.mustCall(() => {
     ch.unsubscribe(handler);
   }));
 }
@@ -144,9 +144,9 @@ const { AsyncLocalStorage } = require('async_hooks');
   ch.unsubscribe(handler);
 
   // Should not throw and should not be called
-  suppressed(key, () => {
+  suppressed(key, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
 }
 
@@ -163,9 +163,9 @@ const { AsyncLocalStorage } = require('async_hooks');
   ch.subscribe(handler);
   ch.bindStore(als, common.mustNotCall(), { subscriberId: key });
 
-  suppressed(key, () => {
+  suppressed(key, common.mustCall(() => {
     ch.publish({});
-  });
+  }));
 
   ch.unsubscribe(handler);
   ch.unbindStore(als);