From 0c8a5490639dc927b51c6ceef1498ceab9d90414 Mon Sep 17 00:00:00 2001 From: Maple Miao <522856232@qq.com> Date: Tue, 24 Nov 2020 14:42:14 +0800 Subject: [PATCH] no longer detecting enumerable when add domain Considering we drop support for Node 8, this compromise can be removed. https://github.com/facebook/jest/pull/9136 --- packages/jest-util/src/createProcessObject.ts | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/packages/jest-util/src/createProcessObject.ts b/packages/jest-util/src/createProcessObject.ts index 569977a339c1..6f996ee92dc3 100644 --- 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; }