Skip to content

Commit c11fd3d

Browse files
huytdps13400kraenhansen
authored andcommitted
test: enable lambda Node-API examples (#457)
1 parent 2b42c35 commit c11fd3d

2 files changed

Lines changed: 17 additions & 3 deletions

File tree

‎packages/node-addon-examples/scripts/copy-examples.mts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,11 @@ const ALLOW_LIST = [
1717
"1-getting-started/4_object_factory/node-addon-api/",
1818
"1-getting-started/5_function_factory/napi/",
1919
// "1-getting-started/5_function_factory/node-addon-api/",
20-
// "1-getting-started/6_object_wrap/napi/", // TODO: Fix C++ support to allow lambda functions
20+
"1-getting-started/6_object_wrap/napi/",
2121
// "1-getting-started/6_object_wrap/node-addon-api/",
22-
// "1-getting-started/7_factory_wrap/napi/", // TODO: Fix C++ support to allow lambda functions
22+
"1-getting-started/7_factory_wrap/napi/",
2323
// "1-getting-started/7_factory_wrap/node-addon-api/",
24-
// "2-js-to-native-conversion/8_passing_wrapped/napi/", // TODO: Fix C++ support to allow lambda functions
24+
"2-js-to-native-conversion/8_passing_wrapped/napi/",
2525
// "2-js-to-native-conversion/8_passing_wrapped/node-addon-api/",
2626
// "2-js-to-native-conversion/array_buffer_to_native/node-addon-api/",
2727
// "2-js-to-native-conversion/object-template-demo/napi/", // TODO: Fix C++ support to allow noexcept

‎packages/node-addon-examples/src/index.ts‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,20 @@ export const suites: Record<
7575
assertLogs(() => {
7676
require("../examples/1-getting-started/5_function_factory/napi/addon.js");
7777
}, ["hello world"]),
78+
"6_object_wrap/napi": () =>
79+
assertLogs(() => {
80+
require("../examples/1-getting-started/6_object_wrap/napi/addon.js");
81+
}, ["11", "12", "13", "13", "130", "-13", "false"]),
82+
"7_factory_wrap/napi": () =>
83+
assertLogs(() => {
84+
require("../examples/1-getting-started/7_factory_wrap/napi/addon.js");
85+
}, ["11", "12", "13", "21", "22", "23"]),
86+
},
87+
"2-js-to-native-conversion": {
88+
"8_passing_wrapped/napi": () =>
89+
assertLogs(() => {
90+
require("../examples/2-js-to-native-conversion/8_passing_wrapped/napi/addon.js");
91+
}, ["30"]),
7892
},
7993
"5-async-work": {
8094
async_work_thread_safe_function: () => {

0 commit comments

Comments
 (0)