test: don't remove empty.txt on win32

on win32 we use empty.txt in the fixtures directory, otherwise we use a file constructed specifically for this test due to POSIX socket path length limitations, in which case we need to do appropriate cleanup Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> PR-URL: #2
nodejs · Nov 29, 2014 · 013527a · 013527a
1 parent d23326b
commit 013527a
Showing 1 changed file with 11 additions and 10 deletions.
diff --git a/test/simple/test-net-pipe-connect-errors.js b/test/simple/test-net-pipe-connect-errors.js
@@ -42,19 +42,20 @@ if (process.platform === 'win32') {
   // use common.PIPE to ensure we stay within POSIX socket path length
   // restrictions, even on CI
   emptyTxt = common.PIPE + '.txt';
-}
 
-function cleanup() {
-  try {
-    fs.unlinkSync(emptyTxt);
-  } catch (e) {
-    if (e.code != 'ENOENT')
-      throw e;
+  function cleanup() {
+    try {
+      fs.unlinkSync(emptyTxt);
+    } catch (e) {
+      if (e.code != 'ENOENT')
+        throw e;
+    }
   }
+  process.on('exit', cleanup);
+  cleanup();
+  fs.writeFileSync(emptyTxt, '');
 }
-process.on('exit', cleanup);
-cleanup();
-fs.writeFileSync(emptyTxt, '');
+
 var notSocketClient = net.createConnection(emptyTxt, function() {
   assert.ok(false);
 });