genome-rcast / karkinos

Tumor genotyper, that detects SNV, absolute CNV and Tumor contents
Apache License 2.0
10 stars 2 forks source link

Add a test for the CopyNumberInterval class #10

Closed xckitahara closed 4 years ago

xckitahara commented 4 years ago

Add a test for the CopyNumberInterval class.

alumi commented 4 years ago

CopyNumberInterval is just a data holder. Adding tests to such classes increases coverage but barely contributes to improving code quality. Please don't focus on the absolute percentage of the coverage itself, but devote your efforts to testing the essential implementation of the algorithm. I recommend starting with classes like EMMethod, GaussianWavelet, and WaveletDenoize.