Skip to content

Commit 9592a48

Browse files
FNJ5MylesBorins
authored andcommitted
test: use fixtures.path for cmd string building
PR-URL: #15982 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
1 parent 4594315 commit 9592a48

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/parallel/test-error-reporting.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@
2323
const common = require('../common');
2424
const assert = require('assert');
2525
const exec = require('child_process').exec;
26-
const path = require('path');
26+
const fixtures = require('../common/fixtures');
2727

2828
function errExec(script, callback) {
29-
const cmd = `"${process.argv[0]}" "${path.join(common.fixturesDir, script)}"`;
29+
const cmd = `"${process.argv[0]}" "${fixtures.path(script)}"`;
3030
return exec(cmd, function(err, stdout, stderr) {
3131
// There was some error
3232
assert.ok(err);

0 commit comments

Comments
 (0)