P4ADEV-1774 fix sonar constructor

src/main/java/it/gov/pagopa/pu/debtpositions/service/DebtPositionServiceImpl.java

@@ -8,6 +8,7 @@
 import it.gov.pagopa.pu.debtpositions.mapper.TransferMapper;
 import it.gov.pagopa.pu.debtpositions.model.DebtPosition;
 import it.gov.pagopa.pu.debtpositions.model.PaymentOption;
+import it.gov.pagopa.pu.debtpositions.util.Repositories;
 import it.gov.pagopa.pu.debtpositions.model.Transfer;
 import it.gov.pagopa.pu.debtpositions.repository.*;
 import jakarta.transaction.Transactional;
@@ -23,11 +24,12 @@ public class DebtPositionServiceImpl implements DebtPositionService {
   private final InstallmentMapper installmentMapper;
   private final TransferMapper transferMapper;
 
-  public DebtPositionServiceImpl(DebtPositionRepository debtPositionRepository, PaymentOptionRepository paymentOptionRepository, InstallmentPIIRepository installmentRepository, TransferRepository transferRepository, DebtPositionMapper debtPositionMapper, PaymentOptionMapper paymentOptionMapper, InstallmentMapper installmentMapper, TransferMapper transferMapper) {
-    this.debtPositionRepository = debtPositionRepository;
-    this.paymentOptionRepository = paymentOptionRepository;
-    this.installmentRepository = installmentRepository;
-    this.transferRepository = transferRepository;
+  public DebtPositionServiceImpl(Repositories repositories, DebtPositionMapper debtPositionMapper, PaymentOptionMapper paymentOptionMapper,
+                                 InstallmentMapper installmentMapper, TransferMapper transferMapper) {
+    this.debtPositionRepository = repositories.debtPositionRepository();
+    this.paymentOptionRepository = repositories.paymentOptionRepository();
+    this.installmentRepository = repositories.installmentRepository();
+    this.transferRepository = repositories.transferRepository();
     this.debtPositionMapper = debtPositionMapper;
     this.paymentOptionMapper = paymentOptionMapper;
     this.installmentMapper = installmentMapper;

src/main/java/it/gov/pagopa/pu/debtpositions/util/Repositories.java

@@ -0,0 +1,12 @@
+package it.gov.pagopa.pu.debtpositions.util;
+
+import it.gov.pagopa.pu.debtpositions.repository.DebtPositionRepository;
+import it.gov.pagopa.pu.debtpositions.repository.InstallmentPIIRepository;
+import it.gov.pagopa.pu.debtpositions.repository.PaymentOptionRepository;
+import it.gov.pagopa.pu.debtpositions.repository.TransferRepository;
+
+public record Repositories(DebtPositionRepository debtPositionRepository,
+                           PaymentOptionRepository paymentOptionRepository,
+                           InstallmentPIIRepository installmentRepository,
+                           TransferRepository transferRepository) {
+}

src/test/java/it/gov/pagopa/pu/debtpositions/service/DebtPositionServiceImplTest.java

@@ -11,14 +11,15 @@
 import it.gov.pagopa.pu.debtpositions.mapper.TransferMapper;
 import it.gov.pagopa.pu.debtpositions.model.DebtPosition;
 import it.gov.pagopa.pu.debtpositions.model.PaymentOption;
+import it.gov.pagopa.pu.debtpositions.util.Repositories;
 import it.gov.pagopa.pu.debtpositions.model.Transfer;
 import it.gov.pagopa.pu.debtpositions.repository.DebtPositionRepository;
 import it.gov.pagopa.pu.debtpositions.repository.InstallmentPIIRepository;
 import it.gov.pagopa.pu.debtpositions.repository.PaymentOptionRepository;
 import it.gov.pagopa.pu.debtpositions.repository.TransferRepository;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
-import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.junit.jupiter.MockitoExtension;
@@ -54,9 +55,20 @@ class DebtPositionServiceImplTest {
   @Mock
   private TransferMapper transferMapper;
 
-  @InjectMocks
   private DebtPositionServiceImpl debtPositionService;
 
+  @BeforeEach
+  void setUp() {
+    Repositories repositories = new Repositories(debtPositionRepository, paymentOptionRepository, installmentRepository, transferRepository);
+    debtPositionService = new DebtPositionServiceImpl(
+      repositories,
+      debtPositionMapper,
+      paymentOptionMapper,
+      installmentMapper,
+      transferMapper
+    );
+  }
+
   @Test
   void givenValidDebtPositionDTO_WhenSaveDebtPosition_ThenSaveAllEntities() {
     DebtPositionDTO inputDto = new DebtPositionDTO();
@@ -103,7 +115,6 @@ void givenValidDebtPositionDTO_WhenSaveDebtPosition_ThenSaveAllEntities() {
     Mockito.verify(transferMapper).mapToModel(transferDTO);
   }
 
-
   @Test
   void givenRepositoryFails_WhenSaveDebtPosition_ThenThrowRuntimeException() {
     DebtPositionDTO inputDto = new DebtPositionDTO();