Skip to content
Closed
Changes from 1 commit
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
e304c42
test: add unhandled rejection guard
babygoat Nov 23, 2017
887dcb9
test: remove literals that obscure assert messages
Trott Dec 13, 2017
bba1442
n-api: fix memory leak in napi_async_destroy()
Dec 17, 2017
bcbb7af
test: remove ambiguous error messages from test_error
nadrane Dec 21, 2017
6bf3769
doc: updates examples to use NULL
mhdawson Jan 5, 2018
f81330f
n-api: throw RangeError in napi_create_dataview() with invalid range
romandev Dec 26, 2017
6d23c24
n-api: expose n-api version in process.versions
mhdawson Jan 9, 2018
40ce083
doc: napi: fix unbalanced emphasis
ofrobots Jan 12, 2018
621117e
doc: napi: make header style consistent
ofrobots Jan 12, 2018
111678b
n-api: throw RangeError napi_create_typedarray()
romandev Jan 8, 2018
94618a5
test: fixed typos in napi test
furstenheim Jan 14, 2018
375d043
doc: remove uannecessary Require
mhdawson Jan 16, 2018
f7fb0c3
timers: allow Immediates to be unrefed
apapirovski Jan 13, 2018
23fd460
test: refactor addons-napi/test_exception/test.js
Trott Jan 24, 2018
021e4a4
n-api: change assert ok check to notStrictEqual.
nbdaaron Jan 27, 2018
bf96235
test: show pending exception error in napi tests
blairwilcox Jan 27, 2018
17862cd
n-api: implement wrapping using private properties
Jan 23, 2018
29bbf0c
n-api: wrap control flow macro in do/while
bnoordhuis Feb 2, 2018
6619a1b
doc: remove usage of you in n-api doc
mhdawson Feb 2, 2018
8923ad5
doc: small typo in n-api.md
iSkore Feb 4, 2018
ed8ac32
n-api: add methods to open/close callback scope
mhdawson Dec 13, 2017
01b28fa
n-api: remove extra reference from test
Feb 3, 2018
d2f0672
doc: fix typo in n-api.md
vsemozhetbyt Feb 6, 2018
9c20b87
test: improve error message output
bshankar Feb 1, 2018
ef50323
test: convert new tests to use error types
jackhorton Feb 5, 2018
c4592e6
doc: fix exporting a function example
Feb 8, 2018
5ab4d31
doc: mark NAPI_AUTO_LENGTH as code
tniessen Feb 10, 2018
a1df466
test: remove unnecessary timer
cjihrig Feb 11, 2018
eb85cd8
n-api: fix object test
Feb 27, 2018
2b3207d
doc: fix n-api asynchronous threading docs
ebickle Mar 1, 2018
7a80a8d
n-api: update reference test
Mar 2, 2018
de2d3a6
n-api: update documentation
Mar 1, 2018
a92796e
n-api: resolve promise in test
Mar 8, 2018
2cfd4fe
n-api,test: add a new.target test to addons-napi
boingoing Jan 17, 2018
4a0fc8a
n-api,test: add int64 bounds tests
kfarnung Mar 12, 2018
ff968b7
doc: fix typos on n-api
shama Mar 16, 2018
01790d4
n-api: add missing exception checking
mhdawson Mar 15, 2018
a19b829
n-api: separate out async_hooks test
Mar 16, 2018
9b1b810
n-api: add napi_fatal_exception
mafintosh Mar 14, 2018
3a50835
n-api: re-write test_make_callback
Mar 19, 2018
b25dec8
doc: Add a missing comma
leaves4j Mar 23, 2018
c33daa4
n-api: bump version of n-api supported
mhdawson Mar 20, 2018
65b06de
n-api: ensure in-module exceptions are propagated
Mar 22, 2018
e3a2737
n-api: back up env before finalize
Apr 1, 2018
d73e802
n-api: add more `int64_t` tests
kfarnung Mar 16, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
n-api: resolve promise in test
The last promise created by the test for the purposes of making sure
that its type is indeed a promise needs to be resolved so as to avoid
having it left in the pending state at the end of the test.

PR-URL: #19245
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
  • Loading branch information
Gabriel Schulhof committed Apr 12, 2018
commit a92796e6aa2cc625202b1e23b10a7c5f79489858
4 changes: 3 additions & 1 deletion test/addons-napi/test_promise/test.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,9 @@ common.crashOnUnhandledRejection();
test_promise.concludeCurrentPromise(Promise.resolve('chained answer'), true);
}

assert.strictEqual(test_promise.isPromise(test_promise.createPromise()), true);
const promiseTypeTestPromise = test_promise.createPromise();
assert.strictEqual(test_promise.isPromise(promiseTypeTestPromise), true);
test_promise.concludeCurrentPromise(undefined, true);

const rejectPromise = Promise.reject(-1);
const expected_reason = -1;
Expand Down