Fix node-tests asserts being swallowed

Author: kraenhansen

packages/node-tests/rolldown.config.mts

@@ -64,6 +64,21 @@ function testSuiteConfig(suitePath: string): RolldownOptions[] {
           delimiters: ["", ""],
         },
       ),
+      replacePlugin(
+        Object.fromEntries(
+          targetNames.map((targetName) => [
+            // 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,