test: only refresh tmpDir for tests that need it
authorRich Trott <rtrott@gmail.com>
Tue, 9 Jun 2015 18:40:55 +0000 (11:40 -0700)
committerRich Trott <rtrott@gmail.com>
Sun, 14 Jun 2015 05:27:17 +0000 (22:27 -0700)
commit7c79490bfbfee3868c1524207fcd11e0149b9c73
tree2b93b80b6921c0f81d36efcd51c2524d875c6669
parent88d7904c0ba66bd2ced87f674d3f6ee098db970c
test: only refresh tmpDir for tests that need it

Expose `common.refreshTmpDir()` and only call it
for tests that use common.tmpDir or common.PIPE.

A positive side effect is the removal of a code
smell where child processes were detected by the
presence of `.send()`. Now each process can decide
for itself if it needs to refresh tmpDir.

PR-URL: https://github.com/nodejs/io.js/pull/1954
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
53 files changed:
.eslintignore
test/common.js
test/fixtures/listen-on-socket-and-exit.js
test/parallel/test-child-process-fork-exec-path.js
test/parallel/test-cluster-http-pipe.js
test/parallel/test-cwd-enoent-repl.js
test/parallel/test-cwd-enoent.js
test/parallel/test-file-write-stream.js
test/parallel/test-file-write-stream2.js
test/parallel/test-file-write-stream3.js
test/parallel/test-fs-access.js
test/parallel/test-fs-append-file-sync.js
test/parallel/test-fs-append-file.js
test/parallel/test-fs-chmod.js
test/parallel/test-fs-long-path.js
test/parallel/test-fs-mkdir.js
test/parallel/test-fs-read-stream-fd.js
test/parallel/test-fs-readfile-pipe-large.js
test/parallel/test-fs-readfilesync-pipe-large.js
test/parallel/test-fs-realpath.js
test/parallel/test-fs-sir-writes-alot.js
test/parallel/test-fs-stream-double-close.js
test/parallel/test-fs-symlink-dir-junction-relative.js
test/parallel/test-fs-symlink-dir-junction.js
test/parallel/test-fs-symlink.js
test/parallel/test-fs-truncate-GH-6233.js
test/parallel/test-fs-truncate-fd.js
test/parallel/test-fs-truncate.js
test/parallel/test-fs-write-buffer.js
test/parallel/test-fs-write-file-buffer.js
test/parallel/test-fs-write-file-sync.js
test/parallel/test-fs-write-file.js
test/parallel/test-fs-write-stream-change-open.js
test/parallel/test-fs-write-stream-encoding.js
test/parallel/test-fs-write-stream-end.js
test/parallel/test-fs-write-stream-err.js
test/parallel/test-fs-write-stream-throw-type-error.js
test/parallel/test-fs-write-stream.js
test/parallel/test-fs-write-string-coerce.js
test/parallel/test-fs-write-sync.js
test/parallel/test-fs-write.js
test/parallel/test-http-curl-chunk-problem.js
test/parallel/test-http-get-pipeline-problem.js
test/parallel/test-http-pipe-fs.js
test/parallel/test-https-truncate.js
test/parallel/test-pipe-file-to-http.js
test/parallel/test-repl-.save.load.js
test/parallel/test-zlib-from-gzip.js
test/sequential/test-fs-watch-recursive.js
test/sequential/test-fs-watch.js
test/sequential/test-mkdir-rmdir.js
test/sequential/test-regress-GH-4027.js
test/sequential/test-stdout-to-file.js