NOAA-EMC / NCEPLIBS-bacio

This library performs binary I/O for the NCEP models.
Other
2 stars 6 forks source link

More test code, trying to fill in gaps in code coverage for bacio.c #63

Closed edwardhartnett closed 2 years ago

edwardhartnett commented 2 years ago

In this PR I took a look at the code coverage report for bacio.c and tried to fill in the gaps to touch as much code as possible.

When #61 and this PR are merged, we can look at code coverage again and see how I did. ;-)

Part of #33 Part of #38

edwardhartnett commented 2 years ago

OK, there's more testing that can be done, but this PR is ready to merge and brings the testing coverage up quite a bit (and test coverage is now working on github).

image

edwardhartnett commented 2 years ago

@Hang-Lei-NOAA can you approve this PR so I can get it merged? Thanks!

edwardhartnett commented 2 years ago

@Hang-Lei-NOAA or @BrianCurtis-NOAA can you inpect and hopefully approve this PR? I need to get it merged...