no longer detecting enumerable when add domain

Considering we drop support for Node 8, this compromise can be removed. jestjs#9136
mapleeit · Nov 24, 2020 · 0c8a549 · 0c8a549
1 parent ff2b65c
commit 0c8a549
Showing 1 changed file with 6 additions and 12 deletions.
diff --git a/packages/jest-util/src/createProcessObject.ts b/packages/jest-util/src/createProcessObject.ts
@@ -111,18 +111,12 @@ export default function (): NodeJS.Process {
 
   newProcess.env = createProcessEnv();
   newProcess.send = () => {};
-
-  const domainPropertyDescriptor = Object.getOwnPropertyDescriptor(
-    newProcess,
-    'domain',
-  );
-  if (domainPropertyDescriptor && !domainPropertyDescriptor.enumerable) {
-    Object.defineProperty(newProcess, 'domain', {
-      get() {
-        return process.domain;
-      },
-    });
-  }
+
+  Object.defineProperty(newProcess, 'domain', {
+    get() {
+      return process.domain;
+    },
+  });
 
   return newProcess;
 }