Tests: Add check for model card existence and test that the FileNotFoundError is not raised.

Author: strangetom

ingredient_parser/_common.py

@@ -95,11 +95,21 @@ def show_model_card(lang: str = "en") -> None:
     ----------
     lang : str, optional
         Selected language to open model card for.
+
+    Raises
+    ------
+    FileNotFoundError
+        Raised if model card not found at expected path.
+    ValueError
+        Raised if unsupported language provided in lang argument.
     """
     if lang not in SUPPORTED_LANGUAGES:
         raise ValueError(f'Unsupported language "{lang}"')
 
     with as_file(files(__package__) / lang / f"data/ModelCard.{lang}.md") as p:
+        if not p.exists():
+            raise FileNotFoundError(f"Could not find Model Card at {p}")
+
         if platform.system() == "Darwin":  # macOS
             subprocess.call(("open", p))
         elif platform.system() == "Windows":  # Windows

tests/test_common.py

@@ -1,10 +1,13 @@
+from unittest.mock import patch
+
 import pytest
 
 from ingredient_parser._common import (
     consume,
     group_consecutive_idx,
     is_float,
     is_range,
+    show_model_card,
 )
 
 
@@ -87,7 +90,7 @@ def test_x(self):
         assert not is_range("1x")
 
 
-class TestPostProcessor_group_consecutive_indices:
+class Test_group_consecutive_indices:
     def test_single_group(self):
         """
         Return single group
@@ -103,3 +106,18 @@ def test_multiple_groups(self):
         input_indices = [0, 1, 2, 4, 5, 6, 8, 9]
         groups = group_consecutive_idx(input_indices)
         assert [list(g) for g in groups] == [[0, 1, 2], [4, 5, 6], [8, 9]]
+
+
+class Test_show_model_card:
+    @patch("os.startfile", create=True)
+    @patch("subprocess.call")
+    def test_model_card_found(self, mock_startfile, mock_subprocess_call):
+        """Test model card found at path derived from selected language.
+
+        The calls to os.startfile and subprocess.call are mocked to prevent the model
+        card from actually opening.
+        """
+        try:
+            show_model_card("en")
+        except FileNotFoundError:
+            pytest.fail("Model card not found.")