Update code in endpoints\user.post.py

endpoints/user.post.py

@@ -5,12 +5,12 @@ import uuid
 router = APIRouter()
 
 @router.get("/users/{user_id}")
-async def get_user_handler(
+async def get_user_profile(
     user_id: str,
     db: Session = Depends(get_db),
     token: str = Depends(oauth2_scheme)
 ):
-    """Get user profile by ID"""
+    """Get user profile"""
     user = fake_users_db.get(user_id)
     if not user:
         raise HTTPException(status_code=404, detail="User not found")
@@ -20,7 +20,7 @@ async def get_user_handler(
     }
 
 @router.put("/users/{user_id}")
-async def update_user_handler(
+async def update_user_profile(
     user_id: str,
     user_data: UserUpdateSchema,
     db: Session = Depends(get_db),
@@ -31,40 +31,38 @@ async def update_user_handler(
     if not user:
         raise HTTPException(status_code=404, detail="User not found")
     
-    user.update(user_data.dict())
-    fake_users_db[user_id] = user
+    updated_user = {**user, **user_data.dict()}
+    fake_users_db[user_id] = updated_user
     
     return {
         "message": "User profile updated",
-        "data": user
+        "data": updated_user
     }
 
 @router.delete("/users/{user_id}")
-async def delete_user_handler(
+async def delete_user_profile(
     user_id: str,
     db: Session = Depends(get_db),
     token: str = Depends(oauth2_scheme)
 ):
     """Delete user profile"""
-    user = fake_users_db.pop(user_id, None)
+    user = fake_users_db.get(user_id)
     if not user:
         raise HTTPException(status_code=404, detail="User not found")
     
+    del fake_users_db[user_id]
+    
     return {
         "message": "User profile deleted"
     }
 
 @router.get("/users")
-async def list_users_handler(
+async def get_all_users(
     db: Session = Depends(get_db),
     token: str = Depends(oauth2_scheme)
 ):
-    """List all user profiles"""
-    users = list(fake_users_db.values())
+    """Get all user profiles"""
     return {
-        "message": "Users retrieved",
-        "data": users,
-        "metadata": {
-            "total": len(users)
-        }
+        "message": "All user profiles retrieved",
+        "data": list(fake_users_db.values())
     }