refactor(service): use delete(entity) instead of deleteById in deleteBySquadNumber (#268)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: nanotaboada

src/main/java/ar/com/nanotaboada/java/samples/spring/boot/services/PlayersService.java

@@ -243,7 +243,7 @@ public boolean deleteBySquadNumber(Integer squadNumber) {
 
         return playersRepository.findBySquadNumber(squadNumber)
                 .map(existing -> {
-                    playersRepository.deleteById(existing.getId());
+                    playersRepository.delete(existing);
                     log.info("Player deleted successfully - Squad Number: {}", squadNumber);
                     return true;
                 })

src/test/java/ar/com/nanotaboada/java/samples/spring/boot/test/services/PlayersServiceTests.java

@@ -415,7 +415,7 @@ void givenPlayerExists_whenDelete_thenReturnsTrue() {
         boolean actual = playersService.deleteBySquadNumber(squadNumber);
         // Then
         verify(playersRepositoryMock, times(1)).findBySquadNumber(squadNumber);
-        verify(playersRepositoryMock, times(1)).deleteById(entity.getId());
+        verify(playersRepositoryMock, times(1)).delete(entity);
         then(actual).isTrue();
     }