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

[stable30] fix(cy): avoid double upload to prevent lock error #6761

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Dec 10, 2024

Backport of #6760

@backportbot backportbot bot requested review from mejo- and max-nextcloud December 10, 2024 10:06
@backportbot backportbot bot added tests If you write them we ♥ you 3. to review labels Dec 10, 2024
@backportbot backportbot bot added this to the Nextcloud 30.0.4 milestone Dec 10, 2024
`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>
@mejo- mejo- force-pushed the backport/6760/stable30 branch from bdd30ae to 8d365d2 Compare December 10, 2024 10:10
@mejo- mejo- marked this pull request as ready for review December 10, 2024 10:11
@blizzz blizzz mentioned this pull request Dec 11, 2024
@juliusknorr juliusknorr merged commit b126cd5 into stable30 Dec 12, 2024
63 checks passed
@juliusknorr juliusknorr deleted the backport/6760/stable30 branch December 12, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review backport-request tests If you write them we ♥ you
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants