Skip to content

Commit

Permalink
test: update webview2 skipme's (#32984)
Browse files Browse the repository at this point in the history
This leaves only reasonable skipmes for WebView2 in the code base.
  • Loading branch information
mxschmitt authored Oct 7, 2024
1 parent 317ddd4 commit 001ba37
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 11 deletions.
4 changes: 2 additions & 2 deletions tests/page/locator-misc-2.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ it('should scroll into view', async ({ page, server, isAndroid }) => {
}
});

it('should scroll zero-sized element into view', async ({ page, isAndroid, isElectron, isWebView2, browserName, isMac, macVersion }) => {
it.fixme(isAndroid || isElectron || isWebView2);
it('should scroll zero-sized element into view', async ({ page, isAndroid, isElectron, browserName, isMac, macVersion }) => {
it.fixme(isAndroid || isElectron);
it.skip(browserName === 'webkit' && isMac && macVersion < 11, 'WebKit for macOS 10.15 is frozen.');

await page.setContent(`
Expand Down
6 changes: 2 additions & 4 deletions tests/page/page-accessibility.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -143,9 +143,8 @@ it('should not report text nodes inside controls', async function({ page, browse
expect(await page.accessibility.snapshot()).toEqual(golden);
});

it('rich text editable fields should have children', async function({ page, browserName, browserVersion, isWebView2 }) {
it('rich text editable fields should have children', async function({ page, browserName, browserVersion }) {
it.skip(browserName === 'webkit', 'WebKit rich text accessibility is iffy');
it.skip(isWebView2, 'WebView2 is missing a Chromium fix');

await page.setContent(`
<div contenteditable="true">
Expand Down Expand Up @@ -177,9 +176,8 @@ it('rich text editable fields should have children', async function({ page, brow
expect(snapshot.children[0]).toEqual(golden);
});

it('rich text editable fields with role should have children', async function({ page, browserName, browserVersion, isWebView2 }) {
it('rich text editable fields with role should have children', async function({ page, browserName, browserVersion }) {
it.skip(browserName === 'webkit', 'WebKit rich text accessibility is iffy');
it.skip(isWebView2, 'WebView2 is missing a Chromium fix');

await page.setContent(`
<div contenteditable="true" role='textbox'>
Expand Down
3 changes: 1 addition & 2 deletions tests/page/page-click-scroll.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,9 +78,8 @@ it('should scroll into view display:contents with position', async ({ page, brow
expect(await page.evaluate('window._clicked')).toBe(true);
});

it('should not crash when force-clicking hidden input', async ({ page, isWebView2 }) => {
it('should not crash when force-clicking hidden input', async ({ page }) => {
it.info().annotations.push({ type: 'issue', description: 'https://github.com/microsoft/playwright/issues/18183' });
it.fixme(isWebView2);

await page.setContent(`<input type=hidden>`);
const error = await page.locator('input').click({ force: true, timeout: 2000 }).catch(e => e);
Expand Down
3 changes: 1 addition & 2 deletions tests/page/page-screenshot.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -291,10 +291,9 @@ it.describe('page screenshot', () => {
expect(screenshot).toMatchSnapshot('screenshot-canvas.png');
});

it('should capture canvas changes', async ({ page, isElectron, browserName, isMac, isWebView2 }) => {
it('should capture canvas changes', async ({ page, isElectron, browserName, isMac }) => {
it.fixme(browserName === 'webkit' && isMac, 'https://github.com/microsoft/playwright/issues/8796,https://github.com/microsoft/playwright/issues/16180');
it.skip(isElectron);
it.skip(isWebView2);
await page.goto('data:text/html,<canvas></canvas>');
await page.evaluate(() => {
const canvas = document.querySelector('canvas');
Expand Down
2 changes: 1 addition & 1 deletion tests/webview2/webview2-app/webview2.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Microsoft.Web.WebView2" Version="1.0.2210.55" />
<PackageReference Include="Microsoft.Web.WebView2" Version="1.0.2792.45" />
</ItemGroup>

</Project>

0 comments on commit 001ba37

Please sign in to comment.