Permalink
Browse files

test: improve error message output

PR-URL: #18498
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information...
bshankar authored and BridgeAR committed Feb 1, 2018
1 parent 4ac7be9 commit 5ecd2e15b8725f67219b746a6291992dbef62f06
Showing with 14 additions and 8 deletions.
  1. +14 −8 test/addons-napi/test_general/test.js
@@ -63,22 +63,25 @@ let w = {};
test_general.wrap(w);
w = null;
global.gc();
assert.strictEqual(test_general.derefItemWasCalled(), true,
const derefItemWasCalled = test_general.derefItemWasCalled();
assert.strictEqual(derefItemWasCalled, true,
'deref_item() was called upon garbage collecting a ' +
'wrapped object');
'wrapped object. test_general.derefItemWasCalled() ' +
`returned ${derefItemWasCalled}`);
// Assert that wrapping twice fails.
const x = {};
test_general.wrap(x);
assert.throws(() => test_general.wrap(x), Error);
common.expectsError(() => test_general.wrap(x),
{ type: Error, message: 'Invalid argument' });
// Ensure that wrapping, removing the wrap, and then wrapping again works.
const y = {};
test_general.wrap(y);
test_general.removeWrap(y);
assert.doesNotThrow(() => test_general.wrap(y), Error,
'Wrapping twice succeeds if a remove_wrap()' +
' separates the instances');
// Wrapping twice succeeds if a remove_wrap() separates the instances
assert.doesNotThrow(() => test_general.wrap(y));
// Ensure that removing a wrap and garbage collecting does not fire the
// finalize callback.
@@ -87,8 +90,11 @@ test_general.testFinalizeWrap(z);
test_general.removeWrap(z);
z = null;
global.gc();
assert.strictEqual(test_general.finalizeWasCalled(), false,
'finalize callback was not called upon garbage collection');
const finalizeWasCalled = test_general.finalizeWasCalled();
assert.strictEqual(finalizeWasCalled, false,
'finalize callback was not called upon garbage collection.' +
' test_general.finalizeWasCalled() ' +
`returned ${finalizeWasCalled}`);
// test napi_adjust_external_memory
const adjustedValue = test_general.testAdjustExternalMemory();

0 comments on commit 5ecd2e1

Please sign in to comment.