LanuguageCode: Add tests to raise coverage to 100% (#104)

Author: tomschr

src/docbuild/models/language.py

@@ -1,10 +1,9 @@
 """Language model for representing language codes."""
 
-from functools import total_ordering, cached_property
-import re
+from functools import cached_property, total_ordering
 from typing import Any, ClassVar
 
-from pydantic import BaseModel, Field, computed_field, model_validator, field_validator
+from pydantic import BaseModel, Field, computed_field, field_validator, model_validator
 from pydantic.config import ConfigDict
 
 from ..constants import ALLOWED_LANGUAGES
@@ -118,7 +117,7 @@ def matches(self, other: 'LanguageCode | str') -> bool:
         return (
             self.language == '*' or other_value == '*' or self.language == other_value
         )
-    
+
     @field_validator('language', mode='before')
     @classmethod
     def _normalize_language_separator(cls, value: str) -> str:
@@ -153,7 +152,7 @@ def _parts(self) -> tuple[str, str] | tuple[str]:
         """
         if self.language == '*':
             return ('*',)
-        
+
         # Use split('-') as the separator is already normalized
         parts = self.language.split('-')
         return (parts[0], parts[1]) if len(parts) > 1 else (parts[0],)
@@ -174,4 +173,4 @@ def lang(self) -> str:
     )
     def country(self) -> str:
         """Extract the country part of the language code (property)."""
-        return self._parts[1] if len(self._parts) > 1 else '*'
+        return self._parts[1] if len(self._parts) > 1 else '*'

tests/models/test_language.py

@@ -138,4 +138,28 @@ def test_compare_with_unknown_type():
 
 def test_language_matches():
     lang1 = LanguageCode(language='de-de')
-    assert lang1.matches('*')
+    assert lang1.matches('*')
+
+
+def test_language_code_init_with_invalid_type_raises_error():
+    """Test LanguageCode initialization with a non-string, non-dict value.
+
+    This covers the non-string path in the `_normalize_language_separator`
+    field validator (line 128) and ensures Pydantic's internal validation
+    raises the expected error.
+    """
+    with pytest.raises(ValueError):
+        LanguageCode(language=123)
+
+
+def test_convert_str_to_dict_accepts_string() -> None:
+    """LanguageCode should accept a plain string and populate .language."""
+    # Use Pydantic v2 entrypoint that accepts raw input so the
+    # `@model_validator(mode='before')` runs and converts the string.
+    lc = LanguageCode.model_validate("en-us")
+    assert isinstance(lc, LanguageCode)
+    assert lc.language == "en-us"
+    # ensure computed properties work as expected
+    assert lc.lang == "en"
+    assert lc.country == "us"
+