Skip to content

Comments

chore: retry integration tests and generate report#782

Merged
njbrake merged 2 commits intomainfrom
robust_int
Jan 28, 2026
Merged

chore: retry integration tests and generate report#782
njbrake merged 2 commits intomainfrom
robust_int

Conversation

@njbrake
Copy link
Contributor

@njbrake njbrake commented Jan 28, 2026

Description

Integration tests fail because of the provider overload/model flakiness, etc. Lets add a little retry logic on our side to smooth that out. This should also generate a report so you can see which ones needed the retry

PR Type

  • 🚦 Infrastructure

Relevant issues

Checklist

  • I understand the code I am submitting.
  • I have added unit tests that prove my fix/feature works
  • I have run this code locally and verified it fixes the issue.
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines
  • AI Usage:
    • No AI was used.
    • AI was used for drafting/refactoring.
    • This is fully AI-generated.

AI Usage Information

  • AI Model used: Opus 4.5
  • AI Developer Tool used: CC + AoE
  • Any other info you'd like to share:

Claude did an analysis of options:

  Recommended Solution: pytest-rerunfailures + Custom GitHub Actions Reporter

  I recommend using pytest-rerunfailures (the most mature solution with 2.2M weekly downloads) combined with a custom conftest.py plugin for GitHub Actions summary reporting.

  Why pytest-rerunfailures?
  ┌──────────────────────┬──────────────────────┬──────────────┬──────────────┐
  │       Criteria       │ pytest-rerunfailures │ pytest-retry │    flaky     │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ Weekly downloads     │ 2.2M                 │ ~Low         │ Moderate     │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ Health score         │ 85/100               │ 50/100       │ N/A          │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ pytest-xdist support │ Yes (crash recovery) │ No           │ Partial      │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ Active maintenance   │ Yes (Oct 2025)       │ Limited      │ Moderate     │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ Exception filtering  │ Yes                  │ Yes          │ Via callback │
  ├──────────────────────┼──────────────────────┼──────────────┼──────────────┤
  │ Per-test config      │ Yes                  │ Yes          │ Yes          │
  └──────────────────────┴──────────────────────┴──────────────┴──────────────┘
 . ```

When answering questions by the reviewer, please respond yourself, do not copy/paste the reviewer comments into an AI system and paste back its answer. We want to discuss with you, not your AI :) 

@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 45 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njbrake
Copy link
Contributor Author

njbrake commented Jan 28, 2026

@njbrake njbrake requested a review from daavoo January 28, 2026 17:24
@njbrake njbrake requested a review from tbille January 28, 2026 17:24
Copy link
Contributor

@daavoo daavoo left a comment

Choose a reason for hiding this comment

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

❤️

@njbrake njbrake merged commit 567b887 into main Jan 28, 2026
11 checks passed
@njbrake njbrake deleted the robust_int branch January 28, 2026 18:20
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