Disable FinalizationRegistry if NODE_V8_COVERAGE is set (nodejs#2298)

Signed-off-by: Matteo Collina <hello@matteocollina.com>
crysmags · Feb 27, 2024 · 623b14d · 623b14d
1 parent 7193068
commit 623b14d
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/lib/compat/dispatcher-weakref.js b/lib/compat/dispatcher-weakref.js
@@ -31,6 +31,14 @@ class CompatFinalizer {
 }
 
 module.exports = function () {
+  // FIXME: remove workaround when the Node bug is fixed
+  // https://github.com/nodejs/node/issues/49344#issuecomment-1741776308
+  if (process.env.NODE_V8_COVERAGE) {
+    return {
+      WeakRef: CompatWeakRef,
+      FinalizationRegistry: CompatFinalizer
+    }
+  }
   return {
     WeakRef: global.WeakRef || CompatWeakRef,
     FinalizationRegistry: global.FinalizationRegistry || CompatFinalizer