test: make sure tmpdir is created before using it

Otherwise it throws ENOENT when the folder happens to be cleaned PR-URL: #25224 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · May 16, 2019 · d9aa19f · d9aa19f
1 parent b061a08
commit d9aa19f
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/test/parallel/test-child-process-spawnsync-args.js b/test/parallel/test-child-process-spawnsync-args.js
@@ -15,6 +15,8 @@ const tmpdir = require('../common/tmpdir');
 const command = common.isWindows ? 'cd' : 'pwd';
 const options = { cwd: tmpdir.path };
 
+tmpdir.refresh();
+
 if (common.isWindows) {
   // This test is not the case for Windows based systems
   // unless the `shell` options equals to `true`
@@ -31,12 +33,13 @@ const testCases = [
 const expectedResult = tmpdir.path.trim().toLowerCase();
 
 const results = testCases.map((testCase) => {
-  const { stdout, stderr } = spawnSync(
+  const { stdout, stderr, error } = spawnSync(
     command,
     testCase,
     options
   );
 
+  assert.ifError(error);
   assert.deepStrictEqual(stderr, Buffer.alloc(0));
 
   return stdout.toString().trim().toLowerCase();