Skip to content

Commit

Permalink
cherry-pick(#17879): test: rebaseline requestStorageAccess test for f…
Browse files Browse the repository at this point in the history
…irefox-beta (#17884)
  • Loading branch information
dgozman authored Oct 6, 2022
1 parent 3bf42ce commit 03c7f9c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tests/library/browsercontext-cookies.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -350,9 +350,11 @@ it('should be able to send third party cookies via an iframe', async ({ browser,
}
});

it('should support requestStorageAccess', async ({ page, server, browserName, isMac, isLinux, isWindows }) => {
it('should support requestStorageAccess', async ({ page, server, channel, browserName, isMac, isLinux, isWindows }) => {
it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/17285' });
it.skip(browserName === 'chromium', 'requestStorageAccess API is not available in Chromium');
it.fixme(channel === 'firefox-beta', 'hasStorageAccess returns true, but no cookie is sent');

server.setRoute('/set-cookie.html', (req, res) => {
res.setHeader('Set-Cookie', 'name=value; Path=/');
res.end();
Expand All @@ -372,7 +374,6 @@ it('should support requestStorageAccess', async ({ page, server, browserName, is
]);
expect(serverRequest.headers.cookie).toBe('name=value');
}
return;
} else {
if (isLinux && browserName === 'webkit')
expect(await frame.evaluate(() => document.hasStorageAccess())).toBeTruthy();
Expand Down

0 comments on commit 03c7f9c

Please sign in to comment.