search

jazzband / geojson

Python bindings and utilities for GeoJSON
https://pypi.python.org/pypi/geojson/
BSD 3-Clause "New" or "Revised" License
905 stars 120 forks source link

Improve coverage #106

Closed JesseWeinstein closed 6 years ago

JesseWeinstein commented 6 years ago

This branch is based on my other PR #104 (because I want to use it in these changes). The goal of this PR is to get as close to 100% code coverage as I can.

codecov-io commented 6 years ago

Codecov Report

Merging #106 into master will increase coverage by 12.63%. The diff coverage is 66.66%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #106       +/-   ##
===========================================
+ Coverage   79.88%   92.51%   +12.63%     
===========================================
  Files          11       11               
  Lines         338      334        -4     
  Branches       63       62        -1     
===========================================
+ Hits          270      309       +39     
+ Misses         58       16       -42     
+ Partials       10        9        -1
Impacted Files Coverage Δ
geojson/utils.py 93.82% <100%> (+43.82%) :arrow_up:
geojson/codec.py 95.23% <50%> (+13.41%) :arrow_up:
geojson/feature.py 100% <0%> (+17.64%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6850e48...5df6895. Read the comment docs.

frewsxcv commented 6 years ago

wow, nice work! thanks for doing this 🙇

bors r+

bors[bot] commented 6 years ago

Build succeeded