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

node-api: get Node API version used by addon

2023-05-05T15:00:27+00:00

PR-URL: https://github.com/nodejs/node/pull/45715
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Michael Dawson 
Reviewed-By: Chengzhong Wu

src: throw DataCloneError on transfering untransferable objects

2023-05-05T11:22:42+00:00

The HTML StructuredSerializeWithTransfer algorithm defines that when
an untransferable object is in the transfer list, a DataCloneError is
thrown.
An array buffer that is already transferred is also considered as
untransferable.

PR-URL: https://github.com/nodejs/node/pull/47604
Reviewed-By: James M Snell 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Joyee Cheung

node-api: deprecate napi_module_register

2023-03-24T18:00:40+00:00

PR-URL: https://github.com/nodejs/node/pull/46319
Reviewed-By: Chengzhong Wu 
Reviewed-By: Michael Dawson

node-api: verify cleanup hooks order

2023-03-14T16:57:25+00:00

Cleanup hooks are called before the environment shutdown finalizer
invocations.

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

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

process,worker: ensure code after exit() effectless

2022-12-25T09:54:12+00:00

Cope with the delay(to the next function call) of
v8::Isolate::TerminateExecution()

PR-URL: https://github.com/nodejs/node/pull/45620
Reviewed-By: Anna Henningsen 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Minwoo Jung 
Reviewed-By: Antoine du Hamel

node-api: handle no support for external buffers

2022-11-09T18:30:43+00:00

Refs: https://github.com/electron/electron/issues/35801
Refs: https://github.com/nodejs/abi-stable-node/issues/441

Electron recently dropped support for external
buffers. Provide a way for addon authors to:
- hide the methods to create external buffers so they can
  avoid using them if they want the broadest compatibility.
- call the methods that create external buffers at runtime
  to check if external buffers are supported and either
  use them or not based on the return code.

Signed-off-by: Michael Dawson 

PR-URL: https://github.com/nodejs/node/pull/45181
Reviewed-By: Chengzhong Wu 
Reviewed-By: Minwoo Jung

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

src: disambiguate terms used to refer to builtins and addons

2022-08-08T17:36:49+00:00

The term "native module" dates back to some of the oldest code
in the code base. Within the context of Node.js core it usually
refers to modules that are native to Node.js (e.g. fs, http),
but it can cause confusion for people who don't work on this
part of the code base, as "native module" can also refer to
native addons - which is even the case in some of the API
docs and error messages.

This patch tries to make the usage of these terms more consistent.
Now within the context of Node.js core:

- JavaScript scripts that are built-in to Node.js are now referred
  to as "built-in(s)". If they are available as modules,
  they can also be referred to as "built-in module(s)".
- Dynamically-linked shared objects that are loaded into
  the Node.js processes are referred to as "addons".

We will try to avoid using the term "native modules" because it could
be ambiguous.

Changes in this patch:

File names:
- node_native_module.h -> node_builtins.h,
- node_native_module.cc -> node_builtins.cc

C++ binding names:
- `native_module` -> `builtins`

`node::Environment`:
- `native_modules_without_cache` -> `builtins_without_cache`
- `native_modules_with_cache` -> `builtins_with_cache`
- `native_modules_in_snapshot` -> `builtins_in_cache`
- `native_module_require` -> `builtin_module_require`

`node::EnvSerializeInfo`:
- `native_modules` -> `builtins

`node::native_module::NativeModuleLoader`:
- `native_module` namespace -> `builtins` namespace
- `NativeModuleLoader` -> `BuiltinLoader`
- `NativeModuleRecordMap` -> `BuiltinSourceMap`
- `NativeModuleCacheMap` -> `BuiltinCodeCacheMap`
- `ModuleIds` -> `BuiltinIds`
- `ModuleCategories` -> `BuiltinCategories`
- `LoadBuiltinModuleSource` -> `LoadBuiltinSource`

`loader.js`:
- `NativeModule` -> `BuiltinModule` (the `NativeModule` name used in
  `process.moduleLoadList` is kept for compatibility)

And other clarifications in the documentation and comments.

PR-URL: https://github.com/nodejs/node/pull/44135
Fixes: https://github.com/nodejs/node/issues/44036
Reviewed-By: Jacob Smith 
Reviewed-By: Matteo Collina 
Reviewed-By: Michael Dawson 
Reviewed-By: Richard Lau 
Reviewed-By: Jiawen Geng 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Tobias Nießen 
Reviewed-By: Jan Krems

test: mark flaky tests on smartos

2022-06-28T17:06:48+00:00

PR-URL: https://github.com/nodejs/node/pull/43596
Refs: https://github.com/nodejs/node/issues/43457
Reviewed-By: Tobias Nießen 
Reviewed-By: Filip Skokan 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Richard Lau 
Reviewed-By: Darshan Sen