Skip to content

test: reduce flakyness of files version tests - #62627

Merged
CarlSchwan merged 1 commit into
masterfrom
test/versions-perf
Jul 29, 2026
Merged

test: reduce flakyness of files version tests#62627
CarlSchwan merged 1 commit into
masterfrom
test/versions-perf

Conversation

@susnux

@susnux susnux commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Those tests are pretty slow, even locally they took 11s for one test. Timeout is 30s, so on crowded CI this often failed.

Fixed with:

  1. Reduce number of page loads by simply navigating back to parent folder using breadcrumbs (reduces run time by 4s per test locally)
  2. Mark as slow as setup still takes some time.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Those tests are pretty slow, even locally they took 11s for one test.
Timeout is 30s, so on crowded CI this often failed.

Fixed with:
1. Reduce number of page loads by simply navigating back to parent
   folder using breadcrumbs (reduces run time by 4s per test locally)
2. Mark as slow as setup still takes some time.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux requested a review from a team as a code owner July 28, 2026 17:21
@susnux
susnux requested review from kristian-zendato, nfebe and sorbaugh and removed request for a team July 28, 2026 17:21
@susnux susnux added 3. to review Waiting for reviews tests Related to tests labels Jul 28, 2026
@susnux susnux added this to the Nextcloud 35 milestone Jul 28, 2026
@CarlSchwan
CarlSchwan merged commit 77c5d71 into master Jul 29, 2026
221 of 223 checks passed
@CarlSchwan
CarlSchwan deleted the test/versions-perf branch July 29, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants