module: fixing url change in load sync hook chain

[awto]

Fixes: #56376

[nodejs-github-bot]

Review requested:

• @nodejs/loaders

[gurgunday]

lgtm

[JakobJingleheimer]

Thanks for this! It looks okay to me. The test is fine, but I think it could be improved/simplified.

I'm gonna tag the original author of this for review since she has more context than me.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.17%. Comparing base (0dbbaba) to head (406b3d2).
Report is 71 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #56402      +/-   ##
==========================================
+ Coverage   88.53%   89.17%   +0.63%     
==========================================
  Files         657      662       +5     
  Lines      190718   191751    +1033     
  Branches    36603    36905     +302     
==========================================
+ Hits       168861   170993    +2132     
+ Misses      15024    13620    -1404     
- Partials     6833     7138     +305     

Files with missing lines	Coverage Δ
lib/internal/modules/cjs/loader.js	98.11% <100.00%> (ø)

... and 126 files with indirect coverage changes

[joyeecheung]

Can you add a test for import('foo') as well?

[joyeecheung]

LGTM, thanks, I'd still prefer to see a test for import, but this is only touching CJS loader, so it's fine to leave that to a follow-up.

[mcollina]

lgtm

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/64448/