TEST: Reuse rolling GroupBy test inputs - #24118
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 8 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe rolling groupby tests now use reusable module-scoped pandas and cuDF input fixtures. Simple, multi-key, and datetime-indexed groupby cases retain their existing reductions and window-size coverage. ChangesRolling groupby test fixtures
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to This test-only refactor reuses read-only pandas/cuDF inputs without changing rolling GroupBy behavior or coverage, so it is ready to merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Description
Reuses module-scoped, read-only pandas/cuDF GroupBy inputs in the rolling GroupBy tests. Every existing reduction, window size, input shape, and pandas comparison remains unchanged; the tests no longer recreate identical frames and grouping metadata for each reduction parameter.
Checklist