search

desihub / speclite

Lightweight utilities for working with spectroscopic data
14 stars 19 forks source link

Add more resample unit tests #1

Closed dkirkby closed 9 years ago

dkirkby commented 9 years ago

The resample module is currently the only one with less than 100% coverage. This issue is to add additional unit tests to tests/test_resample.py to cover the lines flagged in this coverage report.

dcunning11235 commented 9 years ago

Added additional tests in https://github.com/dkirkby/speclite/tree/resample_units. Will review later: it should be at 100% coverage but it is still showing a few exceptions/tests not covered.

dcunning11235 commented 9 years ago

@dkirkby At 100% coverage now. Travis build for the push done, but a second (I think for the pull) is still running. Back after class.