1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: normalize location test-worker-process-cwd.js runs tests

Browse Source 
This change sets the process' directory to __dirname in order to
normalize where the test is ran.  This addresses the situation that
occurs when node is located in the root, and moving up a directory
results in the same directory.  In that case, an error was thrown
because the test interpreted this as a failed directory change.

Fixes: https://github.com/nodejs/node/issues/28193
PR-URL: https://github.com/nodejs/node/pull/28271
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit is contained in:
Samantha Sample 2019-06-17 15:00:42 -07:00 committed by Rich Trott
parent d5cf51dad5
commit 5ccce52113
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
test/parallel/test-worker-process-cwd.js
View File

@ -10,6 +10,7 @@ if (!process.env.HAS_STARTED_WORKER) {
if (!isMainThread) { if (!isMainThread) {
common.skip('This test can only run as main thread'); common.skip('This test can only run as main thread');
} }
process.chdir(__dirname);
const w = new Worker(__filename); const w = new Worker(__filename);
process.chdir('..'); process.chdir('..');
w.on('message', common.mustCall((message) => { w.on('message', common.mustCall((message) => {