Skip to content

test: reduce flakyness of files version tests - #62627

Open
susnux wants to merge 1 commit into
masterfrom
test/versions-perf
Open

test: reduce flakyness of files version tests#62627
susnux wants to merge 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
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.

2 participants