Skip to content

[main] fix(cy): avoid double upload to prevent lock error#6763

Merged
max-nextcloud merged 1 commit intomainfrom
backport/6760/main
Dec 10, 2024
Merged

[main] fix(cy): avoid double upload to prevent lock error#6763
max-nextcloud merged 1 commit intomainfrom
backport/6760/main

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Dec 10, 2024

Backport of #6760

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.

@backportbot backportbot bot requested review from max-nextcloud and mejo- December 10, 2024 12:51
@backportbot backportbot bot added tests If you write them we ♥ you 3. to review labels Dec 10, 2024
Copy link
Collaborator

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

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

only copyright adjustment was not forward portet - as it's now in a different format all together.

`isolateTest` in `beforeEach` already uploads the file.

Uploading it again in the `no actual conflict - just reload` case

sometimes led to 423 - Locked responses.

Use `cy.createTestDir()` instead and upload only once.

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud marked this pull request as ready for review December 10, 2024 13:01
@max-nextcloud max-nextcloud merged commit 4eea0ad into main Dec 10, 2024
@max-nextcloud max-nextcloud deleted the backport/6760/main branch December 10, 2024 13:06
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.

1 participant