Update the test with fragment of import.meta.url in dedicated-worker-import-meta.html
Categories
(Testing :: web-platform-tests, defect)
Tracking
(firefox115 fixed)
| Tracking | Status | |
|---|---|---|
| firefox115 | --- | fixed |
People
(Reporter: allstars.chh, Assigned: allstars.chh)
Details
Attachments
(1 file)
The wpt test
https://searchfox.org/mozilla-central/rev/85b4f7363292b272eb9b606e00de2c37a6be73f0/testing/web-platform/tests/workers/modules/dedicated-worker-import-meta.html#56
looks wrong, the expected URL should contain the fragment '#1', however the test expects the import.meta.url shouldn't contain the fragment.
Chrome/Firefox/Webkit are all failing on this test
https://wpt.fyi/results/workers/modules/dedicated-worker-import-meta.html?label=master&label=experimental&product=chrome&product=firefox&product=safari&aligned&view=interop&q=label%3Ainterop-2023-modules
The same test for shared-worker did fixed this.
https://searchfox.org/mozilla-central/rev/85b4f7363292b272eb9b606e00de2c37a6be73f0/testing/web-platform/tests/workers/modules/shared-worker-import-meta.html#46
|
Assignee | |
Comment 1•3 years ago
|
||
|
||
Comment 2•3 years ago
•
|
||
(In reply to Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi] from comment #0)
The wpt test
https://searchfox.org/mozilla-central/rev/85b4f7363292b272eb9b606e00de2c37a6be73f0/testing/web-platform/tests/workers/modules/dedicated-worker-import-meta.html#56
looks wrong, the expected URL should contain the fragment '#1', however the test expects the import.meta.url shouldn't contain the fragment.
That was my guess. I queried this with Domenic who wrote the test: https://github.com/web-platform-tests/wpt/pull/10717#issuecomment-1535918639
|
|
||
Comment 3•3 years ago
|
||
(In reply to Jon Coppeard (:jonco) from comment #2)
Reply from Domenic:
@jonco3 yes, I can't understand why we wrote the tests that way. It looks exactly wrong. Please feel free to open a PR to fix it.
|
||
Comment 6•3 years ago
|
||
| bugherder | ||
Description
•