Skip to content

fix(tests): Make the cypress apps management test more reliable - #43273

Merged
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/make-apps-cypress-test-more-reliable
Feb 2, 2024
Merged

fix(tests): Make the cypress apps management test more reliable#43273
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/make-apps-cypress-test-more-reliable

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Summary

I was able to reproduce most failures that I found for the apps.cy.ts test by adding sleeps in the right places:

Checklist

@nickvergessennickvergessen added bug 3. to review Waiting for reviews javascript tests Related to tests labels Feb 1, 2024
@nickvergessennickvergessen added this to the Nextcloud 29 milestone Feb 1, 2024
@nickvergessennickvergessen self-assigned this Feb 1, 2024
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable28

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/make-apps-cypress-test-more-reliable branch from 06c9400 to b99749dCompareFebruary 2, 2024 06:04
@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 2, 2024
@skjnldsv
skjnldsv merged commit 5a8cc42 into masterFeb 2, 2024
@skjnldsv
skjnldsv deleted the bugfix/noid/make-apps-cypress-test-more-reliable branch February 2, 2024 15:37
@blizzzblizzz mentioned this pull request Mar 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugjavascripttestsRelated to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@ChristophWurst@skjnldsv