From a1adc15ea3e4bbca0c7a85c941708a7e7665831f Mon Sep 17 00:00:00 2001 From: Pavel Feldman Date: Mon, 13 Sep 2021 20:13:25 -0700 Subject: [PATCH] tests: mark win32/webkit cookie multivalue test as failing --- tests/page/page-network-response.spec.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/page/page-network-response.spec.ts b/tests/page/page-network-response.spec.ts index 0aa240fb3f..404c629cc7 100644 --- a/tests/page/page-network-response.spec.ts +++ b/tests/page/page-network-response.spec.ts @@ -30,7 +30,8 @@ it('should work', async ({page, server}) => { expect((await response.allHeaders())['BaZ']).toBe(undefined); }); -it('should return multiple header value', async ({page, server}) => { +it('should return multiple header value', async ({page, server, browserName, platform}) => { + it.fixme(browserName === 'webkit' && platform === 'win32', 'libcurl does not support non-set-cookie multivalue headers'); server.setRoute('/headers', (req, res) => { // Headers array is only supported since Node v14.14.0 so we write directly to the socket. // res.writeHead(200, ['name-a', 'v1','name-b', 'v4','Name-a', 'v2', 'name-A', 'v3']); @@ -233,7 +234,8 @@ it('should report multiple set-cookie headers', async ({ page, server }) => { expect(await response.headerValues('set-cookie')).toEqual(['a=b', 'c=d']); }); -it('should behave the same way for headers and allHeaders', async ({ page, server, browserName, channel }) => { +it('should behave the same way for headers and allHeaders', async ({ page, server, browserName, channel, platform }) => { + it.fixme(browserName === 'webkit' && platform === 'win32', 'libcurl does not support non-set-cookie multivalue headers'); it.skip(!!channel, 'Stable chrome uses \n as a header separator in non-raw headers'); server.setRoute('/headers', (req, res) => { const headers = {