test: fix flaky async-hooks/test-tlswrap

There is a race condition in async-hooks/test-tlswrap. This addresses it by waiting 5 more ticks if the client has not been destroyed yet. PR-URL: nodejs#15744 Fixes: nodejs#14404 Reviewed-By: Refael Ackermann <[email protected]>
Trott · Oct 5, 2017 · ad652af · ad652af
1 parent ddbf07a
commit ad652af
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/test/async-hooks/test-tlswrap.js b/test/async-hooks/test-tlswrap.js
@@ -91,6 +91,10 @@ function onsecureConnect() {
     // TODO: why is client not destroyed here even after 5 ticks?
     // or could it be that it isn't actually destroyed until
     // the server is closed?
+    if (client.before.length < 3) {
+      tick(5, tick1);
+      return;
+    }
     checkInvocations(client, { init: 1, before: 3, after: 3 },
                      'client: when client destroyed');
     //