Skip to content

Commit 207b1c9

Browse files
committed
test: remove messages for assert
1 parent 8eef52e commit 207b1c9

File tree

1 file changed

+17
-14
lines changed
  • test/addons-napi/test_promise

1 file changed

+17
-14
lines changed

test/addons-napi/test_promise/test.js

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,18 @@
11
'use strict';
22

33
const common = require('../../common');
4-
const test_promise = require(`./build/${common.buildType}/test_promise`);
54
const assert = require('assert');
65

6+
// Testing api calls for promises
7+
const test_promise = require(`./build/${common.buildType}/test_promise`);
8+
79
// A resolution
810
{
911
const expected_result = 42;
1012
const promise = test_promise.createPromise();
1113
promise.then(
1214
common.mustCall(function(result) {
13-
assert.strictEqual(result, expected_result,
14-
`promise resolved as expected, received ${result}`);
15+
assert.strictEqual(result, expected_result);
1516
}),
1617
common.mustNotCall());
1718
test_promise.concludeCurrentPromise(expected_result, true);
@@ -24,23 +25,25 @@ const assert = require('assert');
2425
promise.then(
2526
common.mustNotCall(),
2627
common.mustCall(function(result) {
27-
assert.strictEqual(result, expected_result,
28-
`promise rejected as expected, received ${result}`);
28+
assert.strictEqual(result, expected_result);
2929
}));
3030
test_promise.concludeCurrentPromise(expected_result, false);
3131
}
3232

3333
// Chaining
34-
const promise = test_promise.createPromise();
35-
promise.then(
36-
common.mustCall(function(result) {
37-
assert.strictEqual(result, 'chained answer',
38-
'resolving with a promise chains properly');
39-
}),
40-
common.mustNotCall());
41-
test_promise.concludeCurrentPromise(Promise.resolve('chained answer'), true);
34+
{
35+
const expected_result = 'chained answer';
36+
const promise = test_promise.createPromise();
37+
promise.then(
38+
common.mustCall(function(result) {
39+
assert.strictEqual(result, expected_result);
40+
}),
41+
common.mustNotCall());
42+
test_promise.concludeCurrentPromise(Promise.resolve('chained answer'), true);
43+
44+
assert.strictEqual(test_promise.isPromise(promise), true);
45+
}
4246

43-
assert.strictEqual(test_promise.isPromise(promise), true);
4447
assert.strictEqual(test_promise.isPromise(Promise.reject(-1)), true);
4548
assert.strictEqual(test_promise.isPromise(2.4), false);
4649
assert.strictEqual(test_promise.isPromise('I promise!'), false);

0 commit comments

Comments
 (0)