[stable31] Migrate to playwright to update cypress by backportbot[bot] · Pull Request #8230 · nextcloud/text

backportbot · 2026-02-01T10:45:17Z

Backport of #8223

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

Review and resolve any conflicts
Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

I was sick of debugging flaky Cypress tests, so I gave Playwright a try. Signed-off-by: Jonas <jonas@freesources.org>

Signed-off-by: Max <max@nextcloud.com>

Preparing for multi context scenarios such as shared files. Signed-off-by: Max <max@nextcloud.com>

Signed-off-by: Max <max@nextcloud.com>

It is included in uploadFileTest and sharedFileTest Signed-off-by: Max <max@nextcloud.com>

Signed-off-by: Max <max@nextcloud.com>

Create a new instance on move to handle the name change Signed-off-by: Max <max@nextcloud.com>

For parameter properties see https://www.typescriptlang.org/docs/handbook/2/classes.html#parameter-properties For the lint exception see https://typescript-eslint.io/rules/no-useless-constructor/ Signed-off-by: Max <max@nextcloud.com>

Include a host identifier in the test name so the screenshot name also includes it. This way screenshots will only be compared to the ones created on the same machine. Only track the CI ones in git. Ignore the others. > Having different screenshots between local and docker environments is expected. > You can only compare screenshots that are taken in the same exact environment, OS, fonts, etc. See microsoft/playwright#20366 (comment) Signed-off-by: Max <max@nextcloud.com>

Signed-off-by: Max <max@nextcloud.com>

Signed-off-by: Jonas <jonas@freesources.org>

…acter Signed-off-by: Jonas <jonas@freesources.org>

Signed-off-by: Max <max@nextcloud.com>

There are no buttons that we would need to leave room for. This way the print view looks the same between editable and readonly shares. Signed-off-by: Max <max@nextcloud.com>

Signed-off-by: Max <max@nextcloud.com>

We have a playwright test that does the same now. Signed-off-by: Max <max@nextcloud.com>

codecov · 2026-02-01T11:22:59Z

Codecov Report

❌ Patch coverage is 0% with 397 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
playwright/support/fixtures/Node.ts	0.00%	80 Missing and 1 partial ⚠️
playwright/support/sections/EditorSection.ts	0.00%	52 Missing and 1 partial ⚠️
playwright/support/fixtures/shared-file.ts	0.00%	49 Missing and 1 partial ⚠️
playwright/start-nextcloud-server.mjs	0.00%	38 Missing and 1 partial ⚠️
playwright.config.ts	0.00%	34 Missing and 1 partial ⚠️
playwright/support/fixtures/offline.ts	0.00%	29 Missing and 1 partial ⚠️
playwright/support/fixtures/random-user.ts	0.00%	24 Missing and 1 partial ⚠️
playwright/support/sections/ViewerSection.ts	0.00%	23 Missing and 1 partial ⚠️
playwright/support/fixtures/upload-file.ts	0.00%	17 Missing and 1 partial ⚠️
playwright/support/fixtures/User.ts	0.00%	13 Missing and 1 partial ⚠️
... and 4 more

📢 Thoughts on this report? Let us know!

Signed-off-by: Max <max@nextcloud.com>

max-nextcloud · 2026-02-01T11:26:13Z

Interesting... cypress seems to crash on almost all tests here... 🤷

backportbot bot requested review from max-nextcloud and mejo- February 1, 2026 10:45

backportbot bot added the 3. to review label Feb 1, 2026

backportbot bot added this to the Nextcloud 31.0.14 milestone Feb 1, 2026

backportbot bot assigned max-nextcloud Feb 1, 2026

mejo- and others added 25 commits February 1, 2026 12:03

chore(test): Migrate cypress offline tests to playwright

88d4b7f

I was sick of debugging flaky Cypress tests, so I gave Playwright a try. Signed-off-by: Jonas <jonas@freesources.org>

chore(test): split setOnline and setOffline

f42e773

Signed-off-by: Max <max@nextcloud.com>

chore(test): add offline fixture to playwright

abd2530

Signed-off-by: Max <max@nextcloud.com>

chore(test): get request token from /csrftoken

0343bb5

Signed-off-by: Max <max@nextcloud.com>

chore(test): split request token fixture from random user

3e1456a

Signed-off-by: Max <max@nextcloud.com>

chore(test): cleanup unused type and context

ebaa4fc

Signed-off-by: Max <max@nextcloud.com>

chore(test): mimetype change in playwright

37a2dc6

Signed-off-by: Max <max@nextcloud.com>

chore(test): basic editor page object model

a2e10ef

Signed-off-by: Max <max@nextcloud.com>

chore(test): file class for handling uploaded file

5194f1a

Signed-off-by: Max <max@nextcloud.com>

chore(test): file.open()

0346d74

Signed-off-by: Max <max@nextcloud.com>

chore(test): with file.move() and file.close()

d90e16d

Signed-off-by: Max <max@nextcloud.com>

