napi: unskip passing tests (#24359)

.prettierignore

@@ -7,4 +7,5 @@ src/react-refresh.js
 *.min.js
 test/snippets
 test/js/node/test
+test/napi/node-napi-tests
 bun.lock

test/napi/node-napi-tests/test/common/index.js

@@ -459,12 +459,14 @@ if (process.env.NODE_TEST_KNOWN_GLOBALS !== '0') {
     return leaked;
   }
 
-  process.on('exit', function() {
-    const leaked = leakedGlobals();
-    if (leaked.length > 0) {
-      assert.fail(`Unexpected global(s) found: ${leaked.join(', ')}`);
-    }
-  });
+  // --- Commmented out for Bun ---
+  // process.on('exit', function () {
+  //   const leaked = leakedGlobals();
+  //   if (leaked.length > 0) {
+  //     assert.fail(`Unexpected global(s) found: ${leaked.join(', ')}`);
+  //   }
+  // });
+  // --- Commmented out for Bun ---
 }
 
 const mustCallChecks = [];

test/napi/node-napi-tests/test/node-api/test_async/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // crash inside uv_sleep
   test.todoIf(["test.js", "test-uncaught.js", "test-async-hooks.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_async_cleanup_hook/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // crash inside uv_async_init
   test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_callback_scope/do.test.ts

@@ -7,6 +7,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // crash inside uv_queue_work
   // https://github.com/oven-sh/bun/issues/12827 is the latter
   test.todoIf(["test-resolve-async.js", "test-async-hooks.js"].includes(file) || (file === "test.js" && isWindows))(
     file,

test/napi/node-napi-tests/test/node-api/test_env_teardown_gc/do.test.ts

@@ -6,7 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
-  test.todoIf(["test.js"].includes(file))(file, () => {
+  test(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });
 }

test/napi/node-napi-tests/test/node-api/test_instance_data/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // crash inside uv_thread_create
   test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_make_callback_recurse/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // Missing expected exception.
   test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_threadsafe_function/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // crash inside uv_thread_create
   test.todoIf(["test.js", "test_legacy_uncaught_exception.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_uv_loop/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // unsupported uv function: uv_check_init
   test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_uv_threadpool_size/do.test.ts

@@ -6,7 +6,8 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
-  test.todoIf(["test.js", "node-options.js"].includes(file))(file, () => {
+  // unsupported uv function: uv_sleep
+  test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });
 }

test/napi/node-napi-tests/test/node-api/test_worker_buffer_callback/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // AssertionError: Missing expected exception (DataCloneError).
   test.todoIf(["test.js", "test-free-called.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });

test/napi/node-napi-tests/test/node-api/test_worker_terminate/do.test.ts

@@ -6,6 +6,7 @@ test("build", async () => {
 });
 
 for (const file of Array.from(new Bun.Glob("*.js").scanSync(import.meta.dir))) {
+  // Assertion failed: (status == napi_pending_exception), function Test, file test_worker_terminate.c, line 21.
   test.todoIf(["test.js"].includes(file))(file, () => {
     run(dirname(import.meta.dir), basename(import.meta.dir) + sep + file);
   });