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>

1 files changed, 9 insertions, 13 deletions

diff --git a/test/common.js b/test/common.js
index e88893cd1..704c64baf 100644
--- a/test/common.js
+++ b/test/common.js
@@ -51,19 +51,17 @@ function rmdirSync(p, originalEr) {
   }
 }
 
-function refreshTmpDir() {
-  if (!process.send) { // Not a child process
-    try {
-      rimrafSync(exports.tmpDir);
-    } catch (e) {
-    }
+exports.refreshTmpDir = function() {
+  try {
+    rimrafSync(exports.tmpDir);
+  } catch (e) {
+  }
 
-    try {
-      fs.mkdirSync(exports.tmpDir);
-    } catch (e) {
-    }
+  try {
+    fs.mkdirSync(exports.tmpDir);
+  } catch (e) {
   }
-}
+};
 
 if (process.env.TEST_THREAD_ID) {
   // Distribute ports in parallel tests
@@ -74,8 +72,6 @@ if (process.env.TEST_THREAD_ID) {
 }
 exports.tmpDir = path.join(exports.testDir, exports.tmpDirName);
 
-refreshTmpDir();
-
 var opensslCli = null;
 var inFreeBSDJail = null;
 var localhostIPv4 = null;