fix(api): add Optional field defaults and pragma: no cover (#66)

- Add = None defaults to Optional fields in PlayerRequestModel and
  PlayerResponseModel (Pydantic v2 requires explicit defaults to make
  fields omissible; Optional[str] alone only allows None when provided)
- Add # pragma: no cover to SQLAlchemyError handlers in player_service.py
  (DB failure paths that require a live SQLAlchemy error to exercise)

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: nanotaboada

models/player_model.py

@@ -55,15 +55,15 @@ class PlayerRequestModel(MainModel):
     """
 
     first_name: str
-    middle_name: Optional[str]
+    middle_name: Optional[str] = None
     last_name: str
-    date_of_birth: Optional[str]
+    date_of_birth: Optional[str] = None
     squad_number: int
     position: str
-    abbr_position: Optional[str]
-    team: Optional[str]
-    league: Optional[str]
-    starting11: Optional[bool]
+    abbr_position: Optional[str] = None
+    team: Optional[str] = None
+    league: Optional[str] = None
+    starting11: Optional[bool] = None
 
 
 class PlayerResponseModel(MainModel):
@@ -89,12 +89,12 @@ class PlayerResponseModel(MainModel):
 
     id: UUID
     first_name: str
-    middle_name: Optional[str]
+    middle_name: Optional[str] = None
     last_name: str
-    date_of_birth: Optional[str]
+    date_of_birth: Optional[str] = None
     squad_number: int
     position: str
-    abbr_position: Optional[str]
-    team: Optional[str]
-    league: Optional[str]
-    starting11: Optional[bool]
+    abbr_position: Optional[str] = None
+    team: Optional[str] = None
+    league: Optional[str] = None
+    starting11: Optional[bool] = None

services/player_service.py

@@ -52,7 +52,7 @@ async def create_async(
         await async_session.commit()
         await async_session.refresh(player)
         return player
-    except SQLAlchemyError as error:
+    except SQLAlchemyError as error:  # pragma: no cover
         logger.exception("Error trying to create the Player: %s", error)
         await async_session.rollback()
         return None
@@ -146,7 +146,7 @@ async def update_async(
     try:
         await async_session.commit()
         return True
-    except SQLAlchemyError as error:
+    except SQLAlchemyError as error:  # pragma: no cover
         logger.exception("Error trying to update the Player: %s", error)
         await async_session.rollback()
         return False
@@ -171,7 +171,7 @@ async def delete_async(async_session: AsyncSession, player_id: UUID) -> bool:
     try:
         await async_session.commit()
         return True
-    except SQLAlchemyError as error:
+    except SQLAlchemyError as error:  # pragma: no cover
         logger.exception("Error trying to delete the Player: %s", error)
         await async_session.rollback()
         return False