onecodex / needletail

Fast FASTX parsing and k-mer methods in Rust
MIT License
174 stars 20 forks source link

Fixed broken tests #42

Closed markhildreth closed 4 years ago

markhildreth commented 4 years ago

Fixed some tests which were not running.

One of the test modules was incorrectly marked with #[cfg(tests)] rather than #[cfg(test)] (note the plural). This caused the tests to not be picked up by the test runner. I made these and a few other fixes for the tests to run and pass.

One notable fix to point out is that the expected values around lines 158-161 appear to have been incorrect, and as such I have changed the expected values to match what I believe should be the correct values.

Keats commented 4 years ago

Thanks!