fixup! fix(sandbox): fix illegal invocation errors in early node 10.x…

… releases
npm · Aug 25, 2021 · 585478a · 585478a
1 parent cce0701
commit 585478a
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/test/fixtures/sandbox.js b/test/fixtures/sandbox.js
@@ -275,6 +275,14 @@ class Sandbox extends EventEmitter {
     this[_npm] = this[_test].mock('../../lib/npm.js', this[_mocks])
     this[_npm].output = (...args) => this[_output].push(args)
     await this[_npm].load()
+    // in some node versions (later 10.x) our executionAsyncId at this point
+    // will for some reason appear to have been triggered by a different parent
+    // so immediately after load, if we can see that we lost our ancestry, we
+    // fix it here with a hammer
+    if (chain.get(executionAsyncId()) !== this[_parent]) {
+      chain.set(executionAsyncId(), this[_parent])
+      process = this[_proxy]
+    }
 
     const cmd = this[_npm].argv.shift()
     const impl = this[_npm].commands[cmd]
@@ -325,6 +333,14 @@ class Sandbox extends EventEmitter {
     this[_npm] = this[_test].mock('../../lib/npm.js', this[_mocks])
     this[_npm].output = (...args) => this[_output].push(args)
     await this[_npm].load()
+    // in some node versions (later 10.x) our executionAsyncId at this point
+    // will for some reason appear to have been triggered by a different parent
+    // so immediately after load, if we can see that we lost our ancestry, we
+    // fix it here with a hammer
+    if (chain.get(executionAsyncId()) !== this[_parent]) {
+      chain.set(executionAsyncId(), this[_parent])
+      process = this[_proxy]
+    }
 
     const impl = this[_npm].commands[command]
     if (!impl) {