fix: Make similar to node sdk

Author: KShivendu

supertokens_python/recipe/userroles/interfaces.py

@@ -1,5 +1,5 @@
 from abc import ABC, abstractmethod
-from typing import Any, Dict, List, Union
+from typing import Any, Dict, List, Union, Optional
 
 
 class AddRoleToUserOkResult:
@@ -58,25 +58,33 @@ def __init__(self, roles: List[str]):
 class RecipeInterface(ABC):
     @abstractmethod
     async def add_role_to_user(
-        self, user_id: str, role: str, tenant_id: str, user_context: Dict[str, Any]
+        self,
+        user_id: str,
+        role: str,
+        tenant_id: Optional[str],
+        user_context: Dict[str, Any],
     ) -> Union[AddRoleToUserOkResult, UnknownRoleError]:
         pass
 
     @abstractmethod
     async def remove_user_role(
-        self, user_id: str, role: str, tenant_id: str, user_context: Dict[str, Any]
+        self,
+        user_id: str,
+        role: str,
+        tenant_id: Optional[str],
+        user_context: Dict[str, Any],
     ) -> Union[RemoveUserRoleOkResult, UnknownRoleError]:
         pass
 
     @abstractmethod
     async def get_roles_for_user(
-        self, user_id: str, tenant_id: str, user_context: Dict[str, Any]
+        self, user_id: str, tenant_id: Optional[str], user_context: Dict[str, Any]
     ) -> GetRolesForUserOkResult:
         pass
 
     @abstractmethod
     async def get_users_that_have_role(
-        self, role: str, tenant_id: str, user_context: Dict[str, Any]
+        self, role: str, tenant_id: Optional[str], user_context: Dict[str, Any]
     ) -> Union[GetUsersThatHaveRoleOkResult, UnknownRoleError]:
         pass
 

supertokens_python/recipe/userroles/recipe_implementation.py

@@ -49,7 +49,8 @@ async def add_role_to_user(
     ) -> Union[AddRoleToUserOkResult, UnknownRoleError]:
         params = {"userId": user_id, "role": role}
         response = await self.querier.send_put_request(
-            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/role"), params
+            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/role"),
+            params,
         )
         if response.get("status") == "OK":
             return AddRoleToUserOkResult(
@@ -66,7 +67,10 @@ async def remove_user_role(
     ) -> Union[RemoveUserRoleOkResult, UnknownRoleError]:
         params = {"userId": user_id, "role": role}
         response = await self.querier.send_post_request(
-            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/role/remove"), params
+            NormalisedURLPath(
+                f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/role/remove"
+            ),
+            params,
         )
         if response["status"] == "OK":
             return RemoveUserRoleOkResult(
@@ -79,7 +83,8 @@ async def get_roles_for_user(
     ) -> GetRolesForUserOkResult:
         params = {"userId": user_id}
         response = await self.querier.send_get_request(
-            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/roles"), params
+            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/user/roles"),
+            params,
         )
         return GetRolesForUserOkResult(roles=response["roles"])
 
@@ -88,7 +93,8 @@ async def get_users_that_have_role(
     ) -> Union[GetUsersThatHaveRoleOkResult, UnknownRoleError]:
         params = {"role": role}
         response = await self.querier.send_get_request(
-            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/role/users"), params
+            NormalisedURLPath(f"{tenant_id or DEFAULT_TENANT_ID}/recipe/role/users"),
+            params,
         )
         if response.get("status") == "OK":
             return GetUsersThatHaveRoleOkResult(users=response["users"])