mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2024-11-28 09:21:13 -05:00
c9e402afdc
Modifies forms: - (new) org team - (new) repo webhook - (new) repo protected branch The forms are not completely rewritten to semantic HTML yet. The focus of this change was on standard elements, some custom solutions were left untouched for now. - swaps the order fo permission radio buttons as per https://codeberg.org/forgejo/forgejo/issues/4983 - uses fieldsets to group related inputs - ensures consistent styling across forms - can be improved later, e.g. using horizontal lines between sections - fixes: previous font size of labels was smaller than the font size of the help text - help text are now part of the label, clicking them now also activates the input - drop unused CSS (no required checkboxes in grouped class remain) - playwright testing: - move login boilerplate to utils - automated form accessibility checking - allow defining the scope, because legacy parts of the forms are not yet accessible - assert some CSS properties that should not be overriden - the Makefile adjustment was necessary, because eslint scanned some internal files in the tests/e2e/reports directory
37 lines
1.6 KiB
JavaScript
37 lines
1.6 KiB
JavaScript
// @ts-check
|
|
import {expect} from '@playwright/test';
|
|
import {test, login_user, login} from './utils_e2e.js';
|
|
import {validate_form} from './shared/forms.js';
|
|
|
|
test.beforeAll(async ({browser}, workerInfo) => {
|
|
await login_user(browser, workerInfo, 'user2');
|
|
});
|
|
|
|
test('repo webhook settings', async ({browser}, workerInfo) => {
|
|
test.skip(workerInfo.project.name === 'Mobile Safari', 'Cannot get it to work - as usual');
|
|
const page = await login({browser}, workerInfo);
|
|
const response = await page.goto('/user2/repo1/settings/hooks/forgejo/new');
|
|
await expect(response?.status()).toBe(200);
|
|
|
|
await page.locator('input[name="events"][value="choose_events"]').click();
|
|
await expect(page.locator('.events.fields')).toBeVisible();
|
|
|
|
await page.locator('input[name="events"][value="push_only"]').click();
|
|
await expect(page.locator('.events.fields')).toBeHidden();
|
|
await page.locator('input[name="events"][value="send_everything"]').click();
|
|
await expect(page.locator('.events.fields')).toBeHidden();
|
|
|
|
// restrict to improved semantic HTML, the rest of the page fails the accessibility check
|
|
// only execute when the ugly part is hidden - would benefit from refactoring, too
|
|
await validate_form({page}, 'fieldset');
|
|
});
|
|
|
|
test('repo branch protection settings', async ({browser}, workerInfo) => {
|
|
test.skip(workerInfo.project.name === 'Mobile Safari', 'Cannot get it to work - as usual');
|
|
const page = await login({browser}, workerInfo);
|
|
const response = await page.goto('/user2/repo1/settings/branches/edit');
|
|
await expect(response?.status()).toBe(200);
|
|
|
|
// not yet accessible :(
|
|
// await validate_form({page}, 'fieldset');
|
|
});
|