fix access to process variable (#2201)

waku-org · Jan 16, 2025 · b5d2c01 · b5d2c01
1 parent ef78e52
commit b5d2c01
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/packages/sdk/src/create/libp2p.ts b/packages/sdk/src/create/libp2p.ts
@@ -17,6 +17,7 @@ import {
 import { derivePubsubTopicsFromNetworkConfig, Logger } from "@waku/utils";
 import { createLibp2p } from "libp2p";
 
+import { isTestEnvironment } from "../env.js";
 import {
   CreateWakuNodeOptions,
   DefaultPingMaxInboundStreams,
@@ -36,7 +37,7 @@ export async function defaultLibp2p(
   options?: Partial<CreateLibp2pOptions>,
   userAgent?: string
 ): Promise<Libp2p> {
-  if (!options?.hideWebSocketInfo && process?.env?.NODE_ENV !== "test") {
+  if (!options?.hideWebSocketInfo && !isTestEnvironment()) {
     /* eslint-disable no-console */
     console.info(
       "%cIgnore WebSocket connection failures",
@@ -54,7 +55,7 @@ export async function defaultLibp2p(
     : {};
 
   const filter =
-    options?.filterMultiaddrs === false || process?.env?.NODE_ENV === "test"
+    options?.filterMultiaddrs === false || isTestEnvironment()
       ? filterAll
       : wss;
 

diff --git a/packages/sdk/src/env.ts b/packages/sdk/src/env.ts
@@ -0,0 +1,8 @@
+export function isTestEnvironment(): boolean {
+  try {
+    return process?.env?.NODE_ENV === "test";
+  } catch (_e) {
+    // process variable is not defined in PROD environment
+    return false;
+  }
+}