From 9c2b14013bfa9e44dcb38602b2b57e4b8698b7b4 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Fri, 28 Dec 2018 08:55:38 -0800
Subject: [PATCH] test: remove try/catch in common.isMainThread

Refactor common.isMainThread.

PR-URL: https://github.com/nodejs/node/pull/25249
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
 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 5ac79f4998a..5d9a4b31e36 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