Fix node-tests asserts being swallowed

Author: kraenhansen

packages/node-tests/rolldown.config.mts

@@ -64,6 +64,18 @@ function testSuiteConfig(suitePath: string): RolldownOptions[] {
           delimiters: ["", ""],
         },
       ),
+      replacePlugin(
+        {
+          // Replace the default export to return a function instead of initializing the addon immediately
+          // This allows the test runner to intercept any errors which would normally be thrown when importing
+          // to work around Metro's `guardedLoadModule` swallowing errors during module initialization
+          // See https://github.com/facebook/metro/blob/34bb8913ec4b5b02690b39d2246599faf094f721/packages/metro-runtime/src/polyfills/require.js#L348-L353
+          "export default require_test();": "export default require_test;",
+        },
+        {
+          delimiters: ["", ""],
+        },
+      ),
       aliasPlugin({
         entries: [
           {

packages/node-tests/scripts/generate-entrypoint.mts

@@ -36,7 +36,7 @@ function suiteToString(suite: TestSuite, indent = 1): string {
   return Object.entries(suite)
     .map(([key, value]) => {
       if (typeof value === "string") {
-        return `${padding}"${key}": () => require("./${value}")`;
+        return `${padding}"${key}": require("./${value}")`;
       } else {
         return `${padding}"${key}": {\n${suiteToString(
           value,