🐛 fix: update user update method to exclude None values and improve error handling for no changes provided

Author: Whyle

app/repositories/user_repository.py

@@ -106,7 +106,7 @@ async def update_user(self, user_id: EmailStr, user_data: schemas.UserUpdate):
         if not user_current_data:
             raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="User not found")
 
-        provided_data = user_data.model_dump(exclude_unset=True)
+        provided_data = user_data.model_dump(exclude_unset=True, exclude_none=True)
         if not provided_data:
             raise HTTPException(
                 status_code=status.HTTP_400_BAD_REQUEST,

tests/repositories/test_user_repository.py

@@ -231,7 +231,7 @@ async def test__unit_test__update_user_error(user_repository, mock_database):
 
 @pytest.mark.asyncio
 async def test__unit_test__update_user_no_changes_provide(user_repository, mock_database):
-    update_data = UserUpdate()
+    update_data = UserUpdate(_id=None)
     with pytest.raises(HTTPException) as exc_info:
         await user_repository.update_user("user@test.com", update_data)
     assert exc_info.value.status_code == 400