test: add env variable test for --run

PR-URL: #52811 Reviewed-By: Daniel Lemire <daniel@lemire.me> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · May 8, 2024 · 30a4248 · 30a4248
1 parent ad9c4bd
commit 30a4248
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 1 deletion.
diff --git a/test/fixtures/run-script/node_modules/.bin/path-env b/test/fixtures/run-script/node_modules/.bin/path-env
diff --git a/test/fixtures/run-script/node_modules/.bin/path-env.bat b/test/fixtures/run-script/node_modules/.bin/path-env.bat
diff --git a/test/fixtures/run-script/package.json b/test/fixtures/run-script/package.json
@@ -6,6 +6,8 @@
     "positional-args": "positional-args",
     "positional-args-windows": "positional-args.bat",
     "custom-env": "custom-env",
-    "custom-env-windows": "custom-env.bat"
+    "custom-env-windows": "custom-env.bat",
+    "path-env": "path-env",
+    "path-env-windows": "path-env.bat"
   }
 }
diff --git a/test/message/node_run_non_existent.out b/test/message/node_run_non_existent.out
@@ -8,3 +8,5 @@ Available scripts are:
   positional-args-windows: positional-args.bat
   custom-env: custom-env
   custom-env-windows: custom-env.bat
+  path-env: path-env
+  path-env-windows: path-env.bat
diff --git a/test/parallel/test-node-run.js b/test/parallel/test-node-run.js
@@ -64,4 +64,15 @@ describe('node run [command]', () => {
     assert.strictEqual(child.stderr, '');
     assert.strictEqual(child.code, 0);
   });
+
+  it('should set PATH environment variable to node_modules/.bin', async () => {
+    const child = await common.spawnPromisified(
+      process.execPath,
+      [ '--no-warnings', '--run', `path-env${envSuffix}`],
+      { cwd: fixtures.path('run-script') },
+    );
+    assert.ok(child.stdout.includes(fixtures.path('run-script/node_modules/.bin')));
+    assert.strictEqual(child.stderr, '');
+    assert.strictEqual(child.code, 0);
+  });
 });