Skip to content

[stable32] test(playwright): run playwright on stable3* branches#8365

Merged
max-nextcloud merged 5 commits intostable32from
backport/8362/stable32
Mar 19, 2026
Merged

[stable32] test(playwright): run playwright on stable3* branches#8365
max-nextcloud merged 5 commits intostable32from
backport/8362/stable32

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Mar 18, 2026

Backport of PR #8362

@backportbot backportbot bot requested a review from max-nextcloud March 18, 2026 11:45
@backportbot backportbot bot added tests If you write them we ♥ you 3. to review labels Mar 18, 2026
@backportbot backportbot bot added this to the Nextcloud 32.0.7 milestone Mar 18, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 2.50000% with 39 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
playwright/start-nextcloud-server.mjs 0.00% 26 Missing and 1 partial ⚠️
playwright.config.ts 0.00% 8 Missing ⚠️
src/components/Menu/ActionInsertLink.vue 20.00% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@mejo-
Copy link
Copy Markdown
Member

mejo- commented Mar 18, 2026

Lots of failing Playwright tests that will need investigation.

@max-nextcloud
Copy link
Copy Markdown
Collaborator

I suspect it's due to selectors that changed in the meantime.

@max-nextcloud max-nextcloud force-pushed the backport/8362/stable32 branch from c1c5ef9 to ea9881b Compare March 19, 2026 09:21
@max-nextcloud
Copy link
Copy Markdown
Collaborator

viewer was not installed, because configure nextcloud was not getting called. Cherry picked the updated approach from main.

mejo- and others added 3 commits March 19, 2026 14:44
Signed-off-by: Jonas <jonas@freesources.org>
Fixed by syncing the start script with what end_to_end encryption uses.
Also renders the setup steps superfluous.

Signed-off-by: Max <max@nextcloud.com>
This provides the `wait` option added to the config in last commit.

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud force-pushed the backport/8362/stable32 branch from d3b02c1 to ad7f474 Compare March 19, 2026 13:46
mejo- and others added 2 commits March 19, 2026 16:35
Fixes: #1868

Signed-off-by: Jonas <jonas@freesources.org>
The class `viewer__file--active` is not set anymore on the current version.

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud force-pushed the backport/8362/stable32 branch from ad7f474 to d3a4312 Compare March 19, 2026 15:35
@max-nextcloud max-nextcloud merged commit fbefb88 into stable32 Mar 19, 2026
70 of 71 checks passed
@max-nextcloud max-nextcloud deleted the backport/8362/stable32 branch March 19, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review tests If you write them we ♥ you

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants