From 0ebb811affc2480efb8d814d70fba854cf5122f5 Mon Sep 17 00:00:00 2001 From: Hyunwoo Gu Date: Fri, 5 Jun 2026 13:52:06 +0900 Subject: [PATCH] test: fix Linux debug skip in SEA test guard process.config.variables.is_debug is only populated by the GN build flow (tools/generate_config_gypi.py), so on the gyp builds used by CI the guard never fired and SEA tests ran on Linux debug builds against the original intent. Switch to process.config.target_defaults.default_configuration === 'Debug', matching the pattern used in test/common/index.js. Fixes: https://github.com/nodejs/node/issues/63749 Refs: https://github.com/nodejs/node/issues/61483 Signed-off-by: Hyunwoo Gu --- test/common/sea.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/common/sea.js b/test/common/sea.js index f7819c365db9d5..f2ce06f4e18145 100644 --- a/test/common/sea.js +++ b/test/common/sea.js @@ -27,7 +27,7 @@ function skipIfSingleExecutableIsNotSupported() { if (!['darwin', 'win32', 'linux'].includes(process.platform)) common.skip(`Unsupported platform ${process.platform}.`); - if (process.platform === 'linux' && process.config.variables.is_debug === 1) + if (process.platform === 'linux' && process.config.target_defaults?.default_configuration === 'Debug') common.skip('Running the resultant binary fails with `Couldn\'t read target executable"`.'); if (process.config.variables.node_shared)