From 362c2a6df6ebf32d9a09e318613e98c3e00ac020 Mon Sep 17 00:00:00 2001 From: Jaggob <37583151+Jaggob@users.noreply.github.com> Date: Tue, 9 Jun 2026 11:47:28 +0200 Subject: [PATCH] docs(e2e): reframe the README intro and tidy issue references - Intro now frames the suite as a complement to the PHPUnit/vitest unit tests that walks through the real user/admin browser flows, instead of "the flows vitest can't cover"; drop the #54 pointer from the headline. - Make the #112 (Docker CI target) reference a clickable link. - Drop the (#26) provenance tag from the pad-template-placeholders bullet. --- tests/e2e/README.md | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/tests/e2e/README.md b/tests/e2e/README.md index 5d4a9c3..6ace789 100644 --- a/tests/e2e/README.md +++ b/tests/e2e/README.md @@ -2,9 +2,11 @@ SPDX-License-Identifier: AGPL-3.0-or-later -Browser-driven tests for the flows that vitest + happy-dom can't cover: -NewFileMenu create, viewer mount inside NC's modal, public-share -interception, trashbin lifecycle, cross-user sharing. See issue #54. +Browser-driven tests that complement our PHPUnit and vitest unit suites: +they drive a real Nextcloud + Etherpad and walk through the flows users +and admins actually perform in the browser — creating and opening pads +from the Files UI, templates, trash and restore, sharing, public-share +access, and the admin health check. ## What it talks to @@ -12,7 +14,8 @@ The specs are **target-agnostic** — they drive whatever Nextcloud instance `E2E_BASE_URL` points at. For local development that's easiest against an existing test instance (your own NC, or the shared test server). A reproducible Docker NC+Etherpad target for CI is a later -phase (#112); because the specs only depend on `E2E_BASE_URL` and the +phase ([#112](https://github.com/Jaggob/etherpad-integration-for-nextcloud/issues/112)); +because the specs only depend on `E2E_BASE_URL` and the documented env vars, adding it won't require rewriting tests. > Use a **dedicated throwaway test account**. The specs create and delete @@ -78,7 +81,7 @@ Each `specs/*.spec.ts` covers one flow: - **pad-author-display-name** — protected pad opens with the NC account's display name visible in Etherpad's user list. - **pad-template-placeholders** — `{{date}}` / `{{user}}` substitution - when creating from a Templates-folder `.pad` (#26). + when creating from a Templates-folder `.pad`. - **pad-move-rename** — the binding (keyed on file id) survives an in-place rename and a move into a subfolder. - **pad-orphan-recovery** — a binding-less `.pad` (WebDAV copy) shows the