remove try except and add EmailNotVerifiedError

Author: santanu2310

server/app/api/user/repository.py

@@ -12,18 +12,13 @@
 async def find_user_by_username_email(
     db: AsyncDatabase, username: str, email: str
 ) -> Optional[UserAuthOut]:
-    try:
-        user_auth = await db.user_auth.find_one(
-            {"$or": [{"username": username}, {"email": email}]}
-        )
-        if not user_auth:
-            return None
-
-        return UserAuthOut(**user_auth)
+    user_auth = await db.user_auth.find_one(
+        {"$or": [{"username": username}, {"email": email}]}
+    )
+    if not user_auth:
+        return None
 
-    except PyMongoError as e:
-        logger.critical(f"Failed to query db for user returning error: {e}")
-        raise InternalServerError()
+    return UserAuthOut(**user_auth)
 
 
 async def create_user(db: AsyncDatabase, user: UserRegistration) -> ObjectId:

server/app/core/exceptions.py

@@ -1,18 +1,35 @@
 from fastapi import status
 
+
 class AppException(Exception):
     """Base application exception."""
+
     def __init__(self, status_code: int, detail: str):
         self.status_code = status_code
         self.detail = detail
         super().__init__(detail)
 
+
 class UserAlreadyExistsError(AppException):
     """Exception raised when a user tries to register with an email or username that already exists."""
-    def __init__(self, detail: str = "User with this email or username already exists."):
+
+    def __init__(
+        self, detail: str = "User with this email or username already exists."
+    ):
         super().__init__(status_code=status.HTTP_409_CONFLICT, detail=detail)
 
+
 class InternalServerError(AppException):
     """Exception raised for internal server errors."""
+
     def __init__(self, detail: str = "An internal server error occurred."):
-        super().__init__(status_code=status.HTTP_500_INTERNAL_SERVER_ERROR, detail=detail)
+        super().__init__(
+            status_code=status.HTTP_500_INTERNAL_SERVER_ERROR, detail=detail
+        )
+
+
+class EmailNotVerifiedError(AppException):
+    """ " Exception raised when a user's email is not verified"""
+
+    def __init__(self, detail: str = "User email is not verified"):
+        super().__init__(status_code=status.HTTP_403_FORBIDDEN, detail=detail)