Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(cypress): Avoid failure when editor is updated on webdav requests #5772

Merged
merged 1 commit into from
May 8, 2024

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented May 6, 2024

This should make the propfind tests more stable.

We manually upload different file content through webdav which might cause the current editor to throw an exception when the file is deleted.

In the end we don't really need the page context for the tests, however as we use the Nextcloud dav client, asserting propfind responses is a lot easier. Mid-term we could migrate to just use axios or cy.request for those.

Example failure can be seen in https://github.com/nextcloud/text/actions/runs/8952598767/job/24590191037?pr=5762.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl added tests If you write them we ♥ you 3. to review labels May 6, 2024
@max-nextcloud max-nextcloud merged commit 7fb4119 into main May 8, 2024
60 checks passed
@max-nextcloud max-nextcloud deleted the cypress/propfind-stable branch May 8, 2024 09:06
@max-nextcloud
Copy link
Collaborator

/backport to stable29

@max-nextcloud

This comment was marked as outdated.

This comment was marked as resolved.

@max-nextcloud
Copy link
Collaborator

/backport to stable28

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