@justaddcoffee I am proposing changes to the mutation tests to make them pass.
The issue with the code was that the functions test_gene and test_hgvs did not include all required fields. I changed the test inputs such that the info is present and the tests only update the tested fields, such as Hugo_Symbol and HGVSc.
Some tests fail even after this PR, but we need to resolve those separately.
Pls let me know if the changes correspond to what you intended to test back in the September..
@justaddcoffee I am proposing changes to the mutation tests to make them pass.
The issue with the code was that the functions
test_gene
andtest_hgvs
did not include all required fields. I changed the test inputs such that the info is present and the tests only update the tested fields, such asHugo_Symbol
andHGVSc
.Some tests fail even after this PR, but we need to resolve those separately.
Pls let me know if the changes correspond to what you intended to test back in the September..