Skip to content

Commit 2196925

Browse files
committed
fixed code
1 parent 67d1723 commit 2196925

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

src/main/java/it/gov/pagopa/paymentupdater/service/PaymentServiceImpl.java

+2-6
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import org.springframework.transaction.annotation.Transactional;
1717
import org.springframework.web.client.HttpClientErrorException;
1818
import org.springframework.web.client.HttpServerErrorException;
19+
import org.springframework.web.client.HttpStatusCodeException;
1920
import org.springframework.web.client.RestTemplate;
2021

2122
import com.fasterxml.jackson.core.JsonProcessingException;
@@ -27,10 +28,8 @@
2728
import it.gov.pagopa.paymentupdater.model.Payment;
2829
import it.gov.pagopa.paymentupdater.producer.PaymentProducer;
2930
import it.gov.pagopa.paymentupdater.repository.PaymentRepository;
30-
import it.gov.pagopa.paymentupdater.restclient.proxy.ApiClient;
3131
import it.gov.pagopa.paymentupdater.restclient.proxy.api.DefaultApi;
3232
import it.gov.pagopa.paymentupdater.restclient.proxy.model.PaymentRequestsGetResponse;
33-
import it.gov.pagopa.paymentupdater.util.Constants;
3433
import it.gov.pagopa.paymentupdater.util.PaymentUtil;
3534
import lombok.extern.slf4j.Slf4j;
3635

@@ -80,7 +79,7 @@ public ProxyResponse checkPayment(Payment payment)
8079
LocalDate dueDate = PaymentUtil.getLocalDateFromString(resp.getDueDate());
8180
proxyResp.setDueDate(dueDate);
8281
return proxyResp;
83-
} catch (HttpClientErrorException errorException) {
82+
} catch (HttpStatusCodeException errorException) {
8483
// manage 4XX errors
8584
ProxyPaymentResponse res = mapper.readValue(errorException.getResponseBodyAsString(),
8685
ProxyPaymentResponse.class);
@@ -115,9 +114,6 @@ public ProxyResponse checkPayment(Payment payment)
115114
} else {
116115
throw errorException;
117116
}
118-
} catch (HttpServerErrorException errorException) {
119-
// manage 5XX errors
120-
throw errorException;
121117
}
122118
}
123119

src/test/java/it/gov/pagopa/paymentupdater/AbstractMock.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,7 @@ public void mockGetPaymentInfoError() throws JsonProcessingException {
149149

150150
protected Payment selectReminderMockObject(String type, String id, String contentType, String fiscalCode,
151151
int numReminder, String rptId, String paymentDataNoticeNumber, String paymentDataFiscalCode) {
152-
Payment returnReminder1 = null;
153-
returnReminder1 = new Payment();
152+
Payment returnReminder1 = new Payment();
154153
returnReminder1.setId(id);
155154
returnReminder1.setContent_type(MessageContentType.valueOf(contentType));
156155
returnReminder1.setFiscalCode(fiscalCode);

0 commit comments

Comments
 (0)