You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As discussed in #16153 (comment), test_resample.py grew sufficiently large (currently ~ 2.7 k sloc), so splitting it up should make it more understandable.
Moreover, readability of some test cases could be improved by using pytest features:
Convert nested for loops and "quasi-parametrized" tests (multiple test methods calling the same test logic with different parameters) to make use of @pytest.mark.parametrize decorator
Use pytest fixtures where appropriate
The text was updated successfully, but these errors were encountered:
As discussed in #16153 (comment), test_resample.py grew sufficiently large (currently ~ 2.7 k sloc), so splitting it up should make it more understandable.
Proposal to split along the existing classes:
tests/resample (sub-dir)
|- common.py
|- test_resample_api.py
|- base.py
|- test_datetime.py
|- test_period.py
|- test_timedelta.py
|- test_resample_grouper.py
|- test_time_grouper.py
Moreover, readability of some test cases could be improved by using pytest features:
The text was updated successfully, but these errors were encountered: