Skip to content

Commit

Permalink
test: don't use cwd for relative path
Browse files Browse the repository at this point in the history
With the introduction of temporary paths in the test runner
realpath tests would bail in scenarios where the temporary folder
wasn't in the same directory as the source code.

PR-URL: #4477
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
jbergstroem authored and Fishrock123 committed Jan 6, 2016
1 parent 5110e4d commit ba24450
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions test/parallel/test-fs-realpath.js
Original file line number Diff line number Diff line change
Expand Up @@ -232,11 +232,10 @@ function test_relative_input_cwd(callback) {
return runNextTest();
}

// we need to get the relative path to the tmp dir from cwd.
// When the test runner is running it, that will be .../node/test
// but it's more common to run `./node test/.../`, so detect it here.
// we need to calculate the relative path to the tmp dir from cwd
var entrydir = process.cwd();
var entry = common.tmpDir.substr(entrydir.length + 1) + '/cycles/realpath-3a';
var entry = path.relative(entrydir,
path.join(common.tmpDir + '/cycles/realpath-3a'));
var expected = common.tmpDir + '/cycles/root.js';
[
[entry, '../cycles/realpath-3b'],
Expand Down

0 comments on commit ba24450

Please sign in to comment.