feat(order,payment): more usecase validations

Author: g-otn

techchallenge/src/main/java/br/com/grupo63/techchallenge/adapter/in/controller/order/OrderController.java

@@ -3,15 +3,13 @@
 import br.com.grupo63.techchallenge.adapter.in.controller.AbstractController;
 import br.com.grupo63.techchallenge.adapter.in.controller.dto.DefaultResponseDTO;
 import br.com.grupo63.techchallenge.adapter.in.controller.order.dto.CreateOrderRequestDTO;
-import br.com.grupo63.techchallenge.core.application.usecase.dto.ClientDTO;
 import br.com.grupo63.techchallenge.core.application.usecase.dto.OrderDTO;
 import br.com.grupo63.techchallenge.core.application.usecase.exception.NotFoundException;
 import br.com.grupo63.techchallenge.core.application.usecase.exception.ValidationException;
 import br.com.grupo63.techchallenge.core.application.usecase.order.IOrderUseCase;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.tags.Tag;
-import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -43,7 +41,7 @@ public ResponseEntity<List<OrderDTO>> listUnfinishedOrders() {
     @PostMapping("/avancar-estado")
     @ResponseStatus(HttpStatus.OK)
     public void advanceOrderStatusFromOrderId(@Parameter(description = "Id do pedido.") @RequestParam Long orderId) throws NotFoundException, ValidationException {
-        orderUseCase.advanceOrderStatus(orderId);
+        orderUseCase.advanceStatus(orderId);
     }
 
     @Operation(

techchallenge/src/main/java/br/com/grupo63/techchallenge/adapter/in/controller/payment/PaymentController.java

@@ -29,7 +29,7 @@ public class PaymentController extends AbstractController {
     @PostMapping("/iniciar")
     public QRCodeResponseDTO startPayment(
             @Parameter(description = "Id do pedido") @RequestParam Long orderId
-    ) throws NotFoundException {
+    ) throws NotFoundException, ValidationException {
         String qrData = paymentUseCase.startPayment(orderId);
         return new QRCodeResponseDTO(qrData);
     }

techchallenge/src/main/java/br/com/grupo63/techchallenge/core/application/usecase/order/IOrderUseCase.java

@@ -10,7 +10,7 @@
 
 public interface IOrderUseCase extends ICRUDUseCase<OrderDTO> {
 
-    void advanceOrderStatus(@NotNull Long orderId) throws NotFoundException, ValidationException;
+    void advanceStatus(@NotNull Long orderId) throws NotFoundException, ValidationException;
 
     List<OrderDTO> listUnfinishedOrders();
 

techchallenge/src/main/java/br/com/grupo63/techchallenge/core/application/usecase/order/OrderUseCase.java

@@ -9,6 +9,7 @@
 import br.com.grupo63.techchallenge.core.domain.model.order.Order;
 import br.com.grupo63.techchallenge.core.domain.model.order.OrderItem;
 import br.com.grupo63.techchallenge.core.domain.model.order.OrderStatus;
+import br.com.grupo63.techchallenge.core.domain.model.payment.PaymentStatus;
 import jakarta.validation.constraints.NotNull;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
@@ -31,7 +32,7 @@ public class OrderUseCase implements IOrderUseCase {
 
     private void fillCurrentPrices(Order order) throws NotFoundException {
         double totalPrice = 0.0D;
-        for (OrderItem orderItem: order.getItems()) {
+        for (OrderItem orderItem : order.getItems()) {
             ProductDTO productDTO = productUseCase.read(orderItem.getProduct().getId());
 
             orderItem.setPrice(productDTO.getPrice());
@@ -41,14 +42,16 @@ private void fillCurrentPrices(Order order) throws NotFoundException {
     }
 
     @Override
-    public void advanceOrderStatus(@NotNull Long orderId) throws NotFoundException, ValidationException {
+    public void advanceStatus(@NotNull Long orderId) throws NotFoundException, ValidationException {
         Order order = orderRepository.findByIdAndDeletedFalse(orderId).orElseThrow(NotFoundException::new);
 
-        if (order.getStatus() == null) {
-            order.setStatus(OrderStatus.RECEIVED);
+        if (order.getPayment() == null || order.getPayment().getStatus() != PaymentStatus.PAID) {
+            throw new ValidationException("order.advanceStatus.title", "order.advanceStatus.notPaid");
         } else if (order.getStatus() == OrderStatus.FINISHED) {
-            throw new ValidationException("order.cantAdvance.done.title", "order.cantAdvance.done.description");
-        } else {
+            throw new ValidationException("order.advanceStatus.title", "order.advanceStatus.finished");
+        } else if (order.getStatus() == null) {
+            order.setStatus(OrderStatus.RECEIVED);
+        }  else {
             order.setStatus(nextOrderMap.get(order.getStatus()));
         }
 

techchallenge/src/main/java/br/com/grupo63/techchallenge/core/application/usecase/payment/IPaymentUseCase.java

@@ -7,7 +7,7 @@
 
 public interface IPaymentUseCase {
 
-    String startPayment(@NotNull Long orderId) throws NotFoundException;
+    String startPayment(@NotNull Long orderId) throws NotFoundException, ValidationException;
 
     void finishPayment(@NotNull Long orderId) throws ValidationException, NotFoundException;
 

techchallenge/src/main/java/br/com/grupo63/techchallenge/core/application/usecase/payment/PaymentUseCase.java

@@ -20,9 +20,15 @@ public class PaymentUseCase implements IPaymentUseCase {
     private final OrderUseCase orderUseCase;
 
     @Override
-    public String startPayment(@NotNull(message = "payment.order.id.notNull") Long orderId) throws NotFoundException {
+    public String startPayment(@NotNull(message = "payment.order.id.notNull") Long orderId) throws NotFoundException, ValidationException {
         OrderDTO orderDTO = orderUseCase.read(orderId);
 
+        if (orderDTO.getStatus() != null) {
+            throw new ValidationException("payment.startPayment.title", "payment.startPayment.alreadyFinished");
+        } else if (orderDTO.getPayment() != null) {
+            throw new ValidationException("payment.startPayment.title", "payment.startPayment.alreadyStarted");
+        }
+
         String qrData = mercadoPagoService.generateQRCode(orderDTO.getId(), orderDTO.getTotalPrice());
 
         orderDTO.setPayment(new PaymentDTO(PaymentStatus.PENDING, PaymentMethod.MERCADO_PAGO_QR_CODE, qrData));
@@ -43,7 +49,7 @@ public void finishPayment(@NotNull(message = "payment.order.id.notNull") Long or
 
         orderDTO.getPayment().setStatus(PaymentStatus.PAID);
         orderUseCase.update(orderDTO, orderId);
-        orderUseCase.advanceOrderStatus(orderId);
+        orderUseCase.advanceStatus(orderId);
     }
 
     @Override

techchallenge/src/main/resources/messages_pt_BR.properties

@@ -6,13 +6,16 @@ default.title.notFoundError=Não encontrado
 default.title.notFoundError.description=O objeto que esta tentando acessar não foi encontrado
 
 # Order
-order.cantAdvance.done.title=Ordem já finalizada
-order.cantAdvance.done.description=Não é possível avançar uma ordem ja finalizada
+order.advanceStatus.title=Erro ao avançar status do pedido
+order.advanceStatus.finished=Não é possível avançar um pedido já finalizado
+order.advanceStatus.notPaid=O pedido ainda não foi pago
 
 # Payment
 payment.order.id.notNull=O id do pedido associado ao pagamento é obrigatório
-payment.qrCode.title=Erro ao criar QR Code
-payment.notStarted=O processo de pagamento ainda não foi iniciado para esse pedido
+payment.startPayment.title=Erro ao iniciar pagamento
+payment.startPayment.alreadyFinished=O pagamento já foi finalizado
+payment.startPayment.alreadyStarted=O pagamento já foi iniciado
+payment.notStarted=O processo de pagamento ainda não foi iniciado para esse pedido
 payment.confirm.title=Erro ao finalizar pagamento
 payment.confirm.alreadyPaid=O pagamento já foi finalizado