fix(BA-5040): Fix test fixtures for RBAC-validated Image processors

Replace MagicMock(spec=ActionValidators) with plain MagicMock and
AsyncMock validators to support nested attribute access and async
validate() calls required by ScopeActionProcessor and
SingleEntityActionProcessor.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: fregataa

tests/component/image/conftest.py

@@ -2,14 +2,13 @@
 
 import uuid
 from collections.abc import AsyncIterator, Callable
-from unittest.mock import MagicMock
+from unittest.mock import AsyncMock, MagicMock
 
 import pytest
 import sqlalchemy as sa
 from sqlalchemy.ext.asyncio.engine import AsyncEngine as SAEngine
 
 from ai.backend.common.container_registry import ContainerRegistryType
-from ai.backend.manager.actions.validators import ActionValidators
 from ai.backend.manager.api.rest.admin.handler import AdminHandler
 from ai.backend.manager.api.rest.admin.registry import register_admin_routes
 from ai.backend.manager.api.rest.image.handler import ImageHandler
@@ -37,8 +36,11 @@ def image_processors(
 ) -> ImageProcessors:
     repo = ImageRepository(database_engine, valkey_clients.image, config_provider)
     service = ImageService(agent_registry, repo, config_provider)
+    mock_validators = MagicMock()
+    mock_validators.rbac.scope.validate = AsyncMock()
+    mock_validators.rbac.single_entity.validate = AsyncMock()
     return ImageProcessors(
-        service=service, action_monitors=[], validators=MagicMock(spec=ActionValidators)
+        service=service, action_monitors=[], validators=mock_validators
     )
 
 

tests/unit/manager/services/image/test_image_service.py

@@ -20,7 +20,6 @@
 from ai.backend.common.dto.agent.response import PurgeImageResp, PurgeImagesResp
 from ai.backend.common.exception import UnknownImageReference
 from ai.backend.common.types import AgentId, ImageCanonical, ImageID, SlotName
-from ai.backend.manager.actions.validators import ActionValidators
 from ai.backend.manager.data.container_registry.types import ContainerRegistryData
 from ai.backend.manager.data.image.types import (
     ImageAliasData,
@@ -119,7 +118,10 @@ def image_service(
     @pytest.fixture
     def processors(self, image_service: ImageService) -> ImageProcessors:
         """Create ImageProcessors with mock ImageService."""
-        return ImageProcessors(image_service, [], MagicMock(spec=ActionValidators))
+        mock_validators = MagicMock()
+        mock_validators.rbac.scope.validate = AsyncMock()
+        mock_validators.rbac.single_entity.validate = AsyncMock()
+        return ImageProcessors(image_service, [], mock_validators)
 
     @pytest.fixture
     def container_registry_id(self) -> uuid.UUID: