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
Trott · Jul 23, 2018 · d6f37e6 · d6f37e6
1 parent c57e11c
commit d6f37e6
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'));