perf(DeleteUser): cambiadas funciones de boolean a void

src/main/java/com/example/apprecetas/user/application/DeleteUserUseCase.java

@@ -2,6 +2,6 @@ package com.example.apprecetas.user.application;
 
 
 public interface DeleteUserUseCase {
 public interface DeleteUserUseCase {
 
 
-    boolean deleteById(Long id);
+    void deleteById(Long id);
 
 
 }
 }

src/main/java/com/example/apprecetas/user/application/impl/DeleteUserUseCaseImpl.java

@@ -3,6 +3,7 @@ package com.example.apprecetas.user.application.impl;
 import com.example.apprecetas.exceptions.EntityNotFoundException;
 import com.example.apprecetas.exceptions.EntityNotFoundException;
 import com.example.apprecetas.user.application.DeleteUserUseCase;
 import com.example.apprecetas.user.application.DeleteUserUseCase;
 import com.example.apprecetas.user.domain.repository.DeleteUserRepository;
 import com.example.apprecetas.user.domain.repository.DeleteUserRepository;
+import com.example.apprecetas.user.domain.repository.ReadUserRepository;
 import lombok.RequiredArgsConstructor;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.stereotype.Service;
 
 
@@ -10,13 +11,14 @@ import org.springframework.stereotype.Service;
 @RequiredArgsConstructor
 @RequiredArgsConstructor
 public class DeleteUserUseCaseImpl implements DeleteUserUseCase {
 public class DeleteUserUseCaseImpl implements DeleteUserUseCase {
 
 
-    private final DeleteUserRepository repository;
+    private final DeleteUserRepository deleteUserRepository;
+
+    private final ReadUserRepository readUserRepository;
 
 
     @Override
     @Override
-    public boolean deleteById(Long id) {
-        boolean isDeleted = repository.deleteById(id);
-        if (!isDeleted)
-            throw new EntityNotFoundException("Usuario con id " + id + " no encontrado");
-        return isDeleted;
+    public void deleteById(Long id) {
+        readUserRepository.readById(id)
+                .orElseThrow(() -> new EntityNotFoundException("El usuario con id " + id + " no existe."));
+        deleteUserRepository.deleteById(id);
     }
     }
 }
 }

src/main/java/com/example/apprecetas/user/domain/repository/DeleteUserRepository.java

@@ -2,6 +2,6 @@ package com.example.apprecetas.user.domain.repository;
 
 
 public interface DeleteUserRepository {
 public interface DeleteUserRepository {
 
 
-    boolean deleteById(Long id);
+    void deleteById(Long id);
 
 
 }
 }

src/main/java/com/example/apprecetas/user/infrastructure/controller/DeleteUserController.java

@@ -18,10 +18,8 @@ public class DeleteUserController {
 
 
     @DeleteMapping("/{id}")
     @DeleteMapping("/{id}")
     public ResponseEntity<String> deleteById(@PathVariable Long id) {
     public ResponseEntity<String> deleteById(@PathVariable Long id) {
-        boolean isDeleted = service.deleteById(id);
-        if(isDeleted)
+        service.deleteById(id);
         return ResponseEntity.ok().body("Usuario con id " + id + " eliminado correctamente");
         return ResponseEntity.ok().body("Usuario con id " + id + " eliminado correctamente");
-        return ResponseEntity.notFound().build();
     }
     }
 
 
 }
 }

src/main/java/com/example/apprecetas/user/infrastructure/repository/impl/DeleteUserRepositoryImpl.java

@@ -12,12 +12,7 @@ public class DeleteUserRepositoryImpl implements DeleteUserRepository {
     private final UserRepositoryJpa repositoryJpa;
     private final UserRepositoryJpa repositoryJpa;
 
 
     @Override
     @Override
-    public boolean deleteById(Long id) {
-        boolean isDeleted = false;
-        if(repositoryJpa.existsById(id)) {
+    public void deleteById(Long id) {
         repositoryJpa.deleteById(id);
         repositoryJpa.deleteById(id);
-            isDeleted = true;
-        }
-        return isDeleted;
     }
     }
 }
 }