-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
TST: Use new arithmetic fixtures, parametrize many more tests #23757
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hello @jbrockmendel! Thanks for submitting the PR.
|
looks fine. lots of tests :-D |
Yep, and you better believe we're not done yet. Working on a test_timedelta64 branch for after #23642. |
Codecov Report
@@ Coverage Diff @@
## master #23757 +/- ##
==========================================
+ Coverage 92.23% 92.24% +0.01%
==========================================
Files 161 161
Lines 51408 51408
==========================================
+ Hits 47416 47422 +6
+ Misses 3992 3986 -6
Continue to review full report at Codecov.
|
thanks @jbrockmendel |
Focus is mostly on test_datetime64.py to keep things close to orthogonal.
De-duplicates some tests that are in e.g. both TestFrameArithmetic and TestSeriesArithmetic