-
-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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
timestamp/timedelta test cleanup #18619
Conversation
Hello @jbrockmendel! Thanks for updating the PR. Cheers ! There are no PEP8 issues in this Pull Request. 🍻 Comment last updated on December 04, 2017 at 01:22 Hours UTC |
Codecov Report
@@ Coverage Diff @@
## master #18619 +/- ##
==========================================
- Coverage 91.46% 91.44% -0.03%
==========================================
Files 157 157
Lines 51449 51449
==========================================
- Hits 47060 47049 -11
- Misses 4389 4400 +11
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #18619 +/- ##
==========================================
- Coverage 91.46% 91.44% -0.03%
==========================================
Files 157 157
Lines 51449 51449
==========================================
- Hits 47060 47049 -11
- Misses 4389 4400 +11
Continue to review full report at Codecov.
|
thanks! |
More informative names in timedeltas tests
remove/merge redundant tests in timestamps. Separate some big test classes thematically.
Use pytest.parametrize in a few places