delta/node-new.git/test/js-native-api/test_object, branch devsnek-patch-1

n-api: support for object freeze/seal

2020-10-07T15:31:43+00:00

PR-URL: https://github.com/nodejs/node/pull/35359
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Michael Dawson 
Reviewed-By: Gireesh Punathil

n-api: support type-tagging objects

2020-08-01T01:30:30+00:00

`napi_instanceof()` is insufficient for reliably establishing the data
type to which a pointer stored with `napi_wrap()` or
`napi_create_external()` inside a JavaScript object points. Thus, we
need a way to "mark" an object with a value that, when later retrieved,
can unambiguously tell us whether it is safe to cast the pointer stored
inside it to a certain structure.

Such a check must survive loading/unloading/multiple instances of an
addon, so we use UUIDs chosen *a priori*.

Fixes: https://github.com/nodejs/node/issues/28164
Co-authored-by: Anna Henningsen 
PR-URL: https://github.com/nodejs/node/pull/28237
Reviewed-By: Anna Henningsen 
Reviewed-By: Michael Dawson 
Reviewed-By: Colin Ihrig 
Signed-off-by: Gabriel Schulhof

n-api: define release 6

2020-03-09T18:58:51+00:00

Mark all N-APIs that have been added since version 5 as stable.

PR-URL: https://github.com/nodejs/node/pull/32058
Fixes: https://github.com/nodejs/abi-stable-node/issues/393
Co-Authored-By: legendecas 
Signed-off-by: Gabriel Schulhof 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Michael Dawson

test: cover property n-api null cases

2020-01-26T05:23:05+00:00

Add test coverage for passing `NULL` to each parameter of
`napi.*(propert|element)` and `napi_set_prototype`. In the case of
`napi_define_properties` also test setting various initializer fields
to `NULL`.

PR-URL: https://github.com/nodejs/node/pull/31488
Reviewed-By: David Carlier 
Reviewed-By: Rich Trott

n-api: add napi_get_all_property_names

2020-01-12T02:50:28+00:00

Co-Authored-By: Gabriel Schulhof 

PR-URL: https://github.com/nodejs/node/pull/30006
Reviewed-By: Anna Henningsen 
Reviewed-By: Chengzhong Wu 
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Michael Dawson 
Reviewed-By: Rich Trott

n-api: refactor a previous commit

2019-07-26T22:09:04+00:00

This is a refactoring of https://github.com/nodejs/node/issues/27628
following https://github.com/nodejs/node/pull/28505.

This change factors out functions `add_last_status()` and
`add_returned_status()` so they may be reused in the tests for passing
information about the last error status and/or a returned `napi_status`
to JavaScript.

PR-URL: https://github.com/nodejs/node/pull/28848
Reviewed-By: Anna Henningsen 
Reviewed-By: Gabriel Schulhof

test: pass null params to napi_xxx_property()

2019-05-13T23:27:13+00:00

    For napi_set_property(), each one of the following arguments is checked:
      napi_env env,
      napi_value object,
      napi_value key,
      napi_value* value.

    For napi_has_property(), each one of the following arguments is checked:

      napi_env env,
      napi_value object,
      napi_value key,
      bool* result

    For napi_get_property, each one of the following arguments is checked:

      napi_env env,
      napi_value object,
      napi_value key,
      napi_value* result

PR-URL: https://github.com/nodejs/node/pull/27628
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Rich Trott

n-api: make napi_get_property_names return strings

2019-05-06T03:12:27+00:00

The documentation says that this method returns an array of strings.
Currently, it does not do so for indices. Resolve that by telling
V8 explicitly to convert to string.

PR-URL: https://github.com/nodejs/node/pull/27524
Fixes: https://github.com/nodejs/node/issues/27496
Reviewed-By: Colin Ihrig 
Reviewed-By: Minwoo Jung 
Reviewed-By: Ruben Bridgewater

test: cover napi_get/set/has_named_property()

2019-04-05T18:56:09+00:00

Add test coverage for these N-APIs.

PR-URL: https://github.com/nodejs/node/pull/26947
Reviewed-By: Daniel Bevenius 
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