esm: handle globalPreload hook returning a nullish value

PR-URL: #48249
Backport-PR-URL: #50669
Fixes: #48240
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>

Author: aduh95

lib/internal/modules/esm/hooks.js

@@ -168,7 +168,7 @@ class Hooks {
         port: insideLoader,
       });
 
-      if (preloaded == null) { return; }
+      if (preloaded == null) { continue; }
 
       if (typeof preloaded !== 'string') { // [2]
         throw new ERR_INVALID_RETURN_VALUE(

test/es-module/test-esm-loader-hooks.mjs

@@ -348,6 +348,20 @@ describe('Loader hooks', { concurrency: true }, () => {
     });
   });
 
+  it('should handle globalPreload returning undefined', async () => {
+    const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
+      '--no-warnings',
+      '--experimental-loader',
+      'data:text/javascript,export function globalPreload(){}',
+      fixtures.path('empty.js'),
+    ]);
+
+    assert.strictEqual(stderr, '');
+    assert.strictEqual(stdout, '');
+    assert.strictEqual(code, 0);
+    assert.strictEqual(signal, null);
+  });
+
   it('should be fine to call `process.removeAllListeners("beforeExit")` from the main thread', async () => {
     const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
       '--no-warnings',