feat(playwright): Clear cookie by name#4693
Conversation
…ince Playwright 1.43)
…ince Playwright 1.43)
…ince Playwright 1.43)
…ince Playwright 1.43)
|
@ngraf PR looks good but tests failed. Could you check if it is needed to update Playwright in the pipeline? |
|
Hi @DavertMik , At least I am able to reproduce the failing Playwright helper test locally now after following the steps in the GitHub pipeline. |
…ince Playwright 1.43)
|
Alright. I fixed the tests. Now my final challenge is to resolve the branch conflict that arose in the meantime. That is a bit challenging for me, but I am confident I will resolve it .. somehow. |
|
Alright. I resolved all conflicts. I think we are good now. :) |
kobenguyent
changed the title
| ```js | ||
| I.clearCookie(); | ||
| I.clearCookie('test'); // Playwright currently doesn't support clear a particular cookie name | ||
| I.clearCookie('test'); |
There was a problem hiding this comment.
maybe it's worth mentioning since Playwright 1.43 so that this won't surprise anyone when they are using playwright < 1.43 and this doesn't work for them.
There was a problem hiding this comment.
I think it's edge case, and I hope everyone will use latest PW
3 participants
Motivation/Description of the PR
Applicable helpers:
Type of change
Checklist:
npm run docs)->
npm run docsdoes not work for me :/npm run lint)->
npm run lintdoes not work for me :/npm test)->
npm testdoes not work for me :/