test: correct conditional secure heap flags test

codebytere · aduh95 · commit a15b795b3481 · 2025-11-13T22:44:19.000+01:00
PR-URL: #60385 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/parallel/test-process-env-allowed-flags-are-documented.js b/test/parallel/test-process-env-allowed-flags-are-documented.js
@@ -49,6 +49,8 @@ if (!hasOpenSSL3){
 documented.delete('--openssl-shared-config');
 }
 
+const isV8Sandboxed = process.config.variables.v8_enable_sandbox;
+
 // Filter out options that are conditionally present.
 const conditionalOpts = [
{
@@ -74,6 +76,9 @@ const conditionalOpts = [
 },{
 include: process.features.inspector,
 filter: (opt) => opt.startsWith('--inspect') || opt === '--debug-port'
+ },{
+ include: !isV8Sandboxed,
+ filter: (opt) => ['--secure-heap', '--secure-heap-min'].includes(opt)
 },
 ];
 documented.forEach((opt) =>{

-Original file line number
+Diff line change
 documented.delete('--openssl-shared-config');
+}
 +constisV8Sandboxed=process.config.variables.v8_enable_sandbox;
++
 // Filter out options that are conditionally present.
 constconditionalOpts=[
+{
 },{
 include: process.features.inspector,
 filter: (opt)=>opt.startsWith('--inspect')||opt==='--debug-port'
 +},{
 +include: !isV8Sandboxed,
 +filter: (opt)=>['--secure-heap','--secure-heap-min'].includes(opt)
 },
 ];
 documented.forEach((opt)=>{