Closed Bug 1842139 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 40911 - Do not check contents of fetch exception message

Categories

(Core :: DOM: Service Workers, task, P4)

Product:
Core
Component:
DOM: Service Workers
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
117 Branch
Tracking Flags:
Tracking Status
firefox117 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 40911 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/40911
Details from upstream follow.

Adam Rice <ricea@chromium.org> wrote:

Do not check contents of fetch exception message

The services-workers/service-worker/fetch-error.https.html made an assertion about the contents of the message on the exception raised by response.text(), however the standard only guarantees that the exception be a TypeError and makes no guarantees about its contents.

Relax the assertion to only require a TypeError.

See https://crbug.com/1017861.

Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 3 subtests

Status Summary

Firefox

OK : 1
PASS: 2
FAIL: 1

Chrome

OK : 1
PASS: 3

Safari

OK : 1
PASS: 3

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

New Tests That Don't Pass

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79474a7213eb
[wpt PR 40911] - Do not check contents of fetch exception message, a=testonly

https://hg.mozilla.org/mozilla-central/rev/79474a7213eb

Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox117: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
You need to log in before you can comment on or make changes to this bug.