Skip to content

chore(deps): setup Playwright for screenshots and e2e tests - #15977

Open
Antreesy wants to merge 7 commits into
mainfrom
fix/noid/playwright
Open

Antreesy wants to merge 7 commits into
mainfrom
fix/noid/playwright

Conversation

@Antreesy

@Antreesy Antreesy commented Sep 19, 2025 •

Copy link
Copy Markdown
Contributor

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

Admin settings:
image

Main app:
image

Room details:
image

Test chats:
image

Docs screnshot run:
image

🚧 Tasks

  • package.json script
  • CI .yml runner
  • mocked data (conversations, messages, users) for screenshots
  • media devices?
  • HPB setup?

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@Antreesy Antreesy added the tests label Aug 24, 2026
@Antreesy
Antreesy marked this pull request as ready for review August 24, 2026 12:05
@Antreesy
Antreesy force-pushed the fix/noid/playwright branch from f76aa4d to e7acc0e Compare August 24, 2026 12:08
@Antreesy

This comment was marked as resolved.

@Antreesy

This comment was marked as resolved.

susnux

This comment was marked as resolved.

@DorraJaouad

This comment was marked as resolved.

@susnux

This comment was marked as resolved.

@Antreesy

This comment was marked as resolved.

@DorraJaouad

This comment was marked as resolved.

@Antreesy

This comment was marked as resolved.

- @playwright/test itself
- @nextcloud/e2e-test-server for Nextcloud specific utils

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- replicated from files_antivirus app and adjusted
- yml copied from nextcloud/.github#806

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- user generator
- main app page test
- admin page test

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- concurrent requests can clash in parallel Playwright workers, as they are targeting the same database
- implement minimal db lock to work around it

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- room name
- room description
- room avatar

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
The `docs` project runs `@docs`-tagged e2e specs in docs mode:
sessions log in as demo users, and `docsScreenshot()` writes files.
Run with `npm run playwright:docs`.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the fix/noid/playwright branch from 245f799 to 3b7ff5a Compare October 7, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file feature: frontend 🖌️ "Web UI" client tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants