🧪 Add tests to improve coverage

Author: tsvikas

tests/test_jinja.py

@@ -11,6 +11,7 @@
 from hebrew_numbers import InvalidNumberError
 from hebrew_numbers.jinja import (
     HebrewNumbersExtension,
+    _map_hebrew_boolean,
     hebrew_cardinal_filter,
     hebrew_count_filter,
     hebrew_indefinite_filter,
@@ -255,3 +256,13 @@ def test_filter_chaining(self) -> None:
         template = env.from_string("{{ 42 | hebrew_indefinite | upper }}")
         result = template.render()
         assert result == "ארבעים ושתיים"  # Hebrew text doesn't change with upper()
+
+
+class TestHebrewBooleanMapErrors:
+    """Test _map_hebrew_boolean error handling."""
+
+    def test_hebrew_boolean_invalid_value(self) -> None:
+        """Test _map_hebrew_boolean with invalid value."""
+        # This tests lines 98-101 in jinja.py
+        with pytest.raises(ValueError, match="Invalid Hebrew boolean value: 'invalid'"):
+            _map_hebrew_boolean("invalid")

tests/test_numbers.py

@@ -5,7 +5,13 @@
 import pytest
 
 import hebrew_numbers
-from hebrew_numbers import ConstructState, GrammaticalGender, InvalidNumberError
+from hebrew_numbers import (
+    ConstructState,
+    GrammaticalGender,
+    InvalidNumberError,
+    cardinal_number,
+)
+from hebrew_numbers.hebrew_numbers import _join_words
 
 if TYPE_CHECKING:
     from pytest_regressions.data_regression import DataRegressionFixture
@@ -202,3 +208,49 @@ def test_over_10(gender: GrammaticalGender) -> None:
                 valid_exceptions=InvalidNumberError,
             )
         )
+
+
+class TestConstructStateFromBoolean:
+    """Test ConstructState.from_boolean method edge cases."""
+
+    def test_from_boolean_with_construct_state_enum(self) -> None:
+        """Test passing ConstructState enum directly returns same value."""
+        # This tests the else clause in from_boolean (line 68)
+        result = ConstructState.from_boolean(ConstructState.ABSOLUTE)
+        assert result == ConstructState.ABSOLUTE
+
+        result = ConstructState.from_boolean(ConstructState.CONSTRUCT)
+        assert result == ConstructState.CONSTRUCT
+
+
+class TestJoinWordsEdgeCases:
+    """Test _join_words function with edge cases."""
+
+    def test_join_empty_words_list(self) -> None:
+        """Test that empty words list raises ValueError."""
+        # This tests line 93 in hebrew_numbers.py
+        with pytest.raises(
+            ValueError, match="must contain at least one non-empty string"
+        ):
+            _join_words([])
+
+    def test_join_only_empty_strings(self) -> None:
+        """Test that list with only empty strings raises ValueError."""
+        # This also tests line 93 in hebrew_numbers.py
+        with pytest.raises(
+            ValueError, match="must contain at least one non-empty string"
+        ):
+            _join_words(["", "", ""])
+
+
+class TestCardinalNumberEdgeCases:
+    """Test cardinal_number function with invalid inputs."""
+
+    def test_cardinal_number_invalid_range(self) -> None:
+        """Test cardinal_number with numbers outside valid range."""
+        # Test zero (line 333 in hebrew_numbers.py)
+        with pytest.raises(InvalidNumberError, match="Number must be positive"):
+            cardinal_number(0, GrammaticalGender.MASCULINE, ConstructState.ABSOLUTE)
+
+        with pytest.raises(InvalidNumberError, match="Number must be positive"):
+            cardinal_number(-1, GrammaticalGender.MASCULINE, ConstructState.ABSOLUTE)