Disable FinalizationRegistry if NODE_V8_COVERAGE is set (#2298)

Signed-off-by: Matteo Collina <[email protected]>
nodejs · Oct 1, 2023 · 5f190b1 · 5f190b1
1 parent 5addbb7
commit 5f190b1
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