Merge pull request #115 from Clinical-Genomics/fix_multiple_conditions_parsing

Fix multiple conditions parsing

Author: northwestwitch

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## [unreleased]
 ### Fixed
 - Upload to Codecov step in `tests_n_coverage.yml` action
+- Fix `multipleConditionExplanation` to be lowercase
 
 ## [2.5.1]
 ### Fixed

preClinVar/file_parser.py

@@ -94,7 +94,7 @@ def set_item_condition_set(item: dict, variant_dict: dict):
         item["conditionSet"] = {"condition": conditions}
         if multi_condition_explanation:
             item["conditionSet"][
-                "MultipleConditionExplanation"
+                "multipleConditionExplanation"
             ] = multi_condition_explanation.capitalize()
 
 

tests/test_file_parser.py

@@ -1,6 +1,5 @@
 from preClinVar.constants import CLNSIG_TERMS
 from preClinVar.file_parser import (
-    csv_lines,
     set_item_clin_sig,
     set_item_condition_set,
     set_item_variant_set,
@@ -50,7 +49,7 @@ def test_set_item_condition_set():
     set_item_condition_set(item=item, variant_dict=variant_dict)
 
     # THEN it should contain the expected key/values
-    assert item["conditionSet"]["MultipleConditionExplanation"] == MULTIPLE_COND_EXPLANATION
+    assert item["conditionSet"]["multipleConditionExplanation"] == MULTIPLE_COND_EXPLANATION
     for condition in item["conditionSet"]["condition"]:
         assert condition["db"] == CONDITION_DB
         assert condition["id"] in OMIM_NUMBERS