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
added test class for CheckFingerprint #1006
added test class for CheckFingerprint #1006
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! thanks for testing.
some minor comments.
"H=" + SUBSETTED_HAPLOTYPE_DATABASE_FOR_TESTING | ||
}; | ||
Assert.assertEquals(runPicardCommandLine(args), 0); | ||
Assert.assertEquals(checkResult( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to have two asserts, one for each metrics file....that way when something breaks it's easier to understand where the probem is
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
new File(TEST_DATA_DIR + "/tempCheckFPDir/detail")), true); | ||
} | ||
|
||
@Test(dataProvider = "badData", expectedExceptions = {TribbleException.MalformedFeatureFile.class, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the fully qualified class name needed?
Hello, thanks for your feedback!
|
c412469
to
63a6ed1
Compare
I'm sorry, had some troubles with git branch, so all my small local commits were shown. Have made force push with amend, which replaced all of them with the last one. All mentioned changes are inside. Sorry for inconvenience. |
63a6ed1
to
e781bd7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Description
Added:
Checklist (never delete this)
Never delete this, it is our record that procedure was followed. If you find that for whatever reason one of the checklist points doesn't apply to your PR, you can leave it unchecked but please add an explanation below.
Content
Review
For more detailed guidelines, see https://github.com/broadinstitute/picard/wiki/Guidelines-for-pull-requests