Skip to content

Comments

chore(test): format text with playwright#8001

Merged
max-nextcloud merged 2 commits intomainfrom
chore/convert-cy-to-play
Dec 2, 2025
Merged

chore(test): format text with playwright#8001
max-nextcloud merged 2 commits intomainfrom
chore/convert-cy-to-play

Conversation

@max-nextcloud
Copy link
Collaborator

Summary

Migrating some more tests to playwright.

@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@max-nextcloud max-nextcloud force-pushed the chore/convert-cy-to-play branch 3 times, most recently from e8f11db to 612b380 Compare December 1, 2025 14:53
@max-nextcloud max-nextcloud marked this pull request as ready for review December 1, 2025 14:53
@max-nextcloud max-nextcloud requested a review from mejo- as a code owner December 1, 2025 14:53
@max-nextcloud
Copy link
Collaborator Author

Print spec looks like somehow the table width shrank
grafik

@mejo-
Copy link
Member

mejo- commented Dec 1, 2025

Print spec looks like somehow the table width shrank

The table width in edit mode is smaller than in readonly mode nowadays, as the edit buttons don't exist in readonly mode and we use the available extra space there. That was requested by the design team some months back and I implemented it.

@max-nextcloud
Copy link
Collaborator Author

The table width in edit mode is smaller than in readonly mode nowadays

Nice! That explains it as i just made the share editable - but maybe we should not apply this in print view as no buttons are visible there either.

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>
@max-nextcloud max-nextcloud force-pushed the chore/convert-cy-to-play branch from 3e51cd2 to d78ee67 Compare December 1, 2025 17:34
@max-nextcloud max-nextcloud requested a review from a team December 1, 2025 17:46
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I wonder whether the cypress test "Initializes the workspace without errors" is necessary as I would expect the workspace to not be visible if an error was thrown, but maybe I'm missing something there.

@max-nextcloud
Copy link
Collaborator Author

I wonder whether the cypress test "Initializes the workspace without errors" is necessary as I would expect the workspace to not be visible if an error was thrown, but maybe I'm missing something there.

We are logging some exceptions rather than throwing them. I think this is a regression test for a fix that removed one of these errors.

@max-nextcloud max-nextcloud merged commit 2620bc9 into main Dec 2, 2025
63 checks passed
@max-nextcloud max-nextcloud deleted the chore/convert-cy-to-play branch December 2, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants