Uh oh!
There was an error while loading. Please reload this page.
test: stabilize macOS downloader retry assertions - #1189
Conversation
Uh oh!
There was an error while loading. Please reload this page.
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughTwo retry-policy tests now avoid asserting platform-dependent truncated-body error text. They capture the final error and retain the stable assertion that retries exhaust after five attempts. No production download logic changed. ChangesRetry test stability
Estimated code review effort: 1 (Trivial) | ~3 minutes ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Fixes the macOS retry-test flake in #1187's main workflow. The final retry can surface as either a truncated-body or request-setup transport failure; both are retryable, and the behavioral contract is exhaustion of exactly five attempts.\n\nValidated locally:\n- soldr cargo fmt --all -- --check\n- soldr cargo test -p fbuild-packages-fetch truncated_bodies (4 passed)\n- soldr cargo clippy -p fbuild-packages-fetch --lib -- -D warnings\n- diff review: clean
Summary by CodeRabbit