-
Notifications
You must be signed in to change notification settings - Fork 587
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
VS-759 (and VS-760) #8137
VS-759 (and VS-760) #8137
Conversation
…re tractable for our python scripts.
…es_bqloadjson_from_annotations.py
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## ah_var_store #8137 +/- ##
================================================
Coverage ? 86.242%
Complexity ? 35199
================================================
Files ? 2173
Lines ? 165045
Branches ? 17793
================================================
Hits ? 142338
Misses ? 16383
Partials ? 6324 |
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.
OOC why are the expected files checked in compressed?
self.assertEqual(contents, expected_contents) | ||
|
||
# This test tries to read an old-style json file and verifies that the program will fail if fed such a file | ||
@unittest.expectedFailure |
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 there a way to be sure this fails for the expected reason?
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've updated the code to capture stderr and verify that the failure is for the expected reason.
scripts/variantstore/wdl/extract/test_create_vt_bqloadjson_from_annotations.py
Outdated
Show resolved
Hide resolved
scripts/variantstore/wdl/extract/test_create_genes_bqloadjson_from_annotations.py
Outdated
Show resolved
Hide resolved
scripts/variantstore/wdl/extract/test_create_genes_bqloadjson_from_annotations.py
Outdated
Show resolved
Hide resolved
…m_annotations.py Co-authored-by: Miguel Covarrubias <[email protected]>
@mcovarr I checked the test files in compressed as I was trying to save space, but I kind of agree that it's not very necessary (and they're not that big), so will decompress. |
…reate_vt_bqloadjson_from_annotations.py to support testing.
…e_genes_bqloadjson_from_annotations.py so that expected failures when trying to load old style annotations files verified that the failure was as expected (and not just a rando failure).
Add more testing for create_genes_bqloadjson_from_annotations and create_vt_bqloadjson_from_annotations