From b85baf738eb806edc0e4f7603084df490720330f Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Sun, 2 Sep 2018 20:15:51 +0200 Subject: [PATCH] test: fix flaky addons/callback-scope/test-resolve-async Fixes: https://github.com/nodejs/node/issues/22668 --- test/addons/callback-scope/test-resolve-async.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/addons/callback-scope/test-resolve-async.js b/test/addons/callback-scope/test-resolve-async.js index 98e1910b49df7b..c2942397737359 100644 --- a/test/addons/callback-scope/test-resolve-async.js +++ b/test/addons/callback-scope/test-resolve-async.js @@ -4,7 +4,9 @@ const common = require('../../common'); const assert = require('assert'); const { testResolveAsync } = require(`./build/${common.buildType}/binding`); +// Checks that resolving promises from C++ works. + let called = false; testResolveAsync().then(() => { called = true; }); -setTimeout(() => { assert(called); }, common.platformTimeout(50)); +process.on('beforeExit', () => { assert(called); });