Skip to content

Retry failed tests - #245

Merged
v-scharf merged 2 commits into
mainfrom
retry-failed-tests
Oct 20, 2026
Merged

v-scharf merged 2 commits into
mainfrom
retry-failed-tests

Conversation

@v-scharf

@v-scharf v-scharf commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor
  • Retry failed integration tests once in CI
  • Show flaky tests in CI test results
  • Hide Kotlin warnings to keep CI test logs clean

@v-scharf
v-scharf marked this pull request as ready for review October 20, 2026 09:36
Copilot AI balanced review requested due to automatic review settings October 20, 2026 09:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

馃煛 Changes recommended

Unescaped shell variables can cause the CI step to report success when integration tests fail.

1 open issue
What changed in this PR

Adds configurable retries for integration tests, enabled once in CI, with account cleanup between attempts.

Changes:

  • Adds retry and account-cleanup rules to login tests.
  • Reports tests that pass on retry in CI.
  • Suppresses Kotlin compiler warnings in CI.
File Description
opencloudApp/鈥媠rc/鈥媔ntegrationTest/鈥媕ava/鈥媏u/鈥媜pencloud/鈥媋ndroid/鈥婻etryRule.kt Adds configurable test retries.
opencloudApp/鈥媠rc/鈥媔ntegrationTest/鈥媕ava/鈥媏u/鈥媜pencloud/鈥媋ndroid/鈥婻emoveAccountsRule.kt Clears accounts before each attempt.
opencloudApp/鈥媠rc/鈥媔ntegrationTest/鈥媕ava/鈥媏u/鈥媜pencloud/鈥媋ndroid/鈥婰oginTest.kt Applies ordered retry and cleanup rules.
build.gradle Suppresses Kotlin warnings in CI.
.woodpecker/鈥媔ntegration-test.yml Enables one retry and reports flaky tests.

Review effort: Balanced

Comment thread .woodpecker/integration-test.yml Outdated
@v-scharf
v-scharf requested a review from guruz October 20, 2026 09:49
@guruz

guruz commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Thank you!
(when merging, please ideally use merge+rebase to avoid merge commits)

@v-scharf
v-scharf merged commit 1be5882 into main Oct 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants