Skip to content

test(core): add Playwright coverage for the guest pages - #64994

Merged
susnux merged 1 commit into
masterfrom
worktree-core-vue3-2
Oct 2, 2026
Merged

susnux merged 1 commit into
masterfrom
worktree-core-vue3-2

Conversation

@susnux

@susnux susnux commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

add some e2e tests:

  • the password prompt of protected link shares
  • the login flow v1 and v2 up to the hand-over of the app password
  • the unsupported browser warning, including the redirect back
  • the legacy unified search that is offered while unified_search.enabled is set

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Cover the core pages no end-to-end test exercised so far, so that moving
them to another rendering engine cannot silently break them:

- the password prompt of protected link shares
- the login flow v1 and v2 up to the hand-over of the app password
- the unsupported browser warning, including the redirect back
- the legacy unified search that is offered while
  `unified_search.enabled` is set

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Assisted-by: ClaudeCode:claude-opus-5-5
@susnux
susnux requested a review from a team as a code owner October 1, 2026 16:12
@susnux
susnux requested review from kristian-zendato, skjnldsv and sorbaugh and removed request for a team October 1, 2026 16:12
@susnux susnux added 3. to review Waiting for reviews tests Related to tests labels Oct 1, 2026
@susnux susnux added this to the Nextcloud 36 milestone Oct 1, 2026
@susnux
susnux added this pull request to stack #64996 October 1, 2026 16:14
@susnux
susnux merged commit cf65b57 into master Oct 2, 2026
185 of 188 checks passed
@susnux
susnux deleted the worktree-core-vue3-2 branch October 2, 2026 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants