Make tenant id compulsory in userrolesl

Author: KShivendu

supertokens_python/recipe/userroles/asyncio/__init__.py

@@ -1,5 +1,6 @@
 from typing import Any, Dict, List, Union, Optional
 
+from supertokens_python.recipe.multitenancy.constants import DEFAULT_TENANT_ID
 from supertokens_python.recipe.userroles.interfaces import (
     AddRoleToUserOkResult,
     CreateNewRoleOrAddPermissionsOkResult,
@@ -25,7 +26,7 @@ async def add_role_to_user(
     if user_context is None:
         user_context = {}
     return await UserRolesRecipe.get_instance().recipe_implementation.add_role_to_user(
-        user_id, role, tenant_id, user_context
+        user_id, role, tenant_id or DEFAULT_TENANT_ID, user_context
     )
 
 
@@ -38,7 +39,7 @@ async def remove_user_role(
     if user_context is None:
         user_context = {}
     return await UserRolesRecipe.get_instance().recipe_implementation.remove_user_role(
-        user_id, role, tenant_id, user_context
+        user_id, role, tenant_id or DEFAULT_TENANT_ID, user_context
     )
 
 
@@ -51,7 +52,7 @@ async def get_roles_for_user(
         user_context = {}
     return (
         await UserRolesRecipe.get_instance().recipe_implementation.get_roles_for_user(
-            user_id, tenant_id, user_context
+            user_id, tenant_id or DEFAULT_TENANT_ID, user_context
         )
     )
 
@@ -64,7 +65,7 @@ async def get_users_that_have_role(
     if user_context is None:
         user_context = {}
     return await UserRolesRecipe.get_instance().recipe_implementation.get_users_that_have_role(
-        role, tenant_id, user_context
+        role, tenant_id or DEFAULT_TENANT_ID, user_context
     )
 
 

supertokens_python/recipe/userroles/interfaces.py

@@ -61,7 +61,7 @@ async def add_role_to_user(
         self,
         user_id: str,
         role: str,
-        tenant_id: Optional[str],
+        tenant_id: str,
         user_context: Dict[str, Any],
     ) -> Union[AddRoleToUserOkResult, UnknownRoleError]:
         pass
@@ -71,20 +71,20 @@ async def remove_user_role(
         self,
         user_id: str,
         role: str,
-        tenant_id: Optional[str],
+        tenant_id: str,
         user_context: Dict[str, Any],
     ) -> Union[RemoveUserRoleOkResult, UnknownRoleError]:
         pass
 
     @abstractmethod
     async def get_roles_for_user(
-        self, user_id: str, tenant_id: Optional[str], user_context: Dict[str, Any]
+        self, user_id: str, tenant_id: str, user_context: Dict[str, Any]
     ) -> GetRolesForUserOkResult:
         pass
 
     @abstractmethod
     async def get_users_that_have_role(
-        self, role: str, tenant_id: Optional[str], user_context: Dict[str, Any]
+        self, role: str, tenant_id: str, user_context: Dict[str, Any]
     ) -> Union[GetUsersThatHaveRoleOkResult, UnknownRoleError]:
         pass
 

supertokens_python/recipe/userroles/recipe_implementation.py

@@ -44,7 +44,7 @@ async def add_role_to_user(
         self,
         user_id: str,
         role: str,
-        tenant_id: Optional[str],
+        tenant_id: str,
         user_context: Dict[str, Any],
     ) -> Union[AddRoleToUserOkResult, UnknownRoleError]:
         params = {"userId": user_id, "role": role}
@@ -62,7 +62,7 @@ async def remove_user_role(
         self,
         user_id: str,
         role: str,
-        tenant_id: Optional[str],
+        tenant_id: str,
         user_context: Dict[str, Any],
     ) -> Union[RemoveUserRoleOkResult, UnknownRoleError]:
         params = {"userId": user_id, "role": role}
@@ -79,7 +79,7 @@ async def remove_user_role(
         return UnknownRoleError()
 
     async def get_roles_for_user(
-        self, user_id: str, tenant_id: Optional[str], user_context: Dict[str, Any]
+        self, user_id: str, tenant_id: str, user_context: Dict[str, Any]
     ) -> GetRolesForUserOkResult:
         params = {"userId": user_id}
         response = await self.querier.send_get_request(
@@ -89,7 +89,7 @@ async def get_roles_for_user(
         return GetRolesForUserOkResult(roles=response["roles"])
 
     async def get_users_that_have_role(
-        self, role: str, tenant_id: Optional[str], user_context: Dict[str, Any]
+        self, role: str, tenant_id: str, user_context: Dict[str, Any]
     ) -> Union[GetUsersThatHaveRoleOkResult, UnknownRoleError]:
         params = {"role": role}
         response = await self.querier.send_get_request(