test: make some tests as flaky (#3672)

This commit is contained in:
Pavel Feldman 2020-08-28 10:03:52 -07:00 committed by GitHub
parent 2edd6f2824
commit eec92630df
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 5 additions and 5 deletions

View File

@ -111,7 +111,7 @@ it.slow()('should restore state from userDataDir', async ({browserType, defaultB
await removeUserDataDir(userDataDir2);
});
it.slow()('should restore cookies from userDataDir', async ({browserType, defaultBrowserOptions, server, launchPersistent}) => {
it.slow().flaky(options.CHROMIUM && WIN)('should restore cookies from userDataDir', async ({browserType, defaultBrowserOptions, server, launchPersistent}) => {
const userDataDir = await makeUserDataDir();
const browserContext = await browserType.launchPersistentContext(userDataDir, defaultBrowserOptions);
const page = await browserContext.newPage();

View File

@ -32,7 +32,7 @@ it.fail(options.WIRE)('should emit crash event when page crashes', async ({page,
await new Promise(f => page.on('crash', f));
});
it.fail(options.WIRE)('should throw on any action after page crashes', async ({page, browserName, toImpl}) => {
it.fail(options.WIRE).flaky(options.FIREFOX && WIN)('should throw on any action after page crashes', async ({page, browserName, toImpl}) => {
await page.setContent(`<div>This page should crash</div>`);
crash(toImpl(page), browserName);
await page.waitForEvent('crash');
@ -59,7 +59,7 @@ it.fixme(options.WIRE)('should cancel navigation when page crashes', async ({pag
expect(error.message).toContain('Navigation failed because page crashed');
});
it.fixme(options.WIRE)('should be able to close context when page crashes', async ({page, browserName, toImpl}) => {
it.fixme(options.WIRE).flaky(options.FIREFOX && WIN)('should be able to close context when page crashes', async ({page, browserName, toImpl}) => {
await page.setContent(`<div>This page should crash</div>`);
crash(toImpl(page), browserName);
await page.waitForEvent('crash');

View File

@ -98,7 +98,7 @@ describe.skip(options.WEBKIT)('permissions', () => {
expect(await getPermission(page, 'geolocation')).toBe('prompt');
});
it.fail(options.WEBKIT || (options.CHROMIUM && !options.HEADLESS))('should trigger permission onchange', async ({page, server, context}) => {
it.fail(options.WEBKIT).fail(options.CHROMIUM && !options.HEADLESS).flaky(options.FIREFOX && LINUX)('should trigger permission onchange', async ({page, server, context}) => {
// TODO: flaky
// - Linux: https://github.com/microsoft/playwright/pull/1790/checks?check_run_id=587327883
// - Win: https://ci.appveyor.com/project/aslushnikov/playwright/builds/32402536

View File

@ -172,7 +172,7 @@ class VideoPlayer {
}
}
it.fixme(options.CHROMIUM)('should capture static page', async ({page, tmpDir, videoPlayer, toImpl}) => {
it.fixme(options.CHROMIUM).flaky(options.FIREFOX && MAC)('should capture static page', async ({page, tmpDir, videoPlayer, toImpl}) => {
if (!toImpl)
return;
const videoFile = path.join(tmpDir, 'v.webm');