Merge pull request #180 from NeurodataWithoutBorders/enh/subject_species

make subject species error message include current species name

Author: CodyCBakerPhD

nwbinspector/checks/nwbfile_metadata.py

@@ -130,7 +130,8 @@ def check_subject_species_latin_binomial(subject: Subject):
     """Check if the subject species follows latin binomial form."""
     if subject.species and not re.fullmatch(species_regex, subject.species):
         return InspectorMessage(
-            message="Species should be in latin binomial form, e.g. 'Mus musculus' and 'Homo sapiens'",
+            message=f"Subject species '{subject.species}' should be in latin binomial form, e.g. 'Mus musculus' and "
+            f"'Homo sapiens'",
         )
 
 

tests/unit_tests/test_nwbfile_metadata.py

@@ -278,7 +278,7 @@ def test_check_subject_species_not_binomial():
     subject = Subject(subject_id="001", species="Human")
 
     assert check_subject_species_latin_binomial(subject) == InspectorMessage(
-        message="Species should be in latin binomial form, e.g. 'Mus musculus' and 'Homo sapiens'",
+        message="Subject species 'Human' should be in latin binomial form, e.g. 'Mus musculus' and 'Homo sapiens'",
         importance=Importance.BEST_PRACTICE_VIOLATION,
         check_function_name="check_subject_species_latin_binomial",
         object_type="Subject",