search

ecmwf / pdbufr

High-level BUFR interface for ecCodes
Apache License 2.0
23 stars 8 forks source link

Major cleanup, more tests and a few fixes. #23

Closed alexamici closed 3 years ago

alexamici commented 3 years ago

Drop all old code, and test to 100% coverage.

codecov-io commented 3 years ago

Codecov Report

Merging #23 (6109e8a) into master (560600f) will increase coverage by 13.69%. The diff coverage is 98.55%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #23       +/-   ##
===========================================
+ Coverage   85.71%   99.40%   +13.69%     
===========================================
  Files          10        9        -1     
  Lines         819      675      -144     
  Branches      125       76       -49     
===========================================
- Hits          702      671       -31     
+ Misses        111        2      -109     
+ Partials        6        2        -4
Impacted Files Coverage Δ
pdbufr/__main__.py 100.00% <ø> (+15.38%) :arrow_up:
pdbufr/bufr_filters.py 100.00% <ø> (+14.75%) :arrow_up:
tests/test_40_sample_data.py 99.14% <ø> (ø)
pdbufr/__init__.py 85.71% <85.71%> (-14.29%) :arrow_down:
pdbufr/bufr_read.py 100.00% <100.00%> (+56.62%) :arrow_up:
pdbufr/bufr_structure.py 100.00% <100.00%> (+5.80%) :arrow_up:
tests/test_20_bufr_structure.py 100.00% <100.00%> (ø)

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 560600f...6109e8a. Read the comment docs.

iainrussell commented 3 years ago

I love the code coverage report!