Permalink
Browse files

test: fix flaky test-resolve-async

PR-URL: #17957
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information...
apapirovski authored and BridgeAR committed Jan 3, 2018
1 parent b43a496 commit d72c78b19c5422197a44100201e771c0319e354e
Showing with 2 additions and 5 deletions.
  1. +2 −5 test/addons/callback-scope/test-resolve-async.js
@@ -5,9 +5,6 @@ const assert = require('assert');
const { testResolveAsync } = require(`./build/${common.buildType}/binding`);
let called = false;
testResolveAsync().then(common.mustCall(() => {
called = true;
}));
testResolveAsync().then(() => { called = true; });
setTimeout(common.mustCall(() => { assert(called); }),
common.platformTimeout(20));
setTimeout(() => { assert(called); }, common.platformTimeout(50));

0 comments on commit d72c78b

Please sign in to comment.