From 39f389eb17463a49f27bb2d3c8c8612a1a0c9d41 Mon Sep 17 00:00:00 2001 From: 0ko <0ko@noreply.codeberg.org> Date: Fri, 10 Oct 2025 15:36:21 +0200 Subject: [PATCH] fix(e2e): disable webkit/safari in tests where they are too falky (#9599) These tests often appear as failing with one failed retry in e2e logs for either browsers. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9599 Reviewed-by: Michael Kriese Reviewed-by: Otto --- tests/e2e/markdown-editor.test.e2e.ts | 4 +++- tests/e2e/profile_actions.test.e2e.ts | 4 +++- tests/e2e/repo-code.test.e2e.ts | 4 +++- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/tests/e2e/markdown-editor.test.e2e.ts b/tests/e2e/markdown-editor.test.e2e.ts index 70e20a97d6..0cebbe7fba 100644 --- a/tests/e2e/markdown-editor.test.e2e.ts +++ b/tests/e2e/markdown-editor.test.e2e.ts @@ -374,7 +374,9 @@ test('Markdown insert table', async ({page}) => { await screenshot(page); }); -test('Markdown insert link', async ({page}) => { +test('Markdown insert link', async ({page}, workerInfo) => { + test.skip(workerInfo.project.name === 'webkit', 'webkit is unreliable in this test'); + const response = await page.goto('/user2/repo1/issues/new'); expect(response?.status()).toBe(200); diff --git a/tests/e2e/profile_actions.test.e2e.ts b/tests/e2e/profile_actions.test.e2e.ts index 5935de5e7f..8f4293d749 100644 --- a/tests/e2e/profile_actions.test.e2e.ts +++ b/tests/e2e/profile_actions.test.e2e.ts @@ -11,7 +11,9 @@ import {screenshot} from './shared/screenshots.ts'; test.use({user: 'user2'}); -test('Follow and block actions', async ({page}) => { +test('Follow and block actions', async ({page}, workerInfo) => { + test.skip(workerInfo.project.name === 'Mobile Safari', 'Mobile Safari is unreliable in this test'); + await page.goto('/user1'); // Check if following and then unfollowing works. diff --git a/tests/e2e/repo-code.test.e2e.ts b/tests/e2e/repo-code.test.e2e.ts index 26ae034b09..2fe3dd9cd3 100644 --- a/tests/e2e/repo-code.test.e2e.ts +++ b/tests/e2e/repo-code.test.e2e.ts @@ -37,7 +37,9 @@ async function assertSelectedLines(page: Page, nums: string[]) { return pageAssertions(); } -test('Line Range Selection', async ({page}) => { +test('Line Range Selection', async ({page}, workerInfo) => { + test.skip(workerInfo.project.name === 'webkit', 'webkit is unreliable in this test'); + const filePath = '/user2/repo1/src/branch/master/README.md?display=source'; const response = await page.goto(filePath);