[wpt-sync] Sync PR 20467 - [fetch] Designate tests for optional functionality
Categories
(Core :: DOM: Networking, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox73 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream][necko-triaged])
Sync web-platform-tests PR 20467 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/20467
Details from upstream follow.
Mike Pennisi <mike@mikepennisi.com> wrote:
[fetch] Designate tests for optional functionality
The subtests for "prefetch" have been observed to produce unstable
results in some browsers. Declaring them in a separate test reduces the
impact of this instability and helps to contextualize failures.
@jdm @domfarolino The instability in Firefox is blocking gh-20454 and gh-20455.
I have the ability to merge despite the CI failure, but since the problem has
the potential to block future pull requests (and since the "failure" is prone
to be misinterpreted), I think it's preferable to move those tests to dedicated
files.Firefox will most likely flag these new tests as unstable, but I'll feel more
comfortable overriding the stability check for this pull request.Ideally, we would express these expectations in a way that produced
deterministic results. I'm all ears if you have ideas for how to do that, but I
think the spec may be too loose to make any guarantees.
Assignee | ||
Comment 1•5 years ago
|
||
PR 20467 applied with additional changes from upstream: dfab246b15077e1d3740a222c25eef112cfd20d4, bddf074a0c1b8d10c5d6e4b3d151e1612178f7a2, d7bdce750674c33f475d82314fd1caf1cb971554
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=8af176f67a54171f238a785112b46f9469c57280
Updated•5 years ago
|
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab4130d3dc86 [wpt PR 20467] - [fetch] Designate tests for optional functionality, a=testonly
Assignee | ||
Comment 4•4 years ago
|
||
Test result changes from PR not available.
Comment 5•4 years ago
|
||
bugherder |
Description
•