test: fix ci

Author: Datata1

src/codesphere/core/handler.py

@@ -3,7 +3,7 @@
 from typing import Any, List, Optional, Type, get_args, get_origin
 
 import httpx
-from pydantic import BaseModel, PrivateAttr, ValidationError
+from pydantic import BaseModel, PrivateAttr, RootModel, ValidationError
 from pydantic.fields import FieldInfo
 
 from ..http_client import APIHttpClient
@@ -93,6 +93,14 @@ async def _make_request(
     def _inject_client_into_model(self, model_instance: BaseModel) -> BaseModel:
         if hasattr(model_instance, "_http_client"):
             model_instance._http_client = self.http_client
+
+        if isinstance(model_instance, RootModel) and isinstance(
+            model_instance.root, list
+        ):
+            for item in model_instance.root:
+                if hasattr(item, "_http_client"):
+                    item._http_client = self.http_client
+
         return model_instance
 
     async def _parse_and_validate_response(

tests/core/test_handler.py

@@ -1,8 +1,8 @@
 import pytest
-from typing import Optional
+from typing import Optional, List
 from unittest.mock import MagicMock
 
-from pydantic import BaseModel, Field, PrivateAttr
+from pydantic import BaseModel, Field, PrivateAttr, RootModel
 
 from codesphere.core.handler import _APIOperationExecutor, APIRequestHandler
 from codesphere.core.operations import APIOperation, AsyncCallable
@@ -147,3 +147,35 @@ class ModelWithClient(BaseModel):
         instance = ModelWithClient(id=1)
         handler._inject_client_into_model(instance)
         assert instance._http_client is mock_executor._http_client
+
+    @pytest.mark.asyncio
+    async def test_inject_client_into_root_model_items(
+        self, mock_executor, sample_operation
+    ):
+        """RootModel containers should have _http_client injected into each item in .root"""
+        mock_client = MagicMock()
+        mock_client.request = MagicMock()
+        mock_executor._http_client = mock_client
+
+        handler = APIRequestHandler(
+            executor=mock_executor,
+            operation=sample_operation,
+            kwargs={},
+        )
+
+        class ItemWithClient(BaseModel):
+            id: int
+            _http_client: Optional[MagicMock] = PrivateAttr(default=None)
+
+        class ResourceList(RootModel[List[ItemWithClient]]):
+            _http_client: Optional[MagicMock] = PrivateAttr(default=None)
+
+        item1 = ItemWithClient(id=1)
+        item2 = ItemWithClient(id=2)
+        resource_list = ResourceList(root=[item1, item2])
+
+        handler._inject_client_into_model(resource_list)
+
+        assert resource_list._http_client is mock_executor._http_client
+        for item in resource_list.root:
+            assert item._http_client is mock_executor._http_client

tests/resources/team/domain/test_domain.py

@@ -30,6 +30,20 @@ async def test_list_domains(self, domain_manager, sample_domain_data):
         assert len(result) == 1
         assert isinstance(result[0], Domain)
 
+    @pytest.mark.asyncio
+    async def test_list_items_have_http_client_injected(
+        self, mock_http_client_for_resource, sample_domain_data
+    ):
+        """Items returned from list() should have _http_client injected."""
+        mock_client = mock_http_client_for_resource([sample_domain_data])
+        manager = TeamDomainManager(http_client=mock_client, team_id=12345)
+
+        result = await manager.list()
+
+        for domain in result:
+            assert hasattr(domain, "_http_client")
+            assert domain._http_client is not None
+
     @pytest.mark.asyncio
     async def test_get_domain(self, mock_http_client_for_resource, sample_domain_data):
         """Get domain should return a single Domain model."""

tests/resources/team/test_team.py

@@ -17,6 +17,21 @@ async def test_list_teams(self, teams_resource_factory, sample_team_list_data):
         assert len(result) == 2
         assert all(isinstance(team, Team) for team in result)
 
+    @pytest.mark.asyncio
+    async def test_list_items_have_http_client_injected(
+        self, teams_resource_factory, sample_team_list_data
+    ):
+        """Items returned from list() should have _http_client injected."""
+        resource, mock_client = teams_resource_factory(sample_team_list_data)
+
+        result = await resource.list()
+
+        for team in result:
+            assert hasattr(team, "_http_client")
+            assert team._http_client is not None
+            # Verify sub-resources are accessible without "detached model" error
+            _ = team.domains
+
     @pytest.mark.asyncio
     async def test_list_teams_empty(self, teams_resource_factory):
         """List teams should handle empty response."""

tests/resources/workspace/test_workspace.py

@@ -63,6 +63,20 @@ async def test_create_workspace(
         assert isinstance(result, Workspace)
         mock_client.request.assert_awaited_once()
 
+    @pytest.mark.asyncio
+    async def test_list_items_have_http_client_injected(
+        self, workspaces_resource_factory, sample_workspace_list_data
+    ):
+        """Items returned from list() should have _http_client injected and be able to call instance methods."""
+        resource, mock_client = workspaces_resource_factory(sample_workspace_list_data)
+
+        result = await resource.list(team_id=12345)
+
+        for workspace in result:
+            assert hasattr(workspace, "_http_client")
+            assert workspace._http_client is not None
+            _ = workspace.env_vars
+
 
 class TestWorkspaceModel:
     """Tests for the Workspace model and its methods."""