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 <fedor@indutny.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
PR-URL: #2

Author: rvagg

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);
 });