test: fix flaky async-hooks/test-graph.signal

Make sure event loop remains open long enough for signal to be received. PR-URL: #17509 Fixes: #14568 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Evan Lucas <[email protected]>
nodejs · Dec 12, 2017 · f5a1e6c · f5a1e6c
1 parent a333e71
commit f5a1e6c
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/test/async-hooks/test-graph.signal.js b/test/async-hooks/test-graph.signal.js
@@ -3,16 +3,16 @@
 const common = require('../common');
 if (common.isWindows) {
   common.skip('no signals on Windows');
-  return;
 }
 
 const initHooks = require('./init-hooks');
 const verifyGraph = require('./verify-graph');
-const exec = require('child_process').exec;
+const { exec } = require('child_process');
 
 const hooks = initHooks();
 
 hooks.enable();
+const interval = setInterval(() => {}, 9999); // keep event loop open
 process.on('SIGUSR2', common.mustCall(onsigusr2, 2));
 
 let count = 0;
@@ -33,7 +33,9 @@ function onsigusr2() {
   }
 }
 
-function onsigusr2Again() {}
+function onsigusr2Again() {
+  clearInterval(interval); // let the event loop close
+}
 
 process.on('exit', onexit);