Skip to content

Commit

Permalink
test: fix flaky test-resolve-async
Browse files Browse the repository at this point in the history
PR-URL: #17957
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
  • Loading branch information
apapirovski authored and MylesBorins committed Jan 9, 2018
1 parent 63c23a1 commit 3cc9882
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions test/addons/callback-scope/test-resolve-async.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 3cc9882

Please sign in to comment.