Skip to content

Commit b73ec84

Browse files
mmaleckiisaacs
authored andcommitted
test: test for process.argv[0] correctness
nodejs/node-v0.x-archive@b0c1541 introduced a regression causing `process.argv[0]` to be invalid in node processes spawned from `PATH` (without explicit path to executable file - for example when using global node installation). Instead of finding a correct path to the executable, `process.cwd()` would be prepended to `process.argv[0]`.
1 parent a84adad commit b73ec84

File tree

1 file changed

+43
-0
lines changed

1 file changed

+43
-0
lines changed

test/simple/test-process-argv-0.js

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
// Copyright Joyent, Inc. and other Node contributors.
2+
//
3+
// Permission is hereby granted, free of charge, to any person obtaining a
4+
// copy of this software and associated documentation files (the
5+
// "Software"), to deal in the Software without restriction, including
6+
// without limitation the rights to use, copy, modify, merge, publish,
7+
// distribute, sublicense, and/or sell copies of the Software, and to permit
8+
// persons to whom the Software is furnished to do so, subject to the
9+
// following conditions:
10+
//
11+
// The above copyright notice and this permission notice shall be included
12+
// in all copies or substantial portions of the Software.
13+
//
14+
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
15+
// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
16+
// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
17+
// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
18+
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
19+
// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
20+
// USE OR OTHER DEALINGS IN THE SOFTWARE.
21+
var util = require('util');
22+
var path = require('path');
23+
var assert = require('assert');
24+
var spawn = require('child_process').spawn;
25+
var common = require('../common');
26+
27+
if (process.argv[2] !== "child") {
28+
var child = spawn('node', [__filename, "child"], {
29+
cwd: common.tmpDir,
30+
env: util._extend({ PATH: path.dirname(process.execPath) }, process.env)
31+
});
32+
33+
var childArgv0 = '';
34+
child.stdout.on('data', function (chunk) {
35+
childArgv0 += chunk;
36+
});
37+
child.on('exit', function () {
38+
assert.equal(childArgv0, process.execPath);
39+
});
40+
}
41+
else {
42+
process.stdout.write(process.argv[0]);
43+
}

0 commit comments

Comments
 (0)