fix access to process variable (#2201)

This commit is contained in:
Sasha 2025-01-16 13:32:40 +01:00 committed by GitHub
parent ef78e52c45
commit b5d2c013a1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 11 additions and 2 deletions

View File

@ -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;

8
packages/sdk/src/env.ts Normal file
View File

@ -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;
}
}