refactor: openapi-diffusion and tests

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoArrondissementEndpoints.java

@@ -73,9 +73,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogarrprec(String code,
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogarrproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoArrondissementMunipalEndpoints.java

@@ -63,9 +63,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogarrmuprec(String code
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogarrmuproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoCantonEndpoints.java

@@ -69,9 +69,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogcanprec(String code,
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogcanproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoCantonOuVilleEndpoints.java

@@ -75,9 +75,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogcanvilprec(String cod
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogcanvilproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoCommuneEndpoints.java

@@ -80,9 +80,9 @@ public ResponseEntity<List<TerritoireBase>> getcogcomprec( String code, LocalDat
 
     @Override
     public ResponseEntity<List<TerritoireBase>> getcogcomproj( String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well//
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoDepartementEndpoints.java

@@ -50,9 +50,9 @@ public ResponseEntity<List<TerritoireBaseChefLieu>>  getcogdepprec(String code,
 
     @Override
     public ResponseEntity<List<TerritoireBaseChefLieu>>  getcogdepproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoIntercommunaliteEndpoints.java

@@ -66,9 +66,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogintercoprec(String co
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogintercoproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/main/java/fr/insee/rmes/magma/diffusion/api/GeoRegionEndpoints.java

@@ -63,9 +63,9 @@ public ResponseEntity<List<TerritoireTousAttributs>> getcogregprec(String code,
 
     @Override
     public ResponseEntity<List<TerritoireTousAttributs>> getcogregproj(String code, LocalDate dateProjection, LocalDate date) {
-        //le booléen previous est calculé en fonction du paramètre dateProjection (paramètre obligatoire) et du paramètre date valorisé à la date du jour si absent
-        // (facultatif). La valorisation de date à la date du jour dans ParameterValueDecoder n'est pas conservée en dehors de la méthode
-        // => obligé de valoriser date ici aussi
+        //The Boolean previous is based on the dateProjection parameter (required parameter) and on the date parameter set to today's date if absent
+        // (optional). Setting the date to today's date in ParameterValueDecoder is not retained outside the method
+        // => must set the date here as well
         if (date == null) {
             date = LocalDate.now();
         }

magma-diffusion-impl/src/test/java/fr/insee/rmes/magma/diffusion/api/testcontainers/queries/ConceptsQueriesTest.java

@@ -12,8 +12,7 @@
 import java.net.URI;
 import java.time.LocalDate;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertAll;
+import static org.junit.jupiter.api.Assertions.*;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
 
@@ -38,11 +37,9 @@ public class ConceptsQueriesTest extends TestcontainerTest {
     void should_return_conceptc2066_when_ConceptsDefinitionCode_codec2066(){
         var response  = endpoints.getconcept("c2066");
         var result = response.getBody();
+        assertNotNull(result);
         assertAll(
-                () -> {
-                    Assertions.assertNotNull(result);
-                    assertEquals("c2066", result.getId());
-                },
+                () -> assertEquals("c2066", result.getId()),
                 () -> assertEquals(URI.create("http://id.insee.fr/concepts/definition/c2066"), result.getUri()),
                 () -> assertEquals("Auto-entrepreneur", result.getIntitule().getFirst().getContenu()),
                 () -> assertEquals("fr", result.getIntitule().getFirst().getLangue()),
@@ -67,11 +64,9 @@ void should_return_conceptc2066_when_ConceptsDefinitionCode_codec2066(){
     void should_return_conceptc1201_when_ConceptsDefinitionCode_codec1201(){
         var response  = endpoints.getconcept("c1201");
         var result = response.getBody();
+        Assertions.assertNotNull(result);
         assertAll(
-                () -> {
-                    Assertions.assertNotNull(result);
-                    assertEquals("c1201", result.getId());
-                },
+                () -> assertEquals("c1201", result.getId()),
                 () -> assertEquals(URI.create("http://id.insee.fr/concepts/definition/c1201"), result.getUri()),
                 () -> assertEquals("Accueils collectifs de mineurs à caractère éducatif", result.getIntitule().getFirst().getContenu()),
                 () -> assertEquals("fr", result.getIntitule().getFirst().getLangue()),
@@ -85,7 +80,7 @@ void should_return_conceptc1201_when_ConceptsDefinitionCode_codec1201(){
                 () -> assertEquals("fr", result.getNoteEditoriale().getFirst().getLangue()),
                 () -> assertEquals("<div xmlns=\"http://www.w3.org/1999/xhtml\"><p>Source : mission Enquêtes, Données et Études Statistiques (MEDES), service statistique ministériel en charge de la jeunesse et des sports (site : https://injep.fr/mesurer/) </p></div>", result.getNoteEditoriale().getLast().getContenu()),
                 () -> assertEquals("en", result.getNoteEditoriale().getLast().getLangue()),
-                () -> assertEquals(null, result.getDateMiseAJour()),
+                () -> assertNull(result.getDateMiseAJour()),
                 () -> assertEquals(3, result.getConceptsPrecedents().size()),
                 () -> assertEquals("c1307", result.getConceptsPrecedents().getFirst().getId()),
                 () -> assertEquals(URI.create("http://id.insee.fr/concepts/definition/c1307"), result.getConceptsPrecedents().getFirst().getUri()),
@@ -110,7 +105,7 @@ void should_return_404_when_ConceptsDefinitionCode_codec1000() throws Exception{
 
 //    geo/concepts/definitions?libelle=elect
     @Test
-    void should_return_16_concepts_when_ConceptsDefinitions_libelleÉlect() {
+    void should_return_16_concepts_when_ConceptsDefinitions_libelleElect() {
         var response  = endpoints.getconceptsliste("Élect");
         var result = response.getBody();
         Assertions.assertNotNull(result);

magma-diffusion-impl/src/test/java/fr/insee/rmes/magma/diffusion/api/testcontainers/queries/GeoArrondissementMunicipalQueriesTest.java

@@ -4,7 +4,6 @@
 import fr.insee.rmes.magma.diffusion.api.GeoArrondissementMunipalEndpoints;
 import fr.insee.rmes.magma.diffusion.model.ArrondissementMunicipal;
 import fr.insee.rmes.magma.diffusion.model.TerritoireTousAttributs;
-import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -15,6 +14,7 @@
 import java.time.LocalDate;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
 
@@ -39,7 +39,7 @@ public class GeoArrondissementMunicipalQueriesTest extends TestcontainerTest {
     void should_return_11_territoires_when_ArrondissementMunicipalCodeAscendants_code13202_date20250904_typeNull(){
         var response  = endpoints.getcogarrmuasc("13202", LocalDate.of(2025, 9, 4), null);
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         assertEquals(11, result.size());
 
         // Vérifie le premier élément (AireDAttractionDesVilles2020)
@@ -71,7 +71,7 @@ void should_return_11_territoires_when_ArrondissementMunicipalCodeAscendants_cod
     void should_return_ArrondissementMunicipal_When_code69385_date20250904() {
         var response = endpoints.getcogarrmu("69385", LocalDate.of(2025, 9, 4));
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         assertEquals("69385", result.getCode());
         assertEquals("http://id.insee.fr/geo/arrondissementMunicipal/cd9f4663-684c-455d-b62e-39e51c6fad99", result.getUri());
         assertEquals(ArrondissementMunicipal.TypeEnum.ARRONDISSEMENT_MUNICIPAL, result.getType());
@@ -100,7 +100,7 @@ void should_return_404_when_ArrondissementMunicipalCode_code69380_date20250904()
     void should_return_45_arrondissementsMunicipaux_when_ArrondissementsMunicipaux_date20250904(){
         var response  = endpoints.getcogarrmuliste ("2025-09-04");
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         var resultItem1= result.getFirst();
         assertEquals(45, result.size());
         assertEquals("13201", resultItem1.getCode());
@@ -126,7 +126,7 @@ void should_return_45_arrondissementsMunicipaux_when_ArrondissementsMunicipaux_d
 void should_return_45_arrondissementsMunicipaux_when_ArrondissementsMunicipaux_dateEtoile(){
     var response  = endpoints.getcogarrmuliste ("2025-09-04");
     var result = response.getBody();
-    Assertions.assertNotNull(result);
+    assertNotNull(result);
     var resultItem1= result.getFirst();
     assertEquals(45, result.size());
     assertEquals("13201", resultItem1.getCode());
@@ -156,7 +156,7 @@ void should_return_45_arrondissementsMunicipaux_when_ArrondissementsMunicipaux_d
     void should_return_1_arrondissementMunicipal_when_ArrondissementMunicipalCodePrecedents_code69385_date20250904(){
         var response  = endpoints.getcogarrmuprec ("69385", LocalDate.of(2025, 9, 4));
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         var resultItem1= result.getFirst();
         assertEquals(1, result.size());
         assertEquals("69385", resultItem1.getCode());
@@ -186,7 +186,7 @@ void should_return_404_when_ArrondissementMunicipalCodePrecedents_code69385_date
     void should_return_2_arrondissementsMunicipaux_when_ArrondissementMunicipalCodeProjetes_code69385_date19600101_dateProjection20111231(){
         var response  = endpoints.getcogarrmuproj ("69385", LocalDate.of(2011,12,31), LocalDate.of(1950,1,1));
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         var resultItem1= result.getFirst();
         assertEquals(2, result.size());
         assertEquals("69385", resultItem1.getCode());
@@ -216,7 +216,7 @@ void should_return_2_arrondissementsMunicipaux_when_ArrondissementMunicipalCodeP
     void should_return_2_arrondissementsMunicipaux_when_ArrondissementMunicipalCodeSuivants_code69385_date19600101(){
         var response  = endpoints.getcogarrmusuiv ("69385", LocalDate.of(1960,1,1));
         var result = response.getBody();
-        Assertions.assertNotNull(result);
+        assertNotNull(result);
         var resultItem1= result.getFirst();
         assertEquals(2, result.size());
         assertEquals("69385", resultItem1.getCode());