delta/node-new.git/test/node-api/test_async, branch main

test: add trailing commas in `test/node-api`

2023-01-29T19:01:09+00:00

PR-URL: https://github.com/nodejs/node/pull/46384
Reviewed-By: Michael Dawson 
Reviewed-By: James M Snell

test: use uv_sleep() where possible

2022-10-27T17:13:50+00:00

PR-URL: https://github.com/nodejs/node/pull/45124
Reviewed-By: Michaël Zasso 
Reviewed-By: Darshan Sen 
Reviewed-By: theanarkh 
Reviewed-By: Colin Ihrig 
Reviewed-By: Tobias Nießen 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Daeyeon Jeong 
Reviewed-By: Juan José Arboleda 
Reviewed-By: Gerhard Stöbich

test: prepare for consistent comma-dangle lint rule

2021-04-02T06:14:29+00:00

Make changes so that tests will pass when the comma-dangle settings
applied to the rest of the code base are also applied to tests.

PR-URL: https://github.com/nodejs/node/pull/37930
Reviewed-By: James M Snell 
Reviewed-By: Derek Lewis

test: rename n-api to node-api

2021-02-06T13:03:38+00:00

This renames the macros used in the tests from `NAPI_*` to
`NODE_API_*`.

PR-URL: https://github.com/nodejs/node/pull/37217
Reviewed-By: Michael Dawson 
Reviewed-By: Beth Griggs 
Reviewed-By: Rich Trott

test: convert most N-API tests from C++ to C

2020-08-05T21:21:51+00:00

* Prefix functions with `static` to make them local
* Remove anonymous namespaces
* `nullptr` -> `NULL`
* .cc -> .c and update binding.gyp
* `static_cast()` -> `(x)()`
* Replace `new`/`delete` with `malloc()`/`free()`
  (only in test_callback_scope)
* Move lambda out and convert to local function
  (only in test_callback_scope)
* Remove superfluous `#include `
  (only in test_callback_scope_recurse)

Some tests are best left as C++.

```bash
ls -l test/{node-api,js-native-api}/*/*.cc
```

for those remaining as C++ tests.

Signed-off-by: Gabriel Schulhof 
PR-URL: https://github.com/nodejs/node/pull/34615
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen 
Reviewed-By: Jiawen Geng 
Reviewed-By: Joyee Cheung 
Reviewed-By: Gerhard Stöbich

test: use assert() in N-API async test

2019-06-28T02:56:34+00:00

The `Execute()` callback is not allowed to call into JS, so
we should use `assert()` instead of potentially throwing JS errors.

PR-URL: https://github.com/nodejs/node/pull/28423
Fixes: https://github.com/nodejs/help/issues/1998
Reviewed-By: Gireesh Punathil 
Reviewed-By: Yongsheng Zhang 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: Michael Dawson

test: partition N-API tests

2018-12-04T21:58:17+00:00

Partition test/addons-napi into test/js-native-api and test/node-api to
isolate the Node.js-agnostic portion of the N-API tests from the
Node.js-specific portion.

PR-URL: https://github.com/nodejs/node/pull/24557
Reviewed-By: Refael Ackermann 
Reviewed-By: Daniel Bevenius