Feature #59 Get All Players

[AZapata27]

📝 Description

This PR introduces the following changes:

• Added endpoint to retrieve all players.
• Implemented unit tests for the GetAllPlayers handler.
• Created mock implementation for PlayerRepository to facilitate testing.
• Adjusted GetAllPlayers handler to return 204 No Content when no players are found.

Type of Change

• Bug fix (non-breaking change which fixes an issue)

• New feature (non-breaking change which adds functionality)

• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• Existing unit tests pass locally with my changes
• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• Any dependent changes have been merged and published in downstream modules

[DarcoProgramador]

No hay ninguna issue creada para este flow, así que te creare una nueva issue

[DarcoProgramador]

Buen código 👍, me parece excelente. Aunque la issue de este flow no estaba creada. puede servirnos mas tarde tener un GetAllPlayers.
Solo faltaría corregir el comentario para el go doc y añadir el comentario para la doc de swagger y listo.

[AZapata27]

Gracias por los comentarios @DarcoProgramador estoy en camino de subir de nivel con GO y me gustaría colaborar en la comunidad.

Reviews ajustadas ok 👍🏼

[DarcoProgramador]

Tienes que buildear de nuevo el mock ya que el linter y los test dan error ya que no existe la función GetAll en los mock de Players

[AZapata27]

ajustado el mock

[jairogloz]

Una pregunta, este endpoint retorna toooodos los players en la base de datos, cierto? 🤔