Skip to content
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

test coverage #128

Closed
perrygeo opened this issue Sep 3, 2016 · 2 comments
Closed

test coverage #128

perrygeo opened this issue Sep 3, 2016 · 2 comments
Labels
Milestone

Comments

@perrygeo
Copy link
Owner

perrygeo commented Sep 3, 2016

this has slipped down to 98%, let's shoot for 💯

The current list of missing lines:

---------- coverage: platform linux2, python 2.7.9-final-0 -----------
Name                          Stmts   Miss  Cover   Missing
-----------------------------------------------------------
src/rasterstats/__init__.py       5      0   100%
src/rasterstats/_version.py       1      0   100%
src/rasterstats/cli.py           54      2    96%   48, 53
src/rasterstats/io.py           171      3    98%   105, 275, 282
src/rasterstats/main.py         105      3    97%   130, 154, 240
src/rasterstats/point.py         66      2    97%   163, 193
src/rasterstats/utils.py         82      0   100%
-----------------------------------------------------------
TOTAL                           484     10    98%
@sgoodm
Copy link
Contributor

sgoodm commented Mar 21, 2017

With PR #144

---------- coverage: platform linux2, python 2.7.9-final-0 -----------
Name                          Stmts   Miss  Cover   Missing
-----------------------------------------------------------
src/rasterstats/__init__.py       5      0   100%
src/rasterstats/_version.py       1      0   100%
src/rasterstats/cli.py           54      1    98%   48
src/rasterstats/io.py           174      1    99%   108
src/rasterstats/main.py         113      1    99%   258
src/rasterstats/point.py         66      2    97%   163, 193
src/rasterstats/utils.py         82      0   100%
-----------------------------------------------------------
TOTAL                           495      5    99%

@sgoodm sgoodm mentioned this issue Mar 21, 2017
@perrygeo
Copy link
Owner Author

Closing for now, #144 pushed through most of the major coverage issues. Thanks @sgoodm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants