Removed Dev comments

Author: Saiyashwanth7

src/app/api/dependencies.py

@@ -34,10 +34,9 @@ async def get_current_user(
         user = await crud_users.get(db=db, username=token_data.username_or_email, is_deleted=False)
 
     if user:
-        # Ensure consistent return type - always return dict
-        if hasattr(user, 'model_dump'):  # It's a Pydantic model
+        if hasattr(user, 'model_dump'):
             return user.model_dump()
-        else:  # It's already a dict
+        else:
             return user
 
     raise UnauthorizedException("User not authenticated.")

src/app/api/v1/posts.py

@@ -1,5 +1,4 @@
 from typing import Annotated, Any, cast
-
 from fastapi import APIRouter, Depends, Request
 from fastcrud.paginated import PaginatedListResponse, compute_offset, paginated_response
 from sqlalchemy.ext.asyncio import AsyncSession

src/app/api/v1/users.py

@@ -72,7 +72,6 @@ async def read_user(request: Request, username: str, db: Annotated[AsyncSession,
     return cast(UserRead, db_user)
 
 
-# In src/app/api/v1/users.py, replace the patch_user function with this:
 
 @router.patch("/user/{username}")
 async def patch_user(
@@ -86,7 +85,6 @@ async def patch_user(
     if db_user is None:
         raise NotFoundException("User not found")
 
-    # Handle both dict and UserRead object types
     if isinstance(db_user, dict):
         db_username = db_user["username"]
         db_email = db_user["email"]
@@ -97,17 +95,14 @@ async def patch_user(
     if db_username != current_user["username"]:
         raise ForbiddenException()
 
-    # Check for email conflicts if email is being updated
     if values.email is not None and values.email != db_email:
         if await crud_users.exists(db=db, email=values.email):
             raise DuplicateValueException("Email is already registered")
-
-    # Check for username conflicts if username is being updated  
+ 
     if values.username is not None and values.username != db_username:
         if await crud_users.exists(db=db, username=values.username):
             raise DuplicateValueException("Username not available")
 
-    # Update the user
     await crud_users.update(db=db, object=values, username=username)
     return {"message": "User updated"}
 

src/app/core/db/models.py

@@ -1,5 +1,5 @@
 import uuid as uuid_pkg
-from uuid6 import uuid7 #126
+from uuid6 import uuid7
 from datetime import UTC, datetime
 
 from sqlalchemy import Boolean, DateTime, text

src/app/core/schemas.py

@@ -1,5 +1,5 @@
 import uuid as uuid_pkg
-from uuid6 import uuid7 #126
+from uuid6 import uuid7
 from datetime import UTC, datetime
 from typing import Any
 

src/app/core/utils/cache.py

@@ -174,12 +174,12 @@ async def _delete_keys_by_pattern(pattern: str) -> None:
     if client is None:
         return
 
-    cursor = 0  # Make sure cursor starts at 0
+    cursor = 0 
     while True:
         cursor, keys = await client.scan(cursor, match=pattern, count=100)
         if keys:
             await client.delete(*keys)
-        if cursor == 0:  # cursor returns to 0 when scan is complete
+        if cursor == 0: 
             break
 
 

src/app/models/post.py

@@ -1,6 +1,6 @@
 import uuid as uuid_pkg
 from datetime import UTC, datetime
-from uuid6 import uuid7 #126
+from uuid6 import uuid7
 
 from sqlalchemy import DateTime, ForeignKey, String,UUID
 from sqlalchemy.orm import Mapped, mapped_column

src/app/models/user.py

@@ -12,7 +12,6 @@
 class User(Base):
     __tablename__ = "user"
 
-    # Option 1: Use integer ID as primary key (recommended for compatibility)
     id: Mapped[int] = mapped_column(autoincrement=True, primary_key=True, init=False)
 
     name: Mapped[str] = mapped_column(String(30))

tests/test_user.py

@@ -119,13 +119,13 @@ async def test_patch_user_success(self, mock_db, current_user_dict, sample_user_
         username = current_user_dict["username"]
         user_update = UserUpdate(name="New Name")
 
-        # Convert the UserRead model to a dictionary for the mock
+        
         user_dict = sample_user_read.model_dump()
         user_dict["username"] = username
 
         with patch("src.app.api.v1.users.crud_users") as mock_crud:
-            mock_crud.get = AsyncMock(return_value=user_dict)  # Return dict instead of UserRead
-            mock_crud.exists = AsyncMock(return_value=False)  # No conflicts
+            mock_crud.get = AsyncMock(return_value=user_dict)  
+            mock_crud.exists = AsyncMock(return_value=False)  
             mock_crud.update = AsyncMock(return_value=None)
 
             result = await patch_user(Mock(), user_update, username, current_user_dict, mock_db)
@@ -138,12 +138,11 @@ async def test_patch_user_forbidden(self, mock_db, current_user_dict, sample_use
         """Test user update when user tries to update another user."""
         username = "different_user"
         user_update = UserUpdate(name="New Name")
-        # Convert the UserRead model to a dictionary for the mock
         user_dict = sample_user_read.model_dump()
         user_dict["username"] = username
 
         with patch("src.app.api.v1.users.crud_users") as mock_crud:
-            mock_crud.get = AsyncMock(return_value=user_dict)  # Return dict instead of UserRead
+            mock_crud.get = AsyncMock(return_value=user_dict)  
 
             with pytest.raises(ForbiddenException):
                 await patch_user(Mock(), user_update, username, current_user_dict, mock_db)