Revert "chore: more tree gardening (#23119)"

This reverts commit e6bc32b022.
This commit is contained in:
Pavel Feldman 2023-05-23 12:04:44 -07:00
parent 6cce93b697
commit fd75b85510
5 changed files with 2 additions and 9 deletions

View File

@ -368,7 +368,6 @@ test.describe('cli codegen', () => {
});
test('should not clash pages', async ({ page, openRecorder, browserName }) => {
test.fixme(browserName === 'firefox', 'https://github.com/microsoft/playwright/issues/23117');
const recorder = await openRecorder();
const [popup1] = await Promise.all([
page.context().waitForEvent('page'),

View File

@ -577,8 +577,7 @@ test('should show action source', async ({ showTraceViewer }) => {
await expect(page.getByTestId('stack-trace').locator('.list-view-entry.selected')).toHaveText(/doClick.*trace-viewer\.spec\.ts:[\d]+/);
});
test('should follow redirects', async ({ page, runAndTrace, server, asset, browserName }) => {
test.fixme(browserName === 'chromium', 'https://github.com/microsoft/playwright/issues/23115');
test('should follow redirects', async ({ page, runAndTrace, server, asset }) => {
server.setRoute('/empty.html', (req, res) => {
res.writeHead(200, { 'Content-Type': 'text/html' });
res.end(`<div><img id=img src="image.png"></img></div>`);

View File

@ -674,11 +674,10 @@ it.describe('screencast', () => {
expect(files.length).toBe(1);
});
it('should capture full viewport', async ({ browserType, browserName, headless, isWindows, isLinux }, testInfo) => {
it('should capture full viewport', async ({ browserType, browserName, headless, isWindows }, testInfo) => {
it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/22411' });
it.fixme(browserName === 'chromium' && !headless, 'The square is not on the video');
it.fixme(browserName === 'firefox' && isWindows, 'https://github.com/microsoft/playwright/issues/14405');
it.fixme(browserName === 'firefox' && !headless && isLinux, 'https://github.com/microsoft/playwright/issues/23118');
const size = { width: 600, height: 400 };
const browser = await browserType.launch();

View File

@ -52,7 +52,6 @@ test('should use baseURL in request fixture', async ({ runInlineTest, server })
});
test('should stop tracing on requestContext.dispose()', async ({ runInlineTest, server }) => {
test.fixme(process.platform === 'darwin', 'https://github.com/microsoft/playwright/issues/23116');
server.setRoute('/slow', (req, resp) => {
resp.writeHead(200, {
'Content-Type': 'text/plain; charset=utf-8',

View File

@ -20,7 +20,6 @@ import { test, expect, retries } from './ui-mode-fixtures';
test.describe.configure({ mode: 'parallel', retries });
test('should merge trace events', async ({ runUITest, server }) => {
test.fixme(true, 'https://github.com/microsoft/playwright/issues/23114');
const { page } = await runUITest({
'a.test.ts': `
import { test, expect } from '@playwright/test';
@ -52,7 +51,6 @@ test('should merge trace events', async ({ runUITest, server }) => {
});
test('should merge web assertion events', async ({ runUITest }, testInfo) => {
test.fixme(process.platform === 'darwin' || process.platform === 'win32', 'https://github.com/microsoft/playwright/issues/23114');
const { page } = await runUITest({
'a.test.ts': `
import { test, expect } from '@playwright/test';
@ -127,7 +125,6 @@ test('should locate sync assertions in source', async ({ runUITest, server }) =>
});
test('should show snapshots for sync assertions', async ({ runUITest, server }) => {
test.fixme(true, 'https://github.com/microsoft/playwright/issues/23114');
const { page } = await runUITest({
'a.test.ts': `
import { test, expect } from '@playwright/test';