scim2-models 0.4.0__tar.gz → 0.4.1__tar.gz

{scim2_models-0.4.0 → scim2_models-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: scim2-models
-Version: 0.4.0
+Version: 0.4.1
 Summary: SCIM2 models serialization and validation with pydantic
 Project-URL: documentation, https://scim2-models.readthedocs.io
 Project-URL: repository, https://github.com/python-scim/scim2-models

{scim2_models-0.4.0 → scim2_models-0.4.1}/doc/changelog.rst

@@ -1,6 +1,13 @@
 Changelog
 =========
 
+[0.4.1] - 2025-07-23
+--------------------
+
+Fixed
+^^^^^
+- Allow ``TypeVar`` as type parameters for :class:`~scim2_models.PatchOp`.
+
 [0.4.0] - 2025-07-23
 --------------------
 

{scim2_models-0.4.0 → scim2_models-0.4.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "scim2-models"
-version = "0.4.0"
+version = "0.4.1"
 description = "SCIM2 models serialization and validation with pydantic"
 authors = [{name="Yaal Coop", email="contact@yaal.coop"}]
 license = {file = "LICENSE"}

{scim2_models-0.4.0 → scim2_models-0.4.1}/scim2_models/messages/patch_op.py

@@ -143,7 +143,7 @@ class PatchOp(Message, Generic[T]):
         - Using PatchOp without a type parameter raises TypeError
     """
 
-    def __new__(cls, *args, **kwargs):
+    def __new__(cls, *args: Any, **kwargs: Any):
         """Create new PatchOp instance with type parameter validation.
 
         Only handles the case of direct instantiation without type parameter (PatchOp()).
@@ -165,9 +165,23 @@ class PatchOp(Message, Generic[T]):
     def __class_getitem__(cls, item):
         """Validate type parameter when creating parameterized type.
 
-        Ensures the type parameter is a concrete Resource subclass (not Resource itself).
-        Rejects invalid types (str, int, etc.) and Union types.
+        Ensures the type parameter is a concrete Resource subclass (not Resource itself)
+        or a TypeVar bound to Resource. Rejects invalid types (str, int, etc.) and Union types.
         """
+        # Allow TypeVar as type parameter
+        if isinstance(item, TypeVar):
+            # Check if TypeVar is bound to Resource or its subclass
+            if item.__bound__ is not None and (
+                item.__bound__ is Resource
+                or (isclass(item.__bound__) and issubclass(item.__bound__, Resource))
+            ):
+                return super().__class_getitem__(item)
+            else:
+                raise TypeError(
+                    f"PatchOp TypeVar must be bound to Resource or its subclass, got {item}. "
+                    "Example: T = TypeVar('T', bound=Resource)"
+                )
+
         # Check if type parameter is a concrete Resource subclass (not Resource itself)
         if item is Resource:
             raise TypeError(
@@ -176,7 +190,7 @@ class PatchOp(Message, Generic[T]):
             )
         if not (isclass(item) and issubclass(item, Resource) and item is not Resource):
             raise TypeError(
-                f"PatchOp type parameter must be a concrete Resource subclass, got {item}. "
+                f"PatchOp type parameter must be a concrete Resource subclass or TypeVar, got {item}. "
                 "Use PatchOp[User], PatchOp[Group], etc."
             )
 

{scim2_models-0.4.0 → scim2_models-0.4.1}/tests/test_patch_op_validation.py

@@ -379,6 +379,41 @@ def test_patch_error_handling_type_mismatch():
 
 
 T = TypeVar("T", bound=Resource)
+UserT = TypeVar("UserT", bound=User)
+UnboundT = TypeVar("UnboundT")
+
+
+def test_patch_op_with_typevar_bound_to_resource():
+    """Test that PatchOp accepts TypeVar bound to Resource."""
+    # Should not raise any exception
+    patch_type = PatchOp[T]
+    assert patch_type is not None
+
+
+def test_patch_op_with_typevar_bound_to_resource_subclass():
+    """Test that PatchOp accepts TypeVar bound to Resource subclass."""
+    # Should not raise any exception
+    patch_type = PatchOp[UserT]
+    assert patch_type is not None
+
+
+def test_patch_op_with_unbound_typevar():
+    """Test that PatchOp rejects unbound TypeVar."""
+    with pytest.raises(
+        TypeError,
+        match="PatchOp TypeVar must be bound to Resource or its subclass, got ~UnboundT",
+    ):
+        PatchOp[UnboundT]
+
+
+def test_patch_op_with_typevar_bound_to_non_resource():
+    """Test that PatchOp rejects TypeVar bound to non-Resource class."""
+    NonResourceT = TypeVar("NonResourceT", bound=str)
+    with pytest.raises(
+        TypeError,
+        match="PatchOp TypeVar must be bound to Resource or its subclass, got ~NonResourceT",
+    ):
+        PatchOp[NonResourceT]
 
 
 def test_create_parent_object_return_none():

{scim2_models-0.4.0 → scim2_models-0.4.1}/uv.lock

@@ -940,7 +940,7 @@ wheels = [
 
 [[package]]
 name = "scim2-models"
-version = "0.4.0"
+version = "0.4.1"
 source = { editable = "." }
 dependencies = [
     { name = "pydantic", extra = ["email"] },