fix: remove worker_threads require

jestjs · Jul 21, 2020 · c562f9f · sunlaiman · Jul 25, 2020 · c562f9f
1 parent 0167abc
commit c562f9f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/packages/jest-worker/src/Farm.ts b/packages/jest-worker/src/Farm.ts
@@ -85,9 +85,9 @@ export default class Farm {
         const onEnd: OnEnd = (error: Error | null, result: unknown) => {
           customMessageListeners.clear();
           if (error) {
-            return reject(error);
+            reject(error);
           } else {
-            return resolve(result);
+            resolve(result);
           }
         };
 

diff --git a/packages/jest-worker/src/workers/messageParent.ts b/packages/jest-worker/src/workers/messageParent.ts
@@ -8,15 +8,12 @@
 import {isMainThread, parentPort} from 'worker_threads';
 import {PARENT_MESSAGE_CUSTOM} from '../types';
 
-const isWorkerThread = !isMainThread && parentPort;
-
 const messageParent = (
   message: unknown,
   parentProcess: NodeJS.Process = process,
 ): void => {
   try {
-    if (isWorkerThread) {
-      const {parentPort} = require('worker_threads');
+    if (!isMainThread && parentPort) {
       parentPort.postMessage([PARENT_MESSAGE_CUSTOM, message]);
     } else if (typeof parentProcess.send === 'function') {
       parentProcess.send([PARENT_MESSAGE_CUSTOM, message]);