Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump test coverage for SV utils package #5139

Open
SHuang-Broad opened this issue Aug 27, 2018 · 0 comments
Open

Bump test coverage for SV utils package #5139

SHuang-Broad opened this issue Aug 27, 2018 · 0 comments

Comments

@SHuang-Broad
Copy link
Contributor

We've been implementing new features but the coverage in the sv.utils package has been non-pretty.

Some of the classes like ExternalCommandlineProgramModule.java and GATKSVVCFConstants.java are OK (ExternalCommandlineProgramModule exists for historical reason, when we had SGA as the assembler; GATKSVVCFConstants is mostly holding String literals), but other classes are doing actual work and have low coverage.

We should test them, not via integration tests only, but also unit tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant