1886669 - [wpt-sync] Sync PR 45240 - Add test for multiple RouterCondition

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Shunya Shishido <[e-mail]> wrote:

Add test for multiple RouterCondition

This CL add the test to ensure multiple RouterConditions in one
condition work with "and" condition. All RouterConditions should be
matched.

We don't have RunningStatus, those are tested separately because that
needs an internal API.

Bug: 40943429
Change-Id: Ieeb2ccc03baffe2ea2cfd0a88d4ea091de708ef9
Reviewed-on: https://chromium-review.googlesource.com/5379636
WPT-Export-Revision: ed8e8f5de14af8a37a26cb5e1fc8a27699402b03

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=9772d21babf531fa2deaa350222c835a2cfd0e3d

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=fc2ab4fa8d237f7adc3c252cca711911ba69995f

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d5605351a867b43a7d317f1a43a5cf2d3536580

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

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

Total 1 tests and 2 subtests

Status Summary

Firefox

OK : 1
FAIL: 2

Chrome

OK : 1
PASS: 2

Safari

OK : 1
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /service-workers/service-worker/tentative/static-router/static-router-mutiple-conditions.https.html [wpt.fyi]
  • Multiple conditions work with and operation: FAIL (Chrome: PASS, Safari: FAIL)
  • Multiple conditions including requestDestination work with and operation: FAIL (Chrome: PASS, Safari: FAIL)

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3f6c9018b4c3
[wpt PR 45240] - Add test for multiple RouterCondition, a=testonly
https://hg.mozilla.org/integration/autoland/rev/cecb9f49d4ef
[wpt PR 45240] - Update wpt metadata, a=testonly

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/3f6c9018b4c3
https://hg.mozilla.org/mozilla-central/rev/cecb9f49d4ef