From e78037b38e0a66b0604af88c8458c905bed3e240 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 28 Dec 2018 08:55:38 -0800 Subject: [PATCH] test: remove try/catch in common.isMainThread Refactor common.isMainThread. --- test/common/index.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/test/common/index.js b/test/common/index.js index 5ac79f4998acc8..5d9a4b31e36534 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -39,12 +39,11 @@ const noop = () => {}; const hasCrypto = Boolean(process.versions.openssl); const isMainThread = (() => { - try { + if (require('module').builtinModules.includes('worker_threads')) { return require('worker_threads').isMainThread; - } catch { - // Worker module not enabled → only a single main thread exists. - return true; } + // Worker module not enabled → only a single main thread exists. + return true; })(); // Check for flags. Skip this for workers (both, the `cluster` module and