-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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
MAINT: Remove self.assertTrue from testing #16158
Conversation
Appveyor failing due to network issues. Travis and Circle seem okay. @jreback : Could you restart the Appveyor build? |
yeah those are 'network' tests that don't have the decorator! (as I think there is some interaction with parametrize). |
Codecov Report
@@ Coverage Diff @@
## master #16158 +/- ##
==========================================
+ Coverage 90.85% 90.85% +<.01%
==========================================
Files 159 159
Lines 50785 50785
==========================================
+ Hits 46141 46143 +2
+ Misses 4644 4642 -2
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #16158 +/- ##
==========================================
+ Coverage 90.85% 90.85% +<.01%
==========================================
Files 159 159
Lines 50785 50785
==========================================
+ Hits 46141 46143 +2
+ Misses 4644 4642 -2
Continue to review full report at Codecov.
|
thanks! |
Is there a reason the |
I think I briefly tried it and couldn't get it to work when the parameterize one was as well. |
The test is actually decorated because the function called in each test has the |
Title is self-explanatory.
Partially addresses #15990.