Skip to content

port test_string to CTS#44

Open
bavulapati wants to merge 1 commit intonodejs:mainfrom
bavulapati:feat/port-test-string
Open

port test_string to CTS#44
bavulapati wants to merge 1 commit intonodejs:mainfrom
bavulapati:feat/port-test-string

Conversation

@bavulapati
Copy link
Copy Markdown

Ports
test_string from the Node.js test suite to the CTS.

  • Most of the files are identical except for the js files and CMakeLists.txt
  • js files use global assert and loadAddon instead of requiring modules
  • CMakeLists.txt sets NAPI_VERSION >= 10, as API like node_api_create_external_string_latin1 are available only from NAPI_VERSION >= 10. Ref
    docs

Ports
[test_string](https://github.com/nodejs/node/tree/main/test/js-native-api/test_string)
from the Node.js test suite to the CTS.
- Most of the files are identical except for the js files and
CMakeLists.txt
- js files use global assert and loadAddon instead of requiring modules
- CMakeLists.txt sets NAPI_VERSION >= 10, as API like
node_api_create_external_string_latin1 are available only from
NAPI_VERSION >= 10. Ref
[docs](https://nodejs.org/docs/latest/api/n-api.html#node-api-create-external-string-latin1)

Signed-off-by: Balakrishna Avulapati <ba@bavulapati.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Need Triage

Development

Successfully merging this pull request may close these issues.

1 participant