fixup! worker: allow copied NODE_OPTIONS in the env setting

nodejs · Jul 4, 2024 · 396ccd9 · 396ccd9
1 parent 780ade2
commit 396ccd9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/test/fixtures/spawn-worker-with-copied-env.js b/test/fixtures/spawn-worker-with-copied-env.js
@@ -2,7 +2,9 @@
 
 // This test is meant to be spawned with NODE_OPTIONS=--title=foo
 const assert = require('assert');
-assert.strictEqual(process.title, 'foo');
+if (process.platform !== 'sunos') {  // --title is unsupported on SmartOS.
+  assert.strictEqual(process.title, 'foo');
+}
 
 // Spawns a worker that may copy NODE_OPTIONS if it's set by the parent.
 const { Worker } = require('worker_threads');

diff --git a/test/parallel/test-worker-node-options.js b/test/parallel/test-worker-node-options.js
@@ -30,6 +30,6 @@ spawnSyncAndAssert(
     }
   },
   {
-    stderr: /spawn-worker-with-trace-exit.js:17/
+    stderr: /spawn-worker-with-trace-exit\.js:17/
   }
 );
-Original file line number
+Diff line change
@@ Expand Up / @@ -30,6 +30,6 @@ spawnSyncAndAssert( @@
         }
       },
       {
-        stderr: /spawn-worker-with-trace-exit.js:17/
+        stderr: /spawn-worker-with-trace-exit\.js:17/
       }
     );