Add helpers for mateCigar and matesOverlap on SamRecord

[clintval]

No description provided.

[codecov-commenter]

Codecov Report

Merging #717 (39fee88) into master (c79a9c9) will increase coverage by 0.03%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #717      +/-   ##
==========================================
+ Coverage   95.39%   95.42%   +0.03%     
==========================================
  Files         114      114              
  Lines        6596     6623      +27     
  Branches      426      444      +18     
==========================================
+ Hits         6292     6320      +28     
+ Misses        304      303       -1     

Flag	Coverage Δ
unittests	95.42% <88.88%> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...crumgenomics/rnaseq/EstimateRnaSeqInsertSize.scala	98.29% <50.00%> (-0.02%)	⬇️
.../scala/com/fulcrumgenomics/bam/api/SamRecord.scala	91.91% <100.00%> (+1.21%)	⬆️
.../scala/com/fulcrumgenomics/fastq/DemuxFastqs.scala	96.51% <0.00%> (+0.30%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c79a9c9...39fee88. Read the comment docs.

[nh13]

One minor change.

[clintval]

Back to you!

[clintval]

Thanks Tim I think that makes a lot of sense!

[clintval]

Wait to merge, I am redundantly using require twice for all these methods since they delegate back to mateCigar but I can revert that change so we don't have the overhead of checking something twice.

[clintval]

@nh13 This is ready for final review!

[clintval]

Ok, done! One more fast path added! I confirmed via coverage reporting all branches are tested too.