-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
Improve testing coverage #6251
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
Comments
Do you have a document that tracks testing coverage? If not would producing such a document/wiki be helpful? |
their is some code for using the coverage package to measure this would appreciate a section / how to on the wiki though it's possible need some more / better exhorting to make this more user friendly pls experiment and let us know |
So you mainly look at code coverage? Within your testing is there any concept of test flows or test cases? Tests that are larger than unit tests, which is all I've really encountered. |
Would there be interest to use coveralls to track testing coverage? That would be a handy way to see how coverage is for the different modules, and to see where more work is needed. Example: https://coveralls.io/files/196422109 You have also the possibility to let coveralls comment on PRs, but I find that sometimes a bit too much (eg see PRs of scipy https://github.com/scipy/scipy/pulls). |
@jorisvandenbossche IMO coveralls would be great! Does this slow down the build much? |
should be pretty straightforward to add this as @jorisvandenbossche PR almost there. any takers? I can turn on the github service so we get notifications of this. |
@jreback Was coveralls enabled? I see There also doesn't seem to be https://coveralls.io/github/pydata/pandas?branch=master |
no why would we need more than one? |
They offer a bit different organization of the same coverage data, so we do not need to have both ... |
We have coverage tools installed and coverage it up to 91%. Respectable progress for now; closing. |
Pandas coverage is currently at 78%, not bad but could be better.
Open issue for first-time contributors: find some ill-tested corner
and bring it into the light.
The text was updated successfully, but these errors were encountered: