Skip to content

fix(ci): add pytest-rerunfailures to integration test dependencies#400

Merged
jariy17 merged 1 commit intomainfrom
fix/ci-pytest-rerunfailures
Apr 9, 2026
Merged

fix(ci): add pytest-rerunfailures to integration test dependencies#400
jariy17 merged 1 commit intomainfrom
fix/ci-pytest-rerunfailures

Conversation

@jariy17
Copy link
Copy Markdown
Contributor

@jariy17 jariy17 commented Apr 9, 2026

Summary

  • Adds pytest-rerunfailures to the integration test CI pip install step
  • Without it, @pytest.mark.flaky(reruns=2, reruns_delay=5) decorators in integration tests are silently ignored, so flaky tests never retry and pytest emits unknown mark warnings

Test plan

  • Verify pytest-rerunfailures is already in dev dependencies in pyproject.toml
  • CI integration tests should now properly retry flaky tests instead of failing on first attempt

The @pytest.mark.flaky(reruns=2) decorator in integration tests requires
pytest-rerunfailures to be installed. Without it, flaky tests silently
skip retries and pytest emits unknown mark warnings.
@jariy17 jariy17 requested a review from a team April 9, 2026 14:41
Copy link
Copy Markdown
Contributor

@Hweinstock Hweinstock left a comment

Choose a reason for hiding this comment

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

Nice find!

@jariy17 jariy17 merged commit 5ccb283 into main Apr 9, 2026
27 of 28 checks passed
@jariy17 jariy17 deleted the fix/ci-pytest-rerunfailures branch April 9, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants