test: fix failing test on node 18

dnlup · Dec 18, 2023 · 80c0c57 · 80c0c57
1 parent 749b00d
commit 80c0c57
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/lib/sampler.js b/lib/sampler.js
@@ -63,7 +63,13 @@ class Sampler extends EventEmitter {
       this.start()
     }
     if (instances.hasSubscribers) {
-      process.nextTick(() => instances.publish(this))
+      process.nextTick(() => {
+        try {
+        // On Node 18 the `hasSubscribers` check is not enough so this call fails
+        // with an uncaught exception if there are no subscribers.
+          instances.publish(this)
+        } catch (_) {}
+      })
     }
   }
 
@@ -121,7 +127,11 @@ class Sampler extends EventEmitter {
     this.emit('sample')
     this[kReset]()
     if (samples.hasSubscribers) {
-      samples.publish(this)
+      try {
+        // On Node 18 the `hasSubscribers` check is not enough so this call fails
+        // with an uncaught exception if there are no subscribers.
+        samples.publish(this)
+      } catch (_) {}
     }
   }