broadinstitute / longbow

Annotation and segmentation of MAS-seq data
https://broadinstitute.github.io/longbow/
BSD 3-Clause "New" or "Revised" License
20 stars 4 forks source link

fixed various bugs and tests #163

Closed jamestwebber closed 2 years ago

jamestwebber commented 2 years ago

Fixed a bunch of test failures, including a couple bugs

main fix: the introduction of get_read_count_from_bam_index was breaking the use of streamed input because it reads from the BAM file. Fixed by adding seek(0) if the input is a file, and returning None otherwise.

A couple missing imports: os in tagfix, ssw in bam_utils

test_annotate had some issue with the way the stream was working. Using CliRunner seems to work better.

test_extract had a bad argument (-t is not a valid arg for extract. In the course of debugging, I also switched to using CliRunner there.