Skip to content

test(version-check): Fix flaky test - #62609

Merged
nickvergessen merged 1 commit into
masterfrom
tests/noid/fix-flaky-test
Jul 28, 2026
Merged

test(version-check): Fix flaky test#62609
nickvergessen merged 1 commit into
masterfrom
tests/noid/fix-flaky-test

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Fix flaky tests like https://github.com/nextcloud/server/actions/runs/30356507708/job/90266119531?pr=62441

Parameter 2 for invocation OCP\IAppConfig::setValueInt('core', 'lastupdatedat', 1785239917, false, false): bool does not match expected value.
Failed asserting that 1785239917 matches expected 1785239918.

Checklist

AI (if applicable)

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

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the Nextcloud 35 milestone Jul 28, 2026
@nickvergessen nickvergessen self-assigned this Jul 28, 2026
@nickvergessen nickvergessen added the tests Related to tests label Jul 28, 2026
@nickvergessen
nickvergessen requested a review from a team as a code owner July 28, 2026 12:32
@nickvergessen
nickvergessen requested review from Altahrim, leftybournes, provokateurin and salmart-dev and removed request for a team July 28, 2026 12:32
@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable34

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable33

@nickvergessen

Copy link
Copy Markdown
Member Author

/backport to stable32

@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Jul 28, 2026

@provokateurin provokateurin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🙈

@nickvergessen
nickvergessen merged commit 6e98de3 into master Jul 28, 2026
203 of 217 checks passed
@nickvergessen
nickvergessen deleted the tests/noid/fix-flaky-test branch July 28, 2026 14:17
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.

6 participants