esm: handle globalPreload hook returning a nullish value

PR-URL: #48249 Backport-PR-URL: #50669Fixes: #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(typeofpreloaded!=='string'){// [2]
 thrownewERR_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 }=awaitspawnPromisified(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 }=awaitspawnPromisified(execPath,[
 '--no-warnings',