test: remove try/catch in common.isMainThread

Refactor common.isMainThread. PR-URL: #25249 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · May 16, 2019 · fcc03c1 · fcc03c1
1 parent 4e2ceba
commit fcc03c1
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/test/common/index.js b/test/common/index.js
@@ -40,12 +40,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