test: address flaky worker test

Make test/parallel/test-worker-message-port-transfer-closed.js more reliable by counting ticks rather than using a single setTimeout(). Fixes: nodejs#21892 PR-URL: nodejs#21893 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]>
Trott · Jul 24, 2018 · f93a19b · f93a19b
1 parent 9453240
commit f93a19b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/test/parallel/test-worker-message-port-transfer-closed.js b/test/parallel/test-worker-message-port-transfer-closed.js
@@ -50,5 +50,9 @@ testSingle(port1, port2);
 port2.close(common.mustCall(testBothClosed));
 testBothClosed();
 
-setTimeout(common.mustNotCall('The communication channel is still open'),
-           common.platformTimeout(1000)).unref();
+function tickUnref(n, fn) {
+  if (n === 0) return fn();
+  setImmediate(tickUnref, n - 1, fn).unref();
+}
+
+tickUnref(10, common.mustNotCall('The communication channel is still open'));