Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

It looks like the loading editor stealed the focus
just when cypress was about to start typing.

This test usually passed on the second or third attempt
but left behind additional pages
that would cause the page list test to fail.

@max-nextcloud max-nextcloud requested a review from mejo- as a code owner September 1, 2025 09:12
@max-nextcloud
Copy link
Collaborator Author

Looks like the link bubble view steals the focus from the title entry when initialized:
image

Somehow i can only reproduce this in cypress with electron right now.

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud force-pushed the chore/fix-cypress-tests branch from 81ec783 to 37c9687 Compare September 1, 2025 19:33
@max-nextcloud
Copy link
Collaborator Author

The actual fix happened here: nextcloud/text#7599

Still dropping a suprefluous describe and making sure the page actually gets the name we use in the following tests seem reasonable.

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.

Thanks!

@mejo- mejo- merged commit 1d5770e into main Sep 8, 2025
52 checks passed
@mejo- mejo- deleted the chore/fix-cypress-tests branch September 8, 2025 10:33
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.

3 participants