1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: remove error messages for readability

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/16022
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
Fadi Asfour 2017-10-06 12:05:11 -07:00 committed by Joyee Cheung
parent eac257f600
commit 5ea88b7496
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
test/addons/async-hooks-promise/test.js
View File

@ -13,16 +13,14 @@ if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) {
// Baseline to make sure the internal field isn't being set. // Baseline to make sure the internal field isn't being set.
assert.strictEqual( assert.strictEqual(
binding.getPromiseField(Promise.resolve(1)), binding.getPromiseField(Promise.resolve(1)),
0, 0);
'Promise internal field used despite missing enabled AsyncHook');
const hook0 = async_hooks.createHook({}).enable(); const hook0 = async_hooks.createHook({}).enable();
// Check that no PromiseWrap is created when there are no hook callbacks. // Check that no PromiseWrap is created when there are no hook callbacks.
assert.strictEqual( assert.strictEqual(
binding.getPromiseField(Promise.resolve(1)), binding.getPromiseField(Promise.resolve(1)),
0, 0);
'Promise internal field used despite missing enabled AsyncHook');
hook0.disable(); hook0.disable();
@ -47,6 +45,5 @@ hook1.disable();
setImmediate(() => { setImmediate(() => {
assert.strictEqual( assert.strictEqual(
binding.getPromiseField(Promise.resolve(1)), binding.getPromiseField(Promise.resolve(1)),
0, 0);
'Promise internal field used despite missing enabled AsyncHook');
}); });