★ wanayoo — archive 1999 https://github.com/nodejs/node/commit/38c9d2bfeaNouvelle recherche | Portail wanayoo
Skip to content
Permalink
Browse files
test: add missing tmpdir.refresh() in recently-added test
Without `tmpdir.refresh()`, the test fails in some situations. This was
missed because using `test.py` will almost always result in a leftover
tmpdir lying around that makes the `refresh()` not needed.

Refs: #24913 (comment)

PR-URL: #25098
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Signed-off-by: Beth Griggs <Bethany.Griggs@uk.ibm.com>
  • Loading branch information
Trott authored and BethGriggs committed Mar 14, 2019
1 parent a193a0f commit 38c9d2bfea5929a1e242e29e8c218b9cc3ae2aed
Showing with 2 additions and 0 deletions.
  1. +2 −0 test/parallel/test-child-process-spawn-args.js
@@ -12,6 +12,8 @@ const { spawn } = require('child_process');
const common = require('../common');
const tmpdir = require('../common/tmpdir');

tmpdir.refresh();

const command = common.isWindows ? 'cd' : 'pwd';
const options = { cwd: tmpdir.path };

0 comments on commit 38c9d2b

Please sign in to comment.