esm: add a fall back when importer in not a file

[aduh95]

There's this assumption in the code that the importing module (which I called the importer in the PR title) URL would be under the file: scheme, but there's no reason this is the case, even without loader it could be under the data: scheme. TBH I'm not sure why we bother converting it to a path instead of exposing the URL and let the user decides if they want to convert it to a path, but for the sake of not introducing a breaking change, I'm simply wrapping the fileURLToPath in a try/catch block.

[nodejs-github-bot]

Review requested:

• @nodejs/loaders

[JakobJingleheimer]

🙌

[nodejs-github-bot]

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

[codecov]

Codecov Report

Attention: Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.

Project coverage is 88.41%. Comparing base (b0ffe9e) to head (1722d93).

Files with missing lines	Patch %	Lines
lib/internal/modules/esm/resolve.js	80.95%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #55471      +/-   ##
==========================================
- Coverage   88.42%   88.41%   -0.01%     
==========================================
  Files         653      653              
  Lines      187498   187516      +18     
  Branches    36100    36106       +6     
==========================================
+ Hits       165791   165798       +7     
- Misses      14957    14968      +11     
  Partials     6750     6750              

Files with missing lines	Coverage Δ
lib/internal/modules/esm/resolve.js	96.32% <80.95%> (-0.22%)	⬇️

... and 29 files with indirect coverage changes

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[github-actions]

Fast-track has been requested by @aduh95. Please 👍 to approve.