From affb161ad81bfdd387896d93a380420a0058bf51 Mon Sep 17 00:00:00 2001 From: pierre-maraval Date: Tue, 30 Apr 2024 15:30:50 +0200 Subject: [PATCH] Refactor : fix TU --- .../LignesFichierProcessor.java | 1 - .../service/impl/DemandeExempService.java | 7 - .../service/impl/DemandeRecouvService.java | 35 +--- .../core/service/TestDemandeExempService.java | 184 ------------------ ...ervice.java => TraitementServiceTest.java} | 2 +- .../service/impl/DemandeExempServiceTest.java | 149 +++++++++++++- .../impl/DemandeRecouvServiceTest.java | 6 +- .../item/web/impl/DemandeRestServiceTest.java | 2 - 8 files changed, 149 insertions(+), 237 deletions(-) delete mode 100644 core/src/test/java/fr/abes/item/core/service/TestDemandeExempService.java rename core/src/test/java/fr/abes/item/core/service/{TestTraitementService.java => TraitementServiceTest.java} (99%) diff --git a/batch/src/main/java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java b/batch/src/main/java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java index e03a3fd7..e9c4526b 100644 --- a/batch/src/main/java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java +++ b/batch/src/main/java/fr/abes/item/batch/traitement/traiterlignesfichierchunk/LignesFichierProcessor.java @@ -114,7 +114,6 @@ private LigneFichierDtoModif processDemandeModif(LigneFichierDto ligneFichierDto * * @param ligneFichierDto ligne du fichier sur laquelle lancer le traitement d'exemplarisation * @return la DTO de la ligne fichier modifiée en fonction du résultat du traitement - * @throws QueryToSudocException 0 ou plus de 1 résultat à la requête che * @throws CBSException : erreur CBS * @throws ZoneException : erreur de construction de la notice * @throws IOException : erreur de communication avec le CBS diff --git a/core/src/main/java/fr/abes/item/core/service/impl/DemandeExempService.java b/core/src/main/java/fr/abes/item/core/service/impl/DemandeExempService.java index 33a4ae10..907b198f 100644 --- a/core/src/main/java/fr/abes/item/core/service/impl/DemandeExempService.java +++ b/core/src/main/java/fr/abes/item/core/service/impl/DemandeExempService.java @@ -45,20 +45,13 @@ public class DemandeExempService extends DemandeService implements IDemandeServi private FichierEnrichiExemp fichierEnrichiExemp; private final IDemandeExempDao demandeExempDao; - private final FileSystemStorageService storageService; - private final ILigneFichierService ligneFichierService; - private final ReferenceService referenceService; - private final JournalService journalService; - private final TraitementService traitementService; private final UtilisateurService utilisateurService; - private final IZonesAutoriseesDao zonesAutoriseesDao; - private final ILigneFichierExempDao ligneFichierExempDao; @Value("${files.upload.path}") diff --git a/core/src/main/java/fr/abes/item/core/service/impl/DemandeRecouvService.java b/core/src/main/java/fr/abes/item/core/service/impl/DemandeRecouvService.java index df12a48f..66208be3 100644 --- a/core/src/main/java/fr/abes/item/core/service/impl/DemandeRecouvService.java +++ b/core/src/main/java/fr/abes/item/core/service/impl/DemandeRecouvService.java @@ -1,7 +1,5 @@ package fr.abes.item.core.service.impl; -import fr.abes.cbs.exception.CBSException; -import fr.abes.cbs.exception.ZoneException; import fr.abes.item.core.components.Fichier; import fr.abes.item.core.components.FichierEnrichiRecouv; import fr.abes.item.core.configuration.factory.FichierFactory; @@ -161,7 +159,7 @@ public Demande closeDemande(Demande demande) throws DemandeCheckingException { } @Override - public String[] getNoticeExemplaireAvantApres(Demande demande, LigneFichier ligneFichier) throws CBSException, IOException, ZoneException { + public String[] getNoticeExemplaireAvantApres(Demande demande, LigneFichier ligneFichier) { return new String[]{"Simulation impossible pour le recouvrement", ""}; } @@ -288,37 +286,6 @@ public int launchQueryToSudoc(String codeIndex, String valeurs) throws IOExcepti return traitementService.getCbs().getNbNotices(); } - public String getInfoFooterFichierResultat(Demande demande) { - DemandeRecouv demandeRecouv = (DemandeRecouv) demande; - int nbRechercheTotal = ligneFichierService.getNbLigneFichierTotalByDemande(demandeRecouv); - int nbNoticesTrouvees = ligneFichierService.getNbReponseTrouveesByDemande(demandeRecouv); - int nbZeroReponse = ligneFichierService.getNbZeroReponseByDemande(demandeRecouv); - int nbUneReponse = ligneFichierService.getNbUneReponseByDemande(demandeRecouv); - int nbReponseMultiple = ligneFichierService.getNbReponseMultipleByDemande(demandeRecouv); - - return System.lineSeparator() + nbNoticesTrouvees + " notices trouvées sur " + - nbRechercheTotal + - System.lineSeparator() + - System.lineSeparator() + - "Nb de 1 réponse : " + - nbUneReponse + - " | Nb sans réponse : " + - nbZeroReponse + - " | Nb plusieurs réponses : " + - nbReponseMultiple + - System.lineSeparator() + - System.lineSeparator() + - "Taux de recouvrement : " + - (nbNoticesTrouvees / nbRechercheTotal) * 100 + - "% Taux de création d'exemplaires : " + - (nbUneReponse / nbRechercheTotal) * 100 + - "%" + - System.lineSeparator() + - System.lineSeparator() + - "Fin du recouvrement : " + - Calendar.getInstance().getTime(); - } - /** * Méthode construisant la requête che en fonction des paramètres d'une demande d'exemplarisation * @param codeIndex code de l'index de la recherche diff --git a/core/src/test/java/fr/abes/item/core/service/TestDemandeExempService.java b/core/src/test/java/fr/abes/item/core/service/TestDemandeExempService.java deleted file mode 100644 index cced5416..00000000 --- a/core/src/test/java/fr/abes/item/core/service/TestDemandeExempService.java +++ /dev/null @@ -1,184 +0,0 @@ -package fr.abes.item.core.service; - -import fr.abes.cbs.exception.CBSException; -import fr.abes.cbs.exception.ZoneException; -import fr.abes.item.core.constant.Constant; -import fr.abes.item.core.repository.baseXml.ILibProfileDao; -import fr.abes.item.core.repository.item.IDemandeExempDao; -import fr.abes.item.core.repository.item.ILigneFichierExempDao; -import fr.abes.item.core.repository.item.IZonesAutoriseesDao; -import fr.abes.item.core.service.impl.DemandeExempService; -import fr.abes.item.core.service.impl.LigneFichierExempService; -import org.junit.jupiter.api.BeforeEach; -import org.junit.jupiter.api.DisplayName; -import org.junit.jupiter.api.Test; -import org.mockito.Mockito; -import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.boot.test.context.SpringBootTest; -import org.springframework.boot.test.mock.mockito.MockBean; - -import java.text.SimpleDateFormat; -import java.util.Date; - -import static org.assertj.core.api.Assertions.assertThat; -import static org.mockito.Mockito.when; - -@DisplayName("Test couche service / demande d'exemplarisation") -@SpringBootTest(classes = DemandeExempService.class) -public class TestDemandeExempService { - @MockBean - IDemandeExempDao demandeExempDao; - @MockBean - FileSystemStorageService storageService; - @MockBean - LigneFichierExempService ligneFichierExempService; - @MockBean - ReferenceService referenceService; - @MockBean - JournalService journalService; - @MockBean - TraitementService traitementService; - @MockBean - IZonesAutoriseesDao dao; - @MockBean - ILigneFichierExempDao ligneFichierExempDao; - @MockBean - ILibProfileDao libProfileDao; - - @Autowired - private DemandeExempService demandeExempService; - - private final String RCR = "341725201"; - private final String NUMEXEMP = "01"; - - SimpleDateFormat formater; - - Date aujourdhui; - - @BeforeEach - void init() { - when(dao.getIndicateursByTypeExempAndLabelZone(Mockito.anyString())).thenReturn("##"); - formater = new SimpleDateFormat("dd-MM-yyyy HH:mm"); - aujourdhui = new Date(); - } - - @Test - void getExempWithOneValue() throws ZoneException, CBSException { - String header = "930 $c"; - String valeur = "test"; - - assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( - "e01 $bx\r" + - "930 ##$b341725201$ctest\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - } - - @Test - void getExempWithOneZoneAndTwoSousZones() throws ZoneException, CBSException { - String header = "930 $d;$c"; - String valeur = "testd;testc"; - assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( - "e01 $bx\r" + - "930 ##$b341725201$ctestc$dtestd\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - } - - @Test - void getExempWithTwoZonesAndOneSousZone() throws ZoneException, CBSException { - String header = "930 $d;915 $a"; - String valeur = "testd;testa"; - assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( - "e01 $bx\r" + - "915 ##$atesta\r" + - "930 ##$b341725201$dtestd\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - } - - @Test - void getExempWithTwoZonesAndTwoSousZone() throws ZoneException, CBSException { - String header = "930 $d;$c;915 $a;$b"; - String valeur = "testd;testc;test2;test3"; - assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( - "e01 $bx\r" + - "915 ##$atest2$btest3\r" + - "930 ##$b341725201$ctestc$dtestd\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - } - - @Test - void getExempWithThreeZonesAndTwoSousZone() throws ZoneException, CBSException { - String header = "E317 $a;E856 $u;$9;930 $j"; - String valeur = "DeGruyter LN;http://buadistant;2018-145;g;"; - String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); - assertThat(notice).contains( - "e01 $bx\r" + - "930 ##$b341725201$jg\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r" + - "E317 ##$aDeGruyter LN\r" + - "E856 ##$uhttp://buadistant$92018-145\r"); - } - - @Test - void getExempWithALotOfEverything() throws ZoneException, CBSException { - String header = "930 $d;$c;915 $a;$b"; - String valeur = "testd;testc;test2;test3"; - String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); - assertThat(notice).isEqualTo( - "e01 $bx\r" + - "915 ##$atest2$btest3\r" + - "930 ##$b341725201$ctestc$dtestd\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - } - - @Test - void getExempWithALotOfEverything2() throws ZoneException, CBSException { - String header = "E856$u;$l;$z;930$a;$j;955 41$a;$k;$4;991$a;E702$3"; - String valeur = "https://federation.unimes.fr:8443/login?url=https://rd.springer.com/journal/766;UN | DIP;[Springer Journal Archives - Licence Nationale - Accès UNîmes] (1996)-(2014);Springer Journal Archives;g;1996;2014;Springer Journal Archives;Springer-revues-UN;test"; - String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); - assertThat(notice).isEqualTo( - "e01 $bx\r" + - "930 ##$b341725201$aSpringer Journal Archives$jg\r" + - "955 ##$a1996$k2014$4Springer Journal Archives\r" + - "991 ##$aSpringer-revues-UN\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r" + - "E702 ##$3test\r" + - "E856 ##$lUN | DIP$uhttps://federation.unimes.fr:8443/login?url=https://rd.springer.com/journal/766$z[Springer Journal Archives - Licence Nationale - Accès UNîmes] (1996)-(2014)\r"); - } - - @Test - void getExempWithMaxZonesAndSouszones() throws ZoneException, CBSException { - // GIVEN - String header = "915 $a;$b;917 $a;930 $c;$d;$e;$a;$i;$v;$2;991 $a;999 $a;$b;$c;$i;$o;$s;$z;E316 $a;E317 $a;E319 $a;$b;$c;$d;$x"; - String valeur = "915a;915b;917a;930c;930d;930e;930a;930i;930v;9302;991a;999a;999b;999c;999i;999o;999s;999z;E316a;E317a;E319a;E319b;E319c;E319d;E319x"; - - // WHEN - String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); - // THEN - assertThat(notice).isEqualTo( - "e01 $bx\r" + - "915 ##$a915a$b915b\r" + - "917 ##$a917a\r" + - "930 ##$b341725201$c930c$d930d$e930e$a930a$i930i$v930v$29302\r" + - "991 ##$a991a\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r" + - "999 ##$a999a$b999b$c999c$i999i$o999o$s999s$z999z\r" + - "E316 ##$aE316a\r" + - "E317 ##$aE317a\r" + - "E319 ##$aE319a$bE319b$cE319c$dE319d$xE319x\r"); - } - - @Test - void getExempWithSomeEmptyTag() throws ZoneException, CBSException { - String header = "915 $a;930 $c;$d;$a;$j"; - String valeur = ";DROIT;SDR;M31733;a"; - - String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); - assertThat(notice).isEqualTo( - "e01 $bx\r" + - "930 ##$b341725201$cDROIT$dSDR$aM31733$ja\r" + - "991 ##$a" + Constant.TEXTE_991_CREA + " le " + formater.format(aujourdhui) + "\r"); - - } - - -} diff --git a/core/src/test/java/fr/abes/item/core/service/TestTraitementService.java b/core/src/test/java/fr/abes/item/core/service/TraitementServiceTest.java similarity index 99% rename from core/src/test/java/fr/abes/item/core/service/TestTraitementService.java rename to core/src/test/java/fr/abes/item/core/service/TraitementServiceTest.java index 51c12d16..5328be68 100644 --- a/core/src/test/java/fr/abes/item/core/service/TestTraitementService.java +++ b/core/src/test/java/fr/abes/item/core/service/TraitementServiceTest.java @@ -14,7 +14,7 @@ @DisplayName("Test Couche service / traitement") @SpringBootTest(classes = {TraitementService.class}) -public class TestTraitementService { +public class TraitementServiceTest { @Autowired TraitementService tManager; @MockBean diff --git a/core/src/test/java/fr/abes/item/core/service/impl/DemandeExempServiceTest.java b/core/src/test/java/fr/abes/item/core/service/impl/DemandeExempServiceTest.java index 659f3551..26cd1862 100644 --- a/core/src/test/java/fr/abes/item/core/service/impl/DemandeExempServiceTest.java +++ b/core/src/test/java/fr/abes/item/core/service/impl/DemandeExempServiceTest.java @@ -1,24 +1,37 @@ package fr.abes.item.core.service.impl; -import fr.abes.item.core.exception.QueryToSudocException; +import fr.abes.cbs.exception.CBSException; +import fr.abes.cbs.exception.ZoneException; +import fr.abes.item.core.constant.Constant; import fr.abes.item.core.repository.baseXml.ILibProfileDao; import fr.abes.item.core.repository.item.IDemandeExempDao; import fr.abes.item.core.repository.item.ILigneFichierExempDao; import fr.abes.item.core.repository.item.IZonesAutoriseesDao; import fr.abes.item.core.service.*; +import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.extension.ExtendWith; +import org.mockito.Mockito; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.test.context.SpringBootTest; import org.springframework.boot.test.mock.mockito.MockBean; +import org.springframework.test.context.junit.jupiter.SpringExtension; + +import java.text.SimpleDateFormat; +import java.util.Calendar; + +import static org.assertj.core.api.Assertions.assertThat; +import static org.mockito.Mockito.when; @SpringBootTest(classes = {DemandeExempService.class}) +@ExtendWith({SpringExtension.class}) class DemandeExempServiceTest { @MockBean IDemandeExempDao demandeExempDao; @MockBean FileSystemStorageService storageService; @MockBean - ILigneFichierService ligneFichierExempService; + LigneFichierExempService ligneFichierExempService; @MockBean ReferenceService referenceService; @MockBean @@ -26,17 +39,141 @@ class DemandeExempServiceTest { @MockBean TraitementService traitementService; @MockBean + UtilisateurService utilisateurService; + @MockBean IZonesAutoriseesDao zonesAutoriseesDao; @MockBean ILigneFichierExempDao ligneFichierExempDao; @MockBean ILibProfileDao libProfileDao; @Autowired - private DemandeExempService service; + DemandeExempService demandeExempService; + final String RCR = "341725201"; + final String NUMEXEMP = "01"; + String dateToInsert; + + @BeforeEach + void init() { + when(zonesAutoriseesDao.getIndicateursByTypeExempAndLabelZone(Mockito.anyString())).thenReturn("##"); + Calendar calendar = Calendar.getInstance(); + SimpleDateFormat formater = new SimpleDateFormat("dd-MM-yyyy HH:mm"); + dateToInsert = formater.format(calendar.getTime()); + } + + @Test + void getExempWithOneValue() throws ZoneException, CBSException { + String header = "930 $c"; + String valeur = "test"; + assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP).contains("e01 $bx\r" + + "930 ##$b341725201$ctest\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r")); + + } @Test - void getQueryToSudoc() throws QueryToSudocException { - String result = service.getQueryToSudoc("DAT", "Autres ressources", new String[]{"a", "Téoàrtü", "ÛioëÂÄ"}); - System.out.printf(result); + void getExempWithOneZoneAndTwoSousZones() throws ZoneException, CBSException { + String header = "930 $d;$c"; + String valeur = "testd;testc"; + assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( + "e01 $bx\r" + + "930 ##$b341725201$ctestc$dtestd\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r"); + } + + @Test + void getExempWithTwoZonesAndOneSousZone() throws ZoneException, CBSException { + String header = "930 $d;915 $a"; + String valeur = "testd;testa"; + assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( + "e01 $bx\r" + + "915 ##$atesta\r" + + "930 ##$b341725201$dtestd\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r"); + } + + @Test + void getExempWithTwoZonesAndTwoSousZone() throws ZoneException, CBSException { + String header = "930 $d;$c;915 $a;$b"; + String valeur = "testd;testc;test2;test3"; + assertThat(demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP)).contains( + "e01 $bx\r" + + "915 ##$atest2$btest3\r" + + "930 ##$b341725201$ctestc$dtestd\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r"); + } + + @Test + void getExempWithThreeZonesAndTwoSousZone() throws ZoneException, CBSException { + String header = "E317 $a;E856 $u;$9;930 $j"; + String valeur = "DeGruyter LN;http://buadistant;2018-145;g;"; + String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); + assertThat(notice).contains( + "e01 $bx\r" + + "930 ##$b341725201$jg\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r" + + "E317 ##$aDeGruyter LN\r" + + "E856 ##$uhttp://buadistant$92018-145\r"); + } + + @Test + void getExempWithALotOfEverything() throws ZoneException, CBSException { + String header = "930 $d;$c;915 $a;$b"; + String valeur = "testd;testc;test2;test3"; + String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); + assertThat(notice).isEqualTo( + "e01 $bx\r" + + "915 ##$atest2$btest3\r" + + "930 ##$b341725201$ctestc$dtestd\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r"); + } + + @Test + void getExempWithALotOfEverything2() throws ZoneException, CBSException { + String header = "E856$u;$l;$z;930$a;$j;955 41$a;$k;$4;991$a;E702$3"; + String valeur = "https://federation.unimes.fr:8443/login?url=https://rd.springer.com/journal/766;UN | DIP;[Springer Journal Archives - Licence Nationale - Accès UNîmes] (1996)-(2014);Springer Journal Archives;g;1996;2014;Springer Journal Archives;Springer-revues-UN;test"; + String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); + assertThat(notice).isEqualTo( + "e01 $bx\r" + + "930 ##$b341725201$aSpringer Journal Archives$jg\r" + + "955 ##$a1996$k2014$4Springer Journal Archives\r" + + "991 ##$aSpringer-revues-UN\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r" + + "E702 ##$3test\r" + + "E856 ##$lUN | DIP$uhttps://federation.unimes.fr:8443/login?url=https://rd.springer.com/journal/766$z[Springer Journal Archives - Licence Nationale - Accès UNîmes] (1996)-(2014)\r"); + } + + @Test + void getExempWithMaxZonesAndSouszones() throws ZoneException, CBSException { + // GIVEN + String header = "915 $a;$b;917 $a;930 $c;$d;$e;$a;$i;$v;$2;991 $a;999 $a;$b;$c;$i;$o;$s;$z;E316 $a;E317 $a;E319 $a;$b;$c;$d;$x"; + String valeur = "915a;915b;917a;930c;930d;930e;930a;930i;930v;9302;991a;999a;999b;999c;999i;999o;999s;999z;E316a;E317a;E319a;E319b;E319c;E319d;E319x"; + + // WHEN + String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); + // THEN + assertThat(notice).isEqualTo( + "e01 $bx\r" + + "915 ##$a915a$b915b\r" + + "917 ##$a917a\r" + + "930 ##$b341725201$c930c$d930d$e930e$a930a$i930i$v930v$29302\r" + + "991 ##$a991a\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r" + + "999 ##$a999a$b999b$c999c$i999i$o999o$s999s$z999z\r" + + "E316 ##$aE316a\r" + + "E317 ##$aE317a\r" + + "E319 ##$aE319a$bE319b$cE319c$dE319d$xE319x\r"); + } + + @Test + void getExempWithSomeEmptyTag() throws ZoneException, CBSException { + String header = "915 $a;930 $c;$d;$a;$j"; + String valeur = ";DROIT;SDR;M31733;a"; + + String notice = demandeExempService.creerExemplaireFromHeaderEtValeur(header, valeur, RCR, NUMEXEMP); + assertThat(notice).isEqualTo( + "e01 $bx\r" + + "930 ##$b341725201$cDROIT$dSDR$aM31733$ja\r" + + "991 ##$a" + Constant.TEXTE_991_CREA + " le " + dateToInsert + "\r"); + } } \ No newline at end of file diff --git a/core/src/test/java/fr/abes/item/core/service/impl/DemandeRecouvServiceTest.java b/core/src/test/java/fr/abes/item/core/service/impl/DemandeRecouvServiceTest.java index 16457e9c..f5b3e028 100644 --- a/core/src/test/java/fr/abes/item/core/service/impl/DemandeRecouvServiceTest.java +++ b/core/src/test/java/fr/abes/item/core/service/impl/DemandeRecouvServiceTest.java @@ -5,9 +5,9 @@ import fr.abes.item.core.repository.item.IDemandeRecouvDao; import fr.abes.item.core.repository.item.ILigneFichierRecouvDao; import fr.abes.item.core.service.FileSystemStorageService; -import fr.abes.item.core.service.ILigneFichierService; import fr.abes.item.core.service.ReferenceService; import fr.abes.item.core.service.TraitementService; +import fr.abes.item.core.service.UtilisateurService; import org.junit.jupiter.api.Test; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.test.context.SpringBootTest; @@ -28,7 +28,9 @@ class DemandeRecouvServiceTest { @MockBean TraitementService traitementService; @MockBean - ILigneFichierService ligneFichierRecouvService; + LigneFichierRecouvService ligneFichierRecouvService; + @MockBean + UtilisateurService utilisateurService; @MockBean ILibProfileDao libProfileDao; @Test diff --git a/web/src/test/java/fr/abes/item/web/impl/DemandeRestServiceTest.java b/web/src/test/java/fr/abes/item/web/impl/DemandeRestServiceTest.java index 206bf49e..de28ab5d 100644 --- a/web/src/test/java/fr/abes/item/web/impl/DemandeRestServiceTest.java +++ b/web/src/test/java/fr/abes/item/web/impl/DemandeRestServiceTest.java @@ -39,10 +39,8 @@ class DemandeRestServiceTest { @Autowired WebApplicationContext context; - @InjectMocks DemandeRestService controller; - @MockBean DemandeExempService demandeExempService; @MockBean