mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2024-11-25 20:22:45 +01:00
Merge pull request 'tests(e2e): Allow running in parallel' (#5288) from fnetx/e2e-parallel into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5288 Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
This commit is contained in:
commit
192a5ea68b
3 changed files with 8 additions and 12 deletions
|
@ -11,18 +11,14 @@ export default {
|
|||
testDir: './tests/e2e/',
|
||||
testMatch: /.*\.test\.e2e\.js/, // Match any .test.e2e.js files
|
||||
|
||||
/**
|
||||
* Only run one test at a time, running multiple could lead to a inconsistent
|
||||
* database state.
|
||||
*/
|
||||
fullyParallel: false,
|
||||
workers: 1,
|
||||
// you can adjust this value locally to match your machine's power,
|
||||
// or pass `--workers x` to playwright
|
||||
workers: process.env.CI ? 1 : 2,
|
||||
|
||||
/* Maximum time one test can run for. */
|
||||
timeout: 30 * 1000,
|
||||
|
||||
expect: {
|
||||
|
||||
/**
|
||||
* Maximum time expect() should wait for the condition to be met.
|
||||
* For example in `await expect(locator).toHaveText();`
|
||||
|
@ -34,7 +30,7 @@ export default {
|
|||
forbidOnly: Boolean(process.env.CI),
|
||||
|
||||
/* Retry on CI only */
|
||||
retries: process.env.CI ? 2 : 0,
|
||||
retries: process.env.CI ? 1 : 0,
|
||||
|
||||
/* Reporter to use. See https://playwright.dev/docs/test-reporters */
|
||||
reporter: process.env.CI ? 'list' : [['list'], ['html', {outputFolder: 'tests/e2e/reports/', open: 'never'}]],
|
||||
|
|
|
@ -16,6 +16,6 @@ test('Correct link and tooltip', async ({browser}, workerInfo) => {
|
|||
|
||||
const repoStatus = page.locator('.dashboard-repos .repo-owner-name-list > li:nth-child(1) > a:nth-child(2)');
|
||||
|
||||
await expect(repoStatus).toHaveAttribute('href', '/user2/test_workflows/actions');
|
||||
await expect(repoStatus).toHaveAttribute('href', '/user2/test_workflows/actions', {timeout: 10000});
|
||||
await expect(repoStatus).toHaveAttribute('data-tooltip-content', 'Failure');
|
||||
});
|
||||
|
|
|
@ -6,12 +6,12 @@ import {expect} from '@playwright/test';
|
|||
import {test, login_user, load_logged_in_context} from './utils_e2e.js';
|
||||
|
||||
test.beforeAll(async ({browser}, workerInfo) => {
|
||||
await login_user(browser, workerInfo, 'user2');
|
||||
await login_user(browser, workerInfo, 'user40');
|
||||
});
|
||||
|
||||
test('WebAuthn register & login flow', async ({browser}, workerInfo) => {
|
||||
test.skip(workerInfo.project.name !== 'chromium', 'Uses Chrome protocol');
|
||||
const context = await load_logged_in_context(browser, workerInfo, 'user2');
|
||||
const context = await load_logged_in_context(browser, workerInfo, 'user40');
|
||||
const page = await context.newPage();
|
||||
|
||||
// Register a security key.
|
||||
|
@ -45,7 +45,7 @@ test('WebAuthn register & login flow', async ({browser}, workerInfo) => {
|
|||
response = await page.goto('/user/login');
|
||||
await expect(response?.status()).toBe(200);
|
||||
|
||||
await page.getByLabel('Username or email address').fill('user2');
|
||||
await page.getByLabel('Username or email address').fill('user40');
|
||||
await page.getByLabel('Password').fill('password');
|
||||
await page.getByRole('button', {name: 'Sign in'}).click();
|
||||
await page.waitForURL(`${workerInfo.project.use.baseURL}/user/webauthn`);
|
||||
|
|
Loading…
Reference in a new issue