reduce test timeouts on appveyor since signals have no impact

raszi · Mar 13, 2019 · 2e5fed9 · 2e5fed9
1 parent 69b1115
commit 2e5fed9
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/lib/tmp.js b/lib/tmp.js
@@ -605,12 +605,12 @@ function _safely_install_listener() {
     rl.on('SIGINT', function () {
       process.emit('SIGINT');
     });
+  } else {
+    process.on('SIGINT', function () {
+      process.exit(0);
+    });
   }
 
-  process.on('SIGINT', function () {
-    process.exit(0);
-  });
-
   process.addListener(EVENT, function _tmp$safe_listener(data) {
     /* istanbul ignore else */
     if (existingListeners.length) {

diff --git a/test/child-process.js b/test/child-process.js
@@ -48,8 +48,6 @@ function _doSpawn(commandArgs, cb) {
   child = spawn(node_path, commandArgs);
   child.stdin.end();
 
-  // TODO we no longer support node 0.6
-  // Cannot use 'close' event because not on node-0.6.
   function _close() {
     var
       stderr = _bufferConcat(stderrBufs).toString(),

diff --git a/test/outband/issue121.js b/test/outband/issue121.js
@@ -11,6 +11,7 @@ tmp.setGracefulCleanup();
 
 // https://github.com/raszi/node-tmp/issues/121
 module.exports = function (signal) {
+
   try {
     fixture.apply(this, [tmp.dirSync({ unsafeCleanup: true }), tmp]);
   }
@@ -20,7 +21,8 @@ module.exports = function (signal) {
   }
 
   // make sure that the process keeps running
-  setTimeout(function () {}, 1000000);
+  setTimeout(function () {}, 10000);
 
+  // we will now end this process, triggering tmp's exit handler
   this.kill(signal);
 };