Fix validation of Range (#943)

We should return the underlying range type, not the
_pydantic._types.Range.

Author: msullivan

gel/_internal/_qbmodel/_pydantic/_types.py

@@ -151,7 +151,7 @@ def _validate(cls, eltype: type, value: Any) -> _range.Range[_T]:
             return value
 
         elif isinstance(value, dict):
-            return cls(
+            return _range.Range(
                 lower=cls._validate_bound(eltype, value["lower"]),
                 upper=cls._validate_bound(eltype, value["upper"]),
                 inc_lower=value["inc_lower"],

tests/test_model_generator.py

@@ -3962,7 +3962,7 @@ def test_modelgen_save_reload_props_04(self):
         self.client.save(tpr)
 
         tpr.model_dump()
-        self.assertPydanticSerializes(tpr, test_pickle=False)
+        self.assertPydanticSerializes(tpr)
 
     def test_modelgen_save_reload_links_01(self):
         from models.orm import default
@@ -4909,9 +4909,8 @@ def test_modelgen_save_range_01(self):
             Range(dt.date(2025, 1, 6), dt.date(2025, 2, 17)),
         )
 
-        # FIXME: pickle is broken for ranges
-        self.assertPydanticSerializes(r, test_pickle=False)
-        self.assertPydanticSerializes(r2, test_pickle=False)
+        self.assertPydanticSerializes(r)
+        self.assertPydanticSerializes(r2)
 
     def test_modelgen_save_range_02(self):
         import datetime as dt