1779639 - [wpt-sync] Sync PR 34852 - Add missing step_func() in a couple of cookie tests

Status: RESOLVED FIXED

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

Description

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

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

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

Chris Dumez <[e-mail]> wrote:

Add missing step_func() in a couple of cookie tests

Without this, any subtest failure would result in a rejected promise
which would cause the test to time out.

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=471d0680019d3b58ce43ea2e83481979a4c2fe43

Comment 2

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

CI Results

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

Total 2 tests and 16 subtests

Status Summary

Firefox

OK : 2
PASS: 16
FAIL: 1

Chrome

OK : 2
PASS: 17

Safari

OK : 2
PASS: 13
FAIL: 4

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

• /cookies/path/default.html [wpt.fyi]
  • Test for default cookie path: FAIL

New Tests That Don't Pass

• /cookies/path/default.html [wpt.fyi]
  • Test for default cookie path: FAIL (Chrome: PASS, Safari: PASS)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/82ca05bfa527
[wpt PR 34852] - Add missing step_func() in a couple of cookie tests, a=testonly
https://hg.mozilla.org/integration/autoland/rev/f69ddcf0e45c
[wpt PR 34852] - Update wpt metadata, a=testonly

Comment 4

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/82ca05bfa527
https://hg.mozilla.org/mozilla-central/rev/f69ddcf0e45c