Closed Bug 1599652 Opened 5 years ago Closed 4 years ago

[wpt-sync] Sync PR 20467 - [fetch] Designate tests for optional functionality

Categories

(Core :: DOM: Networking, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla73
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.

PR 20467 applied with additional changes from upstream: dfab246b15077e1d3740a222c25eef112cfd20d4, bddf074a0c1b8d10c5d6e4b3d151e1612178f7a2, d7bdce750674c33f475d82314fd1caf1cb971554
Component: web-platform-tests → DOM: Networking
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream][necko-triaged]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ab4130d3dc86
[wpt PR 20467] - [fetch] Designate tests for optional functionality, a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
You need to log in before you can comment on or make changes to this bug.