Properly resolve paths in SimpleModuleLoader and add path to Referrer::Script

[hansl]

We do have the path in parsed Script (if the Source has it), so use that if we can.

Fixes #3790.

[hansl]

Note: This should fix many test262 that were broken by the last changes to SimpleModuleLoader, but it won't fix all of them. That is because some of those tests were false positive and the code is not behaving properly. I filed a bug regarding one of those newly failing test. I will have to investigate if there are more.

[codecov]

Codecov Report

Attention: Patch coverage is 73.68421% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 50.23%. Comparing base (6ddc2b4) to head (56f3cde).
Report is 120 commits behind head on main.

Files	Patch %	Lines
core/engine/src/module/loader.rs	78.12%	7 Missing ⚠️
core/engine/src/script.rs	25.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3791      +/-   ##
==========================================
+ Coverage   47.24%   50.23%   +2.98%     
==========================================
  Files         476      456      -20     
  Lines       46892    44816    -2076     
==========================================
+ Hits        22154    22513     +359     
+ Misses      24738    22303    -2435     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jedel1043]

Test262 conformance changes

Test result	main count	PR count	difference
Total	50,268	50,268	0
Passed	42,773	42,753	-20
Ignored	1,391	1,391	0
Failed	6,104	6,124	+20
Panics	18	18	0
Conformance	85.09%	85.05%	-0.04%

Broken tests (20):

test/language/expressions/dynamic-import/for-await-resolution-and-error-agen-yield.js (previously Passed)
test/language/expressions/dynamic-import/custom-primitive.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/yield-assign-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/yield-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/call-expr-identifier.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/logical-or-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/logical-and-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/lhs-assign-operator-assign-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/cover-call-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/additive-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/member-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/ternary.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/await-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/call-expr-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/call-expr-arguments.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/array-literal.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/object-literal.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/lhs-eq-assign-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/cover-parenthesized-expr.js (previously Passed)
test/language/expressions/dynamic-import/assignment-expression/identifier.js (previously Passed)

[jedel1043]

By the way, this doesn't print a nice comment because the action doesn't work on forks. If you want to see the test count, you can check it on the test262 action:
https://github.com/boa-dev/boa/actions/runs/8562670896/job/23466338447?pr=3791

Specifically on the input of the "Write a new comment" task.

[jedel1043]

Fixed the remaining tests. This now just needs some small fixes on the tests and docs.

[hansl]

Apparently the definition of absolute path is different on Windows. Will fix tomorrow.

Also confirming that (at least locally) the test262 results are the same after the latest commit on this PR as before my SimpleModuleLoader change:

[jedel1043]

Really nice work!

[HalidOdat]

Nice work! :)

[HalidOdat]

Nice addition! :)