Closed
Bug 1983822
Opened 7 months ago
Closed 4 months ago
[wdspec] Add WebDriver classic tests for the "Get All Cookies" command
Categories
(Remote Protocol :: Marionette, task, P3)
Remote Protocol
Marionette
Tracking
(firefox147 fixed)
RESOLVED
FIXED
147 Branch
| Tracking | Status | |
|---|---|---|
| firefox147 | --- | fixed |
People
(Reporter: whimboo, Assigned: khalid.alhaddad98, Mentored)
Details
(Whiteboard: [lang=py][wptsync upstream][webdriver:m18][webdriver:external][webdriver:relnote])
Attachments
(1 file)
I checked the existing WebDriver classic tests for cookies and noticed that we do not have a single test for the Get All Cookies command. We probably want to have some basic ones.
Those should be added at:
https://searchfox.org/mozilla-central/source/testing/web-platform/tests/webdriver/tests/classic/get_all_cookies
This might be a good mentored bug.
| Reporter | ||
Updated•7 months ago
|
Mentor: hskupin
Priority: -- → P3
Whiteboard: [lang=py]
| Assignee | ||
Comment 1•4 months ago
|
||
Updated•4 months ago
|
Assignee: nobody → khalid.alhaddad98
Status: NEW → ASSIGNED
Updated•4 months ago
|
Attachment #9525584 -
Attachment description: Bug 1983822 - Add WebDriver classic tests for the "Get All Cookies" command. r?#webdriver-reviewers → Bug 1983822 - [wdspec] Add tests for WebDriver classic's "Get All Cookies" command.
Pushed by hskupin@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/3aba5f11a2ed
https://hg.mozilla.org/integration/autoland/rev/1e3772ae25b1
[wdspec] Add tests for WebDriver classic's "Get All Cookies" command. r=whimboo
Comment 3•4 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
status-firefox147:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 147 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/56406 for changes under testing/web-platform/tests
Whiteboard: [lang=py] → [lang=py], [wptsync upstream]
Upstream PR merged by moz-wptsync-bot
| Reporter | ||
Comment 6•4 months ago
|
||
Khalid, I have to say thanks again for your work. It's great to see that we now have test coverage for this command.
| Reporter | ||
Updated•2 months ago
|
Whiteboard: [lang=py], [wptsync upstream] → [lang=py][wptsync upstream][webdriver:m18][webdriver:external][webdriver:relnote]
You need to log in
before you can comment on or make changes to this bug.
Description
•