Closed Bug 1438937 Opened 2 years ago Closed 2 years ago

[wpt-sync] PR 8583 - Add http-state cookie tests from abart/http-state github repository

Categories

(Core :: Networking: Cookies, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: wptsync, Unassigned)

Details

(Whiteboard: [wptsync downstream error][necko-triaged])

These are the first steps to port the http-state cookie tests from
https://github.com/abarth/http-state/tree/master/tests
to the external/wpt/ directory.

Change-Id: I1fceccf0bc5d080d024998451abd027142d6a16a
Reviewed-on: https://chromium-review.googlesource.com/807988
WPT-Export-Revision: 9f1758b8851a08de60d957863f34eaaa4d738d4a
Component: web-platform-tests → Networking: Cookies
Product: Testing → Core
Josh, do you have any time to take a look at these cookie tests?
Flags: needinfo?(josh)
I have looked at them. What exactly would you like to get out of me looking at them? :)
Flags: needinfo?(josh) → needinfo?(valentin.gosu)
(In reply to Josh Matthews [:jdm] from comment #4)
> I have looked at them. What exactly would you like to get out of me looking
> at them? :)

Oh, I should have been more specific. It seems that the new tests are failing. I can't tell if that's expected or not.
Is the reason they're failing something obvious, or should we get someone to investigate?
Flags: needinfo?(valentin.gosu)
The log output looks quite suspicious to me. I've left a comment at https://github.com/w3c/web-platform-tests/pull/8583 which the author will hopefully reply to.
Priority: -- → P3
Whiteboard: [wptsync downstream] → [wptsync downstream][necko-triaged]
Whiteboard: [wptsync downstream][necko-triaged] → [wptsync downstream error][necko-triaged]
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f369b0fb504c
[wpt PR 8583] - Add http-state cookie tests from abart/http-state github repository, a=testonly
https://hg.mozilla.org/mozilla-central/rev/f369b0fb504c
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.