fix(BA-3696): Fix UserCreateSpec/UserUpdateSpec imports and add missing user_uuid to DeleteUserAction

Import UserCreateSpec and UserUpdateSpec from their canonical location
(services/user/types) instead of action submodules that don't re-export
them. Add required user_uuid argument to DeleteUserAction calls in tests.

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

Author: fregataa

src/ai/backend/manager/api/gql/user/resolver/mutation.py

@@ -39,14 +39,9 @@
 from ai.backend.manager.repositories.base.creator import Creator
 from ai.backend.manager.repositories.user.creators import UserCreatorSpec
 from ai.backend.manager.repositories.user.updaters import UserUpdaterSpec
-from ai.backend.manager.services.user.actions.create_user import (
-    BulkCreateUserAction,
-    UserCreateSpec,
-)
-from ai.backend.manager.services.user.actions.modify_user import (
-    BulkModifyUserAction,
-    UserUpdateSpec,
-)
+from ai.backend.manager.services.user.actions.create_user import BulkCreateUserAction
+from ai.backend.manager.services.user.actions.modify_user import BulkModifyUserAction
+from ai.backend.manager.services.user.types import UserCreateSpec, UserUpdateSpec
 from ai.backend.manager.services.user.actions.purge_user import BulkPurgeUserAction
 from ai.backend.manager.types import OptionalState, TriState
 

tests/unit/manager/repositories/user/test_user_repository.py

@@ -51,8 +51,7 @@
 from ai.backend.manager.repositories.user.creators import UserCreatorSpec
 from ai.backend.manager.repositories.user.repository import UserRepository
 from ai.backend.manager.repositories.user.updaters import UserUpdaterSpec
-from ai.backend.manager.services.user.actions.create_user import UserCreateSpec
-from ai.backend.manager.services.user.actions.modify_user import UserUpdateSpec
+from ai.backend.manager.services.user.types import UserCreateSpec, UserUpdateSpec
 from ai.backend.manager.types import OptionalState
 from ai.backend.testutils.db import with_tables
 

tests/unit/manager/services/test_users.py

@@ -358,7 +358,7 @@ async def test_delete_existing_user_returns_success(
         """Delete existing user should return success result."""
         mock_user_repository.soft_delete_user_validated = AsyncMock(return_value=None)
 
-        action = DeleteUserAction(email="test@example.com")
+        action = DeleteUserAction(user_uuid=uuid.uuid4(), email="test@example.com")
 
         result = await service.delete_user(action)
 
@@ -377,7 +377,7 @@ async def test_delete_nonexistent_user_raises_error(
             side_effect=UserNotFound("User not found")
         )
 
-        action = DeleteUserAction(email="nonexistent@example.com")
+        action = DeleteUserAction(user_uuid=uuid.uuid4(), email="nonexistent@example.com")
 
         with pytest.raises(UserNotFound):
             await service.delete_user(action)

tests/unit/manager/services/user/test_user.py

@@ -189,7 +189,7 @@ async def test_delete_user_action_structure(
         """Test that DeleteUserAction works as expected."""
         mock_dependencies["user_repository"].soft_delete_user_validated = AsyncMock()
 
-        action = DeleteUserAction(email="user@example.com")
+        action = DeleteUserAction(user_uuid=uuid.uuid4(), email="user@example.com")
         await user_service.delete_user(action)
 
         mock_dependencies["user_repository"].soft_delete_user_validated.assert_called_once_with(

tests/unit/manager/services/user/test_user_service.py

@@ -46,14 +46,13 @@
 from ai.backend.manager.services.user.actions.create_user import (
     BulkCreateUserAction,
     CreateUserAction,
-    UserCreateSpec,
 )
+from ai.backend.manager.services.user.types import UserCreateSpec, UserUpdateSpec
 from ai.backend.manager.services.user.actions.delete_user import DeleteUserAction
 from ai.backend.manager.services.user.actions.get_user import GetUserAction
 from ai.backend.manager.services.user.actions.modify_user import (
     BulkModifyUserAction,
     ModifyUserAction,
-    UserUpdateSpec,
 )
 from ai.backend.manager.services.user.actions.purge_user import (
     BulkPurgeUserAction,
@@ -744,7 +743,7 @@ async def test_soft_delete_succeeds(
         """Email soft delete calls repository."""
         mock_user_repository.soft_delete_user_validated = AsyncMock(return_value=None)
 
-        action = DeleteUserAction(email="user@example.com")
+        action = DeleteUserAction(user_uuid=uuid.uuid4(), email="user@example.com")
         result = await service.delete_user(action)
 
         assert result is not None
@@ -762,7 +761,7 @@ async def test_nonexistent_email_raises_not_found(
             side_effect=UserNotFound("User not found")
         )
 
-        action = DeleteUserAction(email="missing@example.com")
+        action = DeleteUserAction(user_uuid=uuid.uuid4(), email="missing@example.com")
 
         with pytest.raises(UserNotFound):
             await service.delete_user(action)