search

sixty-north / segpy

A Python package for reading and writing SEG Y files.
Other
102 stars 54 forks source link

100% coverage for reader.py #80

Closed abingham closed 6 years ago

abingham commented 6 years ago

This adds complete statement coverage for reader.py. I'm not particular proud of some of the tests, but the coverage is there. I assume that application of Cosmic Ray will point out the deficiencies of some of my tests.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.2%) to 96.718% when pulling 2424b396ecb406dc9802bb85e1c61b4bda04c425 on abingham:reader-tests into 5c88210d977c02c185364ea31939fbe707e84209 on sixty-north:master.

abingham commented 6 years ago

The dataset strategies that we have now are a bit haphazard. There's probably good scope and reason for making them a bit more regular, configurable, etc. The current state more or less just evolved as I need new features, and since I was also learning about SEGY as I went, there was not good master plan from the start.