chore(test): with editor.typeHeading()

a875af0

Signed-off-by: Max <max@nextcloud.com>

chore(test): change mimetype both ways

1395840

Signed-off-by: Max <max@nextcloud.com>

chore(test): move File class to separate file

63265bc

Signed-off-by: Max <max@nextcloud.com>

chore(test): use EditorSection in offline test

9b78608

Signed-off-by: Max <max@nextcloud.com>

chore(test) .withOpenMenu() to access submenus

4efa6d8

Signed-off-by: Max <max@nextcloud.com>

chore(test): write offline and come back online

49e9550

Signed-off-by: Max <max@nextcloud.com>

chore(test): reduce nesting in offline test

6befe7d

Signed-off-by: Max <max@nextcloud.com>

chore(test): wait for close request when closing

f8b0607

Signed-off-by: Max <max@nextcloud.com>

chore(test): migrate mobile spec from cy to playwright

ec36494

Signed-off-by: Max <max@nextcloud.com>

chore(test): convert smart picker test to playwright

c005166

Signed-off-by: Max <max@nextcloud.com>

chore(test): convert print spec to playwright

597cc03

Signed-off-by: Max <max@nextcloud.com>

chore(test): new File() with params object

3f25b26

Signed-off-by: Max <max@nextcloud.com>

chore(test): handle request token as part of login

f76fd47

Signed-off-by: Max <max@nextcloud.com>

chore(test): User class to wrap Account and Page

740edf7

Preparing for multi context scenarios such as shared files. Signed-off-by: Max <max@nextcloud.com>

max-nextcloud and others added 19 commits February 1, 2026 12:04

chore(test): print with shared file

cadde9b

Signed-off-by: Max <max@nextcloud.com>

chore(test): no need to merge randomUserTest

58eaddc

It is included in uploadFileTest and sharedFileTest Signed-off-by: Max <max@nextcloud.com>

chore(test): rename editor.locator to editor.el

d514549

Signed-off-by: Max <max@nextcloud.com>

chore(test): make File attributes read only

7f1e50a

Create a new instance on move to handle the name change Signed-off-by: Max <max@nextcloud.com>

chore(test): use parameter properties

60b6b39

For parameter properties see https://www.typescriptlang.org/docs/handbook/2/classes.html#parameter-properties For the lint exception see https://typescript-eslint.io/rules/no-useless-constructor/ Signed-off-by: Max <max@nextcloud.com>

chore(test): add versions spec

155c409

Signed-off-by: Max <max@nextcloud.com>

chore(test): versions with ViewerSection

e204aab

Signed-off-by: Max <max@nextcloud.com>

chore(test): move close() into ViewerSection

1f48a6b

Signed-off-by: Max <max@nextcloud.com>

chore(test): convert version spec from cy to playwright

c0c7ccf

Signed-off-by: Max <max@nextcloud.com>

chore(test): folder description with playwright

5062a99

Signed-off-by: Max <max@nextcloud.com>

chore(playwright): Fix prettier warnings

152dc85

Signed-off-by: Jonas <jonas@freesources.org>

test(playwright): Add regression test for CJK IME input as first char…

117e115

…acter Signed-off-by: Jonas <jonas@freesources.org>

chore(test): format text with playwright

05c4637

Signed-off-by: Max <max@nextcloud.com>

fix(css): full width table in print view

1b4e5bc

There are no buttons that we would need to leave room for. This way the print view looks the same between editable and readonly shares. Signed-off-by: Max <max@nextcloud.com>

chore(deps): update dependency cypress to v15

0926432

Signed-off-by: Max <max@nextcloud.com>

chore(cleanup): unused cypress component.js

360b056

Signed-off-by: Max <max@nextcloud.com>

chore(test): do not run playwright test with vitest

2881de3

Signed-off-by: Max <max@nextcloud.com>

chore(test): remove superfluous cy print spec

f01177d

We have a playwright test that does the same now. Signed-off-by: Max <max@nextcloud.com>

max-nextcloud force-pushed the backport/8223/stable31 branch from fe3b15d to f01177d Compare February 1, 2026 11:19

chore(lint): fix playwright to match stable31 style

9afd996

Signed-off-by: Max <max@nextcloud.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[stable31] Migrate to playwright to update cypress#8230

[stable31] Migrate to playwright to update cypress#8230
backportbot[bot] wants to merge 45 commits intostable31from
backport/8223/stable31

backportbot bot commented Feb 1, 2026 •

edited by max-nextcloud

Loading

Uh oh!

codecov bot commented Feb 1, 2026

Uh oh!

max-nextcloud commented Feb 1, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

backportbot bot commented Feb 1, 2026 • edited by max-nextcloud Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Todo

Uh oh!

codecov bot commented Feb 1, 2026

Codecov Report

Uh oh!

max-nextcloud commented Feb 1, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

backportbot bot commented Feb 1, 2026 •

edited by max-nextcloud

Loading