Skip to content

Commit 6405ef9

Browse files
committed
chore: add coverage
1 parent 0b00227 commit 6405ef9

File tree

3 files changed

+13
-26
lines changed

3 files changed

+13
-26
lines changed

pom.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,9 @@
189189
<exclude>**/br/com/grupo63/techchallenge/payment/api/controller/dto/PaymentStatusResponseDTO.class</exclude>
190190
<exclude>**/br/com/grupo63/techchallenge/payment/api/controller/dto/QRCodeResponseDTO.class</exclude>
191191
<exclude>**/br/com/grupo63/techchallenge/payment/gateway/order/dto/OrderDTO.class</exclude>
192+
<exclude>**/br/com/grupo63/techchallenge/payment/gateway/payment/PaymentPersistenceEntity.class</exclude>
193+
<exclude>**/br/com/grupo63/techchallenge/payment/gateway/payment/PaymentJpaAdapter.class</exclude>
194+
<exclude>**/br/com/grupo63/techchallenge/payment/ServicePaymentApplication.class</exclude>
192195
</excludes>
193196
<formats>
194197
<format>XML</format>

src/main/java/br/com/grupo63/techchallenge/payment/gateway/payment/PaymentJpaRepository.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package br.com.grupo63.techchallenge.payment.gateway.payment;
22

33
import br.com.grupo63.techchallenge.common.gateway.repository.IJpaRepository;
4+
import br.com.grupo63.techchallenge.payment.domain.Payment;
45
import org.springframework.data.jpa.repository.JpaRepository;
56
import org.springframework.data.jpa.repository.Query;
67
import org.springframework.stereotype.Repository;
@@ -12,4 +13,6 @@ public interface PaymentJpaRepository extends JpaRepository<PaymentPersistenceEn
1213

1314
@Query("SELECT p FROM PaymentPersistenceEntity p WHERE p.orderId = :orderId")
1415
Optional<PaymentPersistenceEntity> findByOrderId(Long orderId);
16+
17+
PaymentPersistenceEntity saveAndFlush(PaymentPersistenceEntity paymentPersistenceEntity);
1518
}

src/test/java/br/com/grupo63/techchallenge/payment/PaymentIntegrationTest.java

Lines changed: 7 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,7 @@ class PaymentIntegrationTest {
3838
@Mock
3939
private IOrderGateway orderGateway;
4040
@Mock
41-
private IMercadoPagoGateway mercadoPagoGateway;
42-
@Mock
43-
private IPaymentGateway paymentGateway;
41+
private MercadoPagoGateway mercadoPagoGateway;
4442
private PaymentController paymentController;
4543
private PaymentAPIController paymentAPIController;
4644

@@ -55,15 +53,15 @@ class PaymentIntegrationTest {
5553
@BeforeEach
5654
void setUp() {
5755
MockitoAnnotations.openMocks(this);
58-
paymentUseCase = new PaymentUseCase(mercadoPagoGateway, orderGateway, paymentGateway);
56+
paymentUseCase = new PaymentUseCase(mercadoPagoGateway, orderGateway, paymentJpaAdapter);
5957
paymentController = new PaymentController(paymentUseCase);
6058
paymentAPIController = new PaymentAPIController(paymentController);
6159
}
6260

6361
@SneakyThrows
6462
@Test
6563
public void testGetPaymentStatus_EndToEnd() {
66-
when(paymentGateway.findByOrderId(defaultOrderId)).thenReturn(Optional.of(payment));
64+
when(paymentJpaRepository.findByOrderId(defaultOrderId)).thenReturn(Optional.of(defaultPaymentPersistenceEntity));
6765
when(orderGateway.getOrderById(defaultOrderId)).thenReturn(Optional.of(orderDTO));
6866

6967
ResponseEntity<PaymentStatusResponseDTO> response = paymentAPIController.getStatusByOrderId(defaultOrderId);
@@ -84,30 +82,13 @@ public void testStartPayment_EndToEnd() {
8482
@SneakyThrows
8583
@Test
8684
public void testfinishPayment_EndToEnd() {
87-
when(paymentGateway.findByOrderId(defaultOrderId)).thenReturn(Optional.of(payment));
85+
when(paymentJpaRepository.findByOrderId(defaultOrderId)).thenReturn(Optional.of(defaultPaymentPersistenceEntity));
86+
87+
payment.setStatus(PaymentStatus.PAID);
88+
when(paymentJpaRepository.saveAndFlush(any())).thenReturn(defaultPaymentPersistenceEntity);
8889

8990
ResponseEntity response = paymentAPIController.confirmPaymentFromOrderId(defaultOrderId);
9091
assertEquals(HttpStatus.OK, response.getStatusCode());
9192
}
9293

93-
// @SneakyThrows
94-
// @Test
95-
// public void testUpdate_EndToEnd() {
96-
// when(clientJpaRepository.findByIdAndDeletedFalse(defaultClientPersistenceEntity.getId())).thenReturn(Optional.of(defaultClientPersistenceEntity));
97-
//
98-
// ClientControllerDTO clientControllerDTO = new ClientControllerDTO();
99-
//
100-
// clientControllerDTO.setId(defaultClientPersistenceEntity.getId());
101-
// clientControllerDTO.setNationalId("11111111111");
102-
//
103-
// when(clientJpaRepository.save(any())).thenReturn(new ClientPersistenceEntity(defaultClientPersistenceEntity.getId(), defaultClientPersistenceEntity.isDeleted(), clientControllerDTO.getNationalId()));
104-
//
105-
// ResponseEntity<ClientControllerDTO> response = clientAPIController.update(clientControllerDTO, defaultClientPersistenceEntity.getId());
106-
//
107-
// assertEquals(HttpStatus.OK, response.getStatusCode());
108-
// assertNotNull(response.getBody());
109-
// assertEquals(response.getBody().getNationalId(), clientControllerDTO.getNationalId());
110-
// verify(clientJpaRepository, times(1)).findByIdAndDeletedFalse(any());
111-
// verify(clientJpaRepository, times(1)).save(any());
112-
// }
11394
}

0 commit comments

Comments
 (0)