Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

feat(webkit): roll to r1562 (#9476)

Browse source 
Co-authored-by: Max Schmitt <max@schmitt.mx>
This commit is contained in:
github-actions[bot] 2021-10-14 02:00:25 +02:00 committed by GitHub
parent 1a69b26db1
commit c054d19a5d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/playwright-core/browsers.json
View file

@ -23,7 +23,7 @@
}, },
{ {
"name": "webkit", "name": "webkit",
"revision": "1550", "revision": "1562",
"installByDefault": true, "installByDefault": true,
"revisionOverrides": { "revisionOverrides": {
"mac10.14": "1446" "mac10.14": "1446"

3
tests/page/page-request-fulfill.spec.ts
View file

@ -251,7 +251,6 @@ it('should fetch original request and fulfill', async ({ page, server, isElectro
it('should fulfill with multiple set-cookie', async ({ page, server, browserName, isElectron }) => { it('should fulfill with multiple set-cookie', async ({ page, server, browserName, isElectron }) => {
it.fixme(isElectron, 'Electron 14+ is required'); it.fixme(isElectron, 'Electron 14+ is required');
it.fail(browserName === 'webkit', 'Response contained invalid HTTP headers');
const cookies = ['a=b', 'c=d']; const cookies = ['a=b', 'c=d'];
await page.route('**/empty.html', async route => { await page.route('**/empty.html', async route => {
route.fulfill({ route.fulfill({
@ -271,7 +270,6 @@ it('should fulfill with multiple set-cookie', async ({ page, server, browserName
}); });
it('should fulfill with fetch response that has multiple set-cookie', async ({ playwright, page, server, browserName }) => { it('should fulfill with fetch response that has multiple set-cookie', async ({ playwright, page, server, browserName }) => {
it.fail(browserName === 'webkit', 'Response contained invalid HTTP headers');
server.setRoute('/empty.html', (req, res) => { server.setRoute('/empty.html', (req, res) => {
res.setHeader('Set-Cookie', ['a=b', 'c=d']); res.setHeader('Set-Cookie', ['a=b', 'c=d']);
res.setHeader('Content-Type', 'text/html'); res.setHeader('Content-Type', 'text/html');
@ -289,6 +287,7 @@ it('should fulfill with fetch response that has multiple set-cookie', async ({ p
it('headerValue should return set-cookie from intercepted response', async ({ page, server, browserName }) => { it('headerValue should return set-cookie from intercepted response', async ({ page, server, browserName }) => {
it.fail(browserName === 'chromium', 'Set-Cookie is missing in response after interception'); it.fail(browserName === 'chromium', 'Set-Cookie is missing in response after interception');
it.fixme(browserName === 'webkit', 'Set-Cookie with \n in intercepted response does not pass validation in WebCore, see also https://github.com/microsoft/playwright/pull/9273');
await page.route('**/empty.html', async route => { await page.route('**/empty.html', async route => {
route.fulfill({ route.fulfill({
status: 200, status: 200,