5 commits

Author	SHA1	Message	Date
Julian Schlarb	68d690b6b9	chore(e2e): simplify authentication setup (#6400) ... Replaced manual login and context loading across tests with Playwright's `test.use` configuration for user authentication. This simplifies test setup, improves readability, and reduces repetition. For #6362 Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/6400 Reviewed-by: Gusted <gusted@noreply.codeberg.org> Co-authored-by: Julian Schlarb <julian.schlarb@denktmit.de> Co-committed-by: Julian Schlarb <julian.schlarb@denktmit.de>	2025-01-05 05:17:04 +00:00
Otto Richter	c67d63d88a	Fix issue/comment menus ... Closes https://codeberg.org/forgejo/forgejo/issues/1120 - Adds labels to reaction and context menu. - Fixes taborder in markdown combobox buttons. They are now only one "tab" stop and can be navigated with arrow buttons and in the right order (previously, it would skip the table button). - Generates more verbose output for the reactio selectors to provide content for users who cannot identify the meaning of these buttons visually. Explicit aria-labels are now preferred over auto-generated ones.	2024-12-30 16:06:18 +01:00
Otto Richter	840605cc75	tests(e2e): Generate more screenshots ... - create screenshots on tests that create a custom page - skip screenshot on a page that uses the default page (it is added automatically and already exists) - replace a useless response wait and assert the correct page style instead	2024-12-10 18:12:51 +01:00
Otto Richter	40551de313	tests(e2e): Refactor various tests ... Goals: - speedup - less flakiness - best practices and more use - documentation config: - sync ports in Makefile and playwright config (otherwise, some tests fail locally because they assert the full URL including the (wrong) port) - even more generous timeouts - limit workers to one again (because I finally understand how Playwright works) - allow nested functions to group them together with the related test all: - deprecate waitForLoadState('networkidle') - it is discouraged as per https://playwright.dev/docs/api/class-page#page-wait-for-load-state - I could not find a usage that seems to require it actually (see added documentation in README) - adding an exception should be made explicitly - it does not do what you might expect anyway in most cases - only log in when necessary webauthn: - verify that login is possible after disabling key - otherwise, the cleanup was not necessary after the previous refactor to create a fresh user each issue-sidebar / WIP toggle: - split into smaller chunks - restore original state first - add missed assertion to fix race condition (not waiting before state was reached) - explicitly toggle the state to detect mismatch earlier issue-sidebar / labels: - restore original state first - better waiting for background request	2024-11-13 13:15:37 +01:00
Anbraten	3a3416c069	Migrate playwright to typescript	2024-11-08 09:55:54 +01:00

Renamed from tests/e2e/issue-comment.test.e2e.js (Browse further)