Empty string for user retrieving not allowed

Author: IgorChvyrov-sm

src/conductor/asyncio_client/adapters/api/user_resource_api.py

@@ -1,3 +1,5 @@
+from pydantic import StrictStr
+
 from conductor.asyncio_client.http.api import UserResourceApi
 
 
@@ -11,3 +13,13 @@ async def get_granted_permissions(
         if not user_id:
             user_id = None
         return await super().get_granted_permissions(user_id=user_id, *args, **kwargs)
+
+    async def get_user(
+        self,
+        id: StrictStr,
+        *args,
+        **kwargs,
+    ) -> object:
+        if not id:
+            id = None
+        return await super().get_user(id=id, *args, **kwargs)

src/conductor/client/adapters/api/user_resource_api_adapter.py

@@ -6,3 +6,8 @@ def get_granted_permissions(self, user_id, **kwargs):
         if not user_id:
             user_id = None
         return super().get_granted_permissions(user_id=user_id, **kwargs)
+
+    def get_user(self, id, **kwargs):
+        if not id:
+            id = None
+        return super().get_user(id=id, **kwargs)

tests/unit/orkes/test_async_authorization_client.py

@@ -263,6 +263,15 @@ async def test_get_user(mocker, authorization_client, conductor_user_adapter):
     assert user.uuid == USER_UUID
 
 
+@pytest.mark.asyncio
+async def test_get_user_with_empty_string(mocker, authorization_client, conductor_user_adapter):
+    from conductor.asyncio_client.http.api import UserResourceApi
+    mock = mocker.patch.object(UserResourceApi, "get_user")
+    mock.return_value = conductor_user_adapter
+    await authorization_client.get_user("")
+    mock.assert_called_with(id=None)
+
+
 @pytest.mark.asyncio
 async def test_delete_user(mocker, authorization_client):
     mock = mocker.patch.object(UserResourceApiAdapter, "delete_user")

tests/unit/orkes/test_authorization_client.py

@@ -312,6 +312,14 @@ def test_get_user(mocker, authorization_client, conductor_user, roles):
     assert user.roles == roles
 
 
+def test_get_user_with_empty_string(mocker, authorization_client, conductor_user):
+    from conductor.client.codegen.api.user_resource_api import UserResourceApi
+    mock = mocker.patch.object(UserResourceApi, "get_user")
+    mock.return_value = conductor_user.to_dict()
+    authorization_client.get_user("")
+    mock.assert_called_with(id=None)
+
+
 def test_list_users_with_apps(mocker, authorization_client, conductor_user):
     mock = mocker.patch.object(UserResourceApi, "list_users")
     mock.return_value = [conductor_user]