🐛 Fix handling non-primitive parameter values.

Author: rafalkrupinski

ChangeLog.md

@@ -5,12 +5,20 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html),
 and the format of this file is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
+## [Unreleased]
+
+### Fixed
+
+- Use pydantic to serialize non-encapsulated non-primitive method parameters.
+
+
 ## [0.12.1] - 2024-11-19
 
 ### Fixed
 
 - Don't raise exception on responses without content-type header if an empty response map was provided.
 
+
 ## [0.12.0] - 2024-10-21
 
 ### Added

src/lapidary/runtime/model/request.py

@@ -89,7 +89,7 @@ def http_name(self) -> str:
         return self.param.alias or self.python_name
 
 
-class DictParamContributor(ParamContributor):
+class DictParamContributor(ParamContributor, abc.ABC):
     def update_builder(self, builder: RequestBuilder, value: typing.Any) -> None:
         part = self._get_builder_part(builder)
         http_name = self.http_name()
@@ -141,10 +141,11 @@ class ParamsContributor(RequestContributor):
     contributors: Mapping[str, RequestContributor]
 
     def update_builder(self, builder: RequestBuilder, headers_model: pydantic.BaseModel) -> None:
-        raw_model = headers_model.model_dump()
-        for field_name, field_info in headers_model.model_fields.items():
+        raw_model = headers_model.model_dump(mode='json', exclude_unset=True)
+        for field_name in headers_model.model_fields_set:
+            assert field_name in headers_model.model_fields
             value = raw_model[field_name]
-            if not value and field_name not in headers_model.model_fields_set:
+            if value is None:
                 continue
             contributor = self.contributors[field_name]
             contributor.update_builder(builder, value)

tests/test_request.py

@@ -1,3 +1,4 @@
+import datetime as dt
 from collections.abc import Awaitable
 from unittest.mock import AsyncMock, Mock
 
@@ -143,3 +144,29 @@ async def op(
     async with Client(client=mock_http_client) as client:
         with pytest.raises(TypeError):
             await client.op()
+
+
+@pytest.mark.asyncio
+async def test_build_request_param_date(mock_http_client):
+    class Client(ClientTestBase):
+        @get('/request_date')
+        async def request_date(
+            self: typing.Self,
+            date: typing.Annotated[dt.date, Query()],
+        ) -> typing.Annotated[tuple[dt.date, None], Responses({})]:
+            pass
+
+    today = dt.date.today()
+
+    async with Client(client=mock_http_client) as client:
+        with pytest.raises(UnexpectedResponse):
+            await client.request_date(date=today)
+
+    mock_http_client.build_request.assert_called_with(
+        'GET',
+        '/request_date',
+        content=None,
+        params=httpx.QueryParams({'date': today.isoformat()}),
+        headers=httpx.Headers(),
+        cookies=httpx.Cookies(),
+    )