diff --git a/src/main/java/fr/insee/genesis/Constants.java b/src/main/java/fr/insee/genesis/Constants.java index bdcbff19..e4913b28 100644 --- a/src/main/java/fr/insee/genesis/Constants.java +++ b/src/main/java/fr/insee/genesis/Constants.java @@ -4,7 +4,7 @@ public class Constants { public static final String XSLT_STRUCTURED_VARIABLES = "xslt/structured-variables.xsl"; public static final String ROOT_GROUP_NAME = "RACINE"; - public static final String ROOT_IDENTIFIER_NAME = "IdUE"; + public static final String ROOT_IDENTIFIER_NAME = "interrogationId"; public static final String METADATA_SEPARATOR = "."; public static final String DATE_REGEX = "(^([0-9]{4})[\\-\\/]([0-9]|1[0-2]|0[1-9])[\\-\\/]([0-9]|[0-2][0-9]|3[0-1])$)|" + "(^([0-9]|[0-2][0-9]|3[0-1])[\\-\\/]([0-9]|1[0-2]|0[1-9])[\\-\\/]([0-9]{4})$)"; diff --git a/src/main/java/fr/insee/genesis/controller/adapter/LunaticJsonAdapter.java b/src/main/java/fr/insee/genesis/controller/adapter/LunaticJsonAdapter.java index a818967e..edded928 100644 --- a/src/main/java/fr/insee/genesis/controller/adapter/LunaticJsonAdapter.java +++ b/src/main/java/fr/insee/genesis/controller/adapter/LunaticJsonAdapter.java @@ -13,9 +13,9 @@ public class LunaticJsonAdapter { public SurveyUnitModel convert(LunaticJsonSurveyUnit su){ return SurveyUnitModel.builder() - .idQuest(su.getIdQuest()) - .idCampaign("") - .idUE(su.getIdUE()) + .questionnaireId(su.getQuestionnaireId()) + .campaignId("") + .interrogationId(su.getInterrogationId()) .state(DataState.COLLECTED) .mode(Mode.WEB) .recordDate(LocalDateTime.now()) diff --git a/src/main/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapter.java b/src/main/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapter.java index fd9f7019..b5a5e78a 100644 --- a/src/main/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapter.java +++ b/src/main/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapter.java @@ -23,34 +23,34 @@ public class LunaticXmlAdapter { * Convert a Lunatic XML survey unit into a genesis survey unit model * @param su Lunatic XML survey unit to convert * @param variablesMap variable definitions (used for loops) - * @param idCampaign survey id + * @param campaignId survey id * @return Genesis SurveyUnitModels for each data state */ - public static List convert(LunaticXmlSurveyUnit su, VariablesMap variablesMap, String idCampaign, Mode mode){ + public static List convert(LunaticXmlSurveyUnit su, VariablesMap variablesMap, String campaignId, Mode mode){ //Get COLLECTED Data and external variables List surveyUnitModelList = new ArrayList<>(); - SurveyUnitModel surveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, idCampaign, DataState.COLLECTED, mode); + SurveyUnitModel surveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, campaignId, DataState.COLLECTED, mode); getExternalDataFromSurveyUnit(su, surveyUnitModel, variablesMap); surveyUnitModelList.add(surveyUnitModel); //Get data from other states - SurveyUnitModel editedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, idCampaign, DataState.EDITED,mode); + SurveyUnitModel editedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, campaignId, DataState.EDITED,mode); if(editedSurveyUnitModel != null){ surveyUnitModelList.add(editedSurveyUnitModel); } - SurveyUnitModel inputedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, idCampaign, DataState.INPUTED,mode); + SurveyUnitModel inputedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, campaignId, DataState.INPUTED,mode); if(inputedSurveyUnitModel != null){ surveyUnitModelList.add(inputedSurveyUnitModel); } - SurveyUnitModel forcedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, idCampaign, DataState.FORCED,mode); + SurveyUnitModel forcedSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, campaignId, DataState.FORCED,mode); if(forcedSurveyUnitModel != null){ surveyUnitModelList.add(forcedSurveyUnitModel); } - SurveyUnitModel previousSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, idCampaign, DataState.PREVIOUS,mode); + SurveyUnitModel previousSurveyUnitModel = getStateDataFromSurveyUnit(su, variablesMap, campaignId, DataState.PREVIOUS,mode); if(previousSurveyUnitModel != null){ surveyUnitModelList.add(previousSurveyUnitModel); } @@ -63,15 +63,15 @@ public static List convert(LunaticXmlSurveyUnit su, VariablesMa * Collects data from XML survey unit depending on the data state * @param su source XML Survey Unit * @param variablesMap variable definitions (used for loops) - * @param idCampaign survey id + * @param campaignId survey id * @param dataState state of the SurveyUnitModel to generate * @return SurveyUnitModel with a specific state */ - private static SurveyUnitModel getStateDataFromSurveyUnit(LunaticXmlSurveyUnit su, VariablesMap variablesMap, String idCampaign, DataState dataState, Mode mode) { + private static SurveyUnitModel getStateDataFromSurveyUnit(LunaticXmlSurveyUnit su, VariablesMap variablesMap, String campaignId, DataState dataState, Mode mode) { SurveyUnitModel surveyUnitModel = SurveyUnitModel.builder() - .idQuest(su.getQuestionnaireModelId()) - .idCampaign(idCampaign) - .idUE(su.getId()) + .questionnaireId(su.getQuestionnaireModelId()) + .campaignId(campaignId) + .interrogationId(su.getId()) .state(dataState) .mode(mode) .recordDate(LocalDateTime.now()) @@ -115,6 +115,7 @@ private static SurveyUnitModel getCollectedDataFromSurveyUnit(LunaticXmlSurveyUn default: return null; } + if(valueTypeList == null) { continue; //Go to next data } @@ -123,10 +124,10 @@ private static SurveyUnitModel getCollectedDataFromSurveyUnit(LunaticXmlSurveyUn if (valueTypeList.get(i-1).getValue()!=null) { variableValues.add(valueTypeList.get(i-1).getValue()); variableModels.add(VariableModel.builder() - .idVar(lunaticXmlCollectedData.getVariableName()) + .varId(lunaticXmlCollectedData.getVariableName()) .values(variableValues) - .idLoop(LoopIdentifier.getLoopIdentifier(lunaticXmlCollectedData.getVariableName(), variablesMap, i)) - .idParent(LoopIdentifier.getRelatedVariableName(lunaticXmlCollectedData.getVariableName(), variablesMap)) + .loopId(LoopIdentifier.getLoopIdentifier(lunaticXmlCollectedData.getVariableName(), variablesMap, i)) + .parentId(LoopIdentifier.getRelatedVariableName(lunaticXmlCollectedData.getVariableName(), variablesMap)) .build()); dataCount++; } @@ -161,10 +162,10 @@ private static void getExternalDataFromSurveyUnit(LunaticXmlSurveyUnit su, Surve if (valueTypeList.get(i-1).getValue()!=null) { variableValues.add(valueTypeList.get(i-1).getValue()); variableModels.add(VariableModel.builder() - .idVar(lunaticXmlExternalData.getVariableName()) + .varId(lunaticXmlExternalData.getVariableName()) .values(variableValues) - .idLoop(LoopIdentifier.getLoopIdentifier(lunaticXmlExternalData.getVariableName(), variablesMap, i)) - .idParent(LoopIdentifier.getRelatedVariableName(lunaticXmlExternalData.getVariableName(), variablesMap)) + .loopId(LoopIdentifier.getLoopIdentifier(lunaticXmlExternalData.getVariableName(), variablesMap, i)) + .parentId(LoopIdentifier.getRelatedVariableName(lunaticXmlExternalData.getVariableName(), variablesMap)) .build()); } } diff --git a/src/main/java/fr/insee/genesis/controller/dto/CampaignWithQuestionnaire.java b/src/main/java/fr/insee/genesis/controller/dto/CampaignWithQuestionnaire.java index 17fbeeed..b027afa4 100644 --- a/src/main/java/fr/insee/genesis/controller/dto/CampaignWithQuestionnaire.java +++ b/src/main/java/fr/insee/genesis/controller/dto/CampaignWithQuestionnaire.java @@ -10,6 +10,6 @@ @NoArgsConstructor @Getter public class CampaignWithQuestionnaire { - private String idCampaign; + private String campaignId; private Set questionnaires; } diff --git a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitId.java b/src/main/java/fr/insee/genesis/controller/dto/InterrogationId.java similarity index 63% rename from src/main/java/fr/insee/genesis/controller/dto/SurveyUnitId.java rename to src/main/java/fr/insee/genesis/controller/dto/InterrogationId.java index 18b87005..34d80954 100644 --- a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitId.java +++ b/src/main/java/fr/insee/genesis/controller/dto/InterrogationId.java @@ -1,6 +1,5 @@ package fr.insee.genesis.controller.dto; -import com.fasterxml.jackson.annotation.JsonProperty; import lombok.AllArgsConstructor; import lombok.Getter; import lombok.NoArgsConstructor; @@ -12,21 +11,20 @@ @Getter @NoArgsConstructor @AllArgsConstructor -public class SurveyUnitId { +public class InterrogationId { - @JsonProperty("idUE") - private String idUE; + private String interrogationId; @Override public boolean equals(Object o) { if (this == o) return true; if (o == null || getClass() != o.getClass()) return false; - SurveyUnitId that = (SurveyUnitId) o; - return Objects.equals(idUE, that.idUE); + InterrogationId that = (InterrogationId) o; + return Objects.equals(interrogationId, that.interrogationId); } @Override public int hashCode() { - return Objects.hash(idUE); + return Objects.hash(interrogationId); } } diff --git a/src/main/java/fr/insee/genesis/controller/dto/QuestionnaireWithCampaign.java b/src/main/java/fr/insee/genesis/controller/dto/QuestionnaireWithCampaign.java index f6095337..41815dc8 100644 --- a/src/main/java/fr/insee/genesis/controller/dto/QuestionnaireWithCampaign.java +++ b/src/main/java/fr/insee/genesis/controller/dto/QuestionnaireWithCampaign.java @@ -10,6 +10,6 @@ @NoArgsConstructor @Getter public class QuestionnaireWithCampaign { - private String idQuestionnaire; + private String questionnaireId; private Set campaigns; } diff --git a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitDto.java b/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitDto.java index 249323bb..114670c2 100644 --- a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitDto.java +++ b/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitDto.java @@ -8,7 +8,7 @@ @Builder @Data public class SurveyUnitDto { - private String surveyUnitId; + private String interrogationId; private List collectedVariables; private List externalVariables; } diff --git a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitSimplified.java b/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitSimplified.java index 005f8f59..8ea44a8c 100644 --- a/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitSimplified.java +++ b/src/main/java/fr/insee/genesis/controller/dto/SurveyUnitSimplified.java @@ -11,9 +11,9 @@ @Data public class SurveyUnitSimplified { - private String idQuest; - private String idCampaign; - private String idUE; + private String questionnaireId; + private String campaignId; + private String interrogationId; private Mode mode; private List variablesUpdate; private List externalVariables; diff --git a/src/main/java/fr/insee/genesis/controller/rest/responses/CampaignController.java b/src/main/java/fr/insee/genesis/controller/rest/responses/CampaignController.java index 1e3135c4..549e02f7 100644 --- a/src/main/java/fr/insee/genesis/controller/rest/responses/CampaignController.java +++ b/src/main/java/fr/insee/genesis/controller/rest/responses/CampaignController.java @@ -27,7 +27,7 @@ public CampaignController(SurveyUnitApiPort surveyUnitService) { @Operation(summary = "List campaigns in database") @GetMapping(path = "/") public ResponseEntity> getCampaigns() { - Set campaigns = surveyUnitService.findDistinctIdCampaigns(); + Set campaigns = surveyUnitService.findDistinctCampaignIds(); return ResponseEntity.ok(campaigns); } diff --git a/src/main/java/fr/insee/genesis/controller/rest/responses/IdUEController.java b/src/main/java/fr/insee/genesis/controller/rest/responses/InterrogationController.java similarity index 56% rename from src/main/java/fr/insee/genesis/controller/rest/responses/IdUEController.java rename to src/main/java/fr/insee/genesis/controller/rest/responses/InterrogationController.java index b47df612..427d966a 100644 --- a/src/main/java/fr/insee/genesis/controller/rest/responses/IdUEController.java +++ b/src/main/java/fr/insee/genesis/controller/rest/responses/InterrogationController.java @@ -1,6 +1,6 @@ package fr.insee.genesis.controller.rest.responses; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.domain.ports.api.SurveyUnitApiPort; import io.swagger.v3.oas.annotations.Operation; import lombok.extern.slf4j.Slf4j; @@ -12,23 +12,23 @@ import java.util.List; -@RequestMapping(path = "/idUEs" ) +@RequestMapping(path = "/interrogations" ) @Controller @Slf4j -public class IdUEController { +public class InterrogationController { private final SurveyUnitApiPort surveyUnitService; - public IdUEController(SurveyUnitApiPort surveyUnitService) { + public InterrogationController(SurveyUnitApiPort surveyUnitService) { this.surveyUnitService = surveyUnitService; } - @Operation(summary = "Retrieve all IdUEs for a given questionnaire") + @Operation(summary = "Retrieve all interrogations for a given questionnaire") @GetMapping(path = "/by-questionnaire") - public ResponseEntity> getAllIdUEsByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) { - List responses = surveyUnitService.findDistinctIdUEsByIdQuestionnaire(idQuestionnaire); + public ResponseEntity> getAllInterrogationIdsByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) { + List responses = surveyUnitService.findDistinctInterrogationIdsByQuestionnaireId(questionnaireId); return ResponseEntity.ok(responses); } diff --git a/src/main/java/fr/insee/genesis/controller/rest/responses/ModeController.java b/src/main/java/fr/insee/genesis/controller/rest/responses/ModeController.java index 320af308..3fb31e7e 100644 --- a/src/main/java/fr/insee/genesis/controller/rest/responses/ModeController.java +++ b/src/main/java/fr/insee/genesis/controller/rest/responses/ModeController.java @@ -28,15 +28,15 @@ public ModeController(SurveyUnitApiPort surveyUnitService) { @Operation(summary = "List sources/modes used for a given questionnaire") @GetMapping(path = "/by-questionnaire") - public ResponseEntity> getModesByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) { - List modes = surveyUnitService.findModesByIdQuestionnaire(idQuestionnaire); + public ResponseEntity> getModesByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) { + List modes = surveyUnitService.findModesByQuestionnaireId(questionnaireId); return ResponseEntity.ok(modes); } @Operation(summary = "List sources/modes used for a given campaign") @GetMapping(path = "/by-campaign") - public ResponseEntity> getModesByCampaign(@RequestParam("idCampaign") String idCampaign) { - List modes = surveyUnitService.findModesByIdCampaign(idCampaign); + public ResponseEntity> getModesByCampaign(@RequestParam("campaignId") String campaignId) { + List modes = surveyUnitService.findModesByCampaignId(campaignId); return ResponseEntity.ok(modes); } diff --git a/src/main/java/fr/insee/genesis/controller/rest/responses/QuestionnaireController.java b/src/main/java/fr/insee/genesis/controller/rest/responses/QuestionnaireController.java index ad8303c4..82064575 100644 --- a/src/main/java/fr/insee/genesis/controller/rest/responses/QuestionnaireController.java +++ b/src/main/java/fr/insee/genesis/controller/rest/responses/QuestionnaireController.java @@ -30,7 +30,7 @@ public QuestionnaireController(SurveyUnitApiPort surveyUnitService) { @Operation(summary = "List questionnaires in database") @GetMapping(path = "/") public ResponseEntity> getQuestionnaires() { - Set questionnaires = surveyUnitService.findDistinctIdQuestionnaires(); + Set questionnaires = surveyUnitService.findDistinctQuestionnaireIds(); return ResponseEntity.ok(questionnaires); } @@ -45,8 +45,8 @@ public ResponseEntity> getQuestionnairesWithCamp @Operation(summary = "List questionnaires used for a given campaign") @GetMapping(path = "/by-campaign") - public ResponseEntity> getQuestionnairesByCampaign(@RequestParam("idCampaign") String idCampaign) { - Set questionnaires = surveyUnitService.findIdQuestionnairesByIdCampaign(idCampaign); + public ResponseEntity> getQuestionnairesByCampaign(@RequestParam("campaignId") String campaignId) { + Set questionnaires = surveyUnitService.findQuestionnaireIdsByCampaignId(campaignId); return ResponseEntity.ok(questionnaires); } diff --git a/src/main/java/fr/insee/genesis/controller/rest/responses/ResponseController.java b/src/main/java/fr/insee/genesis/controller/rest/responses/ResponseController.java index 393d1350..a2a662e8 100644 --- a/src/main/java/fr/insee/genesis/controller/rest/responses/ResponseController.java +++ b/src/main/java/fr/insee/genesis/controller/rest/responses/ResponseController.java @@ -8,7 +8,7 @@ import fr.insee.genesis.Constants; import fr.insee.genesis.controller.adapter.LunaticXmlAdapter; import fr.insee.genesis.controller.dto.SurveyUnitDto; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.controller.dto.SurveyUnitInputDto; import fr.insee.genesis.controller.dto.SurveyUnitSimplified; import fr.insee.genesis.controller.sources.xml.LunaticXmlCampaign; @@ -55,12 +55,11 @@ import java.time.LocalDateTime; import java.util.ArrayList; import java.util.List; -import java.util.stream.Collectors; import java.util.stream.Stream; @RequestMapping(path = "/responses" ) @Controller -@Tag(name = "Response services for interrogations", description = "A **response** is considered the entire set of data associated with an interrogation (idUE x idQuestionnaire). \n\n These data may have different state (collected, edited, external, ...) ") +@Tag(name = "Response services", description = "A **response** is considered the entire set of data associated with an interrogation (survey unit x questionnaireId). \n\n These data may have different state (collected, edited, external, ...) ") @Slf4j public class ResponseController { @@ -257,67 +256,67 @@ public ResponseEntity saveResponsesFromAllCampaignFolders(){ //DELETE @Operation(summary = "Delete all responses associated with a questionnaire") @DeleteMapping(path = "/delete/by-questionnaire") - public ResponseEntity deleteAllResponsesByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) { - log.info("Try to delete all responses of questionnaire : {}", idQuestionnaire); - Long ndDocuments = surveyUnitService.deleteByIdQuestionnaire(idQuestionnaire); + public ResponseEntity deleteAllResponsesByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) { + log.info("Try to delete all responses of questionnaire : {}", questionnaireId); + Long ndDocuments = surveyUnitService.deleteByQuestionnaireId(questionnaireId); log.info("{} responses deleted", ndDocuments); return ResponseEntity.ok(String.format("%d responses deleted", ndDocuments)); } //GET - @Operation(summary = "Retrieve responses for an interrogation, using IdUE and IdQuestionnaire from Genesis Database") + @Operation(summary = "Retrieve responses for an interrogation, using interrogationId and questionnaireId from Genesis Database") @GetMapping(path = "/by-ue-and-questionnaire") - public ResponseEntity> findResponsesByUEAndQuestionnaire(@RequestParam("idUE") String idUE, - @RequestParam("idQuestionnaire") String idQuestionnaire) { - List responses = surveyUnitService.findByIdsUEAndQuestionnaire(idUE, idQuestionnaire); + public ResponseEntity> findResponsesByInterrogationAndQuestionnaire(@RequestParam("interrogationId") String interrogationId, + @RequestParam("questionnaireId") String questionnaireId) { + List responses = surveyUnitService.findByIdsInterrogationAndQuestionnaire(interrogationId, questionnaireId); return ResponseEntity.ok(responses); } - @Operation(summary = "Retrieve responses for an interrogation, using IdUE and IdQuestionnaire from Genesis Database with the latest value for each available state of every variable") + @Operation(summary = "Retrieve responses for an interrogation, using interrogationId and questionnaireId from Genesis Database with the latest value for each available state of every variable") @GetMapping(path = "/by-ue-and-questionnaire/latest-states") - public ResponseEntity findResponsesByUEAndQuestionnaireLatestStates( - @RequestParam("idUE") String idUE, - @RequestParam("idQuestionnaire") String idQuestionnaire) { - SurveyUnitDto response = surveyUnitService.findLatestValuesByStateByIdAndByIdQuestionnaire(idUE, idQuestionnaire); + public ResponseEntity findResponsesByInterrogationAndQuestionnaireLatestStates( + @RequestParam("interrogationId") String interrogationId, + @RequestParam("questionnaireId") String questionnaireId) { + SurveyUnitDto response = surveyUnitService.findLatestValuesByStateByIdAndByQuestionnaireId(interrogationId, questionnaireId); return ResponseEntity.ok(response); } @Operation(summary = "Retrieve all responses (for all interrogations) of one questionnaire") @GetMapping(path = "/by-questionnaire") - public ResponseEntity findAllResponsesByQuestionnaire(@RequestParam("idQuestionnaire") String idQuestionnaire) { - log.info("Try to find all responses of questionnaire : {}", idQuestionnaire); + public ResponseEntity findAllResponsesByQuestionnaire(@RequestParam("questionnaireId") String questionnaireId) { + log.info("Try to find all responses of questionnaire : {}", questionnaireId); - //Get all IdUEs/modes of the survey - List idUEsResponses = surveyUnitService.findIdUEsAndModesByIdQuestionnaire(idQuestionnaire); - log.info("Responses found : {}", idUEsResponses.size()); + //Get all interrogationIds/modes of the survey + List interrogationIdsResponses = surveyUnitService.findInterrogationIdsAndModesByQuestionnaireId(questionnaireId); + log.info("Responses found : {}", interrogationIdsResponses.size()); - String filepathString = String.format("OUT/%s/OUT_ALL_%s.json", idQuestionnaire, LocalDateTime.now().toString().replace(":", "")); + String filepathString = String.format("OUT/%s/OUT_ALL_%s.json", questionnaireId, LocalDateTime.now().toString().replace(":", "")); Path filepath = Path.of(fileUtils.getDataFolderSource(), filepathString); - try (Stream responsesStream = surveyUnitService.findByIdQuestionnaire(idQuestionnaire)) { + try (Stream responsesStream = surveyUnitService.findByQuestionnaireId(questionnaireId)) { fileUtils.writeSuUpdatesInFile(filepath, responsesStream); } catch (IOException e) { log.error("Error while writing file", e); return ResponseEntity.internalServerError().body(filepath); } - log.info("End of extraction, responses extracted: {}", idUEsResponses.size()); + log.info("End of extraction, responses extracted: {}", interrogationIdsResponses.size()); return ResponseEntity.ok(filepath); } - @Operation(summary = "Retrieve responses for an interrogation, using IdUE and IdQuestionnaire from Genesis Database. It returns only the latest value of each variable regardless of the state.") + @Operation(summary = "Retrieve responses for an interrogation, using interrogationId and questionnaireId from Genesis Database. It returns only the latest value of each variable regardless of the state.") @GetMapping(path = "/by-ue-and-questionnaire/latest") - public ResponseEntity> getLatestByUE(@RequestParam("idUE") String idUE, - @RequestParam("idQuestionnaire") String idQuestionnaire) { - List responses = surveyUnitService.findLatestByIdAndByIdQuestionnaire(idUE, idQuestionnaire); + public ResponseEntity> getLatestByInterrogation(@RequestParam("interrogationId") String interrogationId, + @RequestParam("questionnaireId") String questionnaireId) { + List responses = surveyUnitService.findLatestByIdAndByQuestionnaireId(interrogationId, questionnaireId); return ResponseEntity.ok(responses); } - @Operation(summary = "Retrieve responses for an interrogation, using IdUE and IdQuestionnaire from Genesis Database. For a given mode, it returns only the latest value of each variable regardless of the state. The result is one object by mode in the output") + @Operation(summary = "Retrieve responses for an interrogation, using interrogationId and questionnaireId from Genesis Database. For a given mode, it returns only the latest value of each variable regardless of the state. The result is one object by mode in the output") @GetMapping(path = "/simplified/by-ue-questionnaire-and-mode/latest") - public ResponseEntity getLatestByUEOneObject(@RequestParam("idUE") String idUE, - @RequestParam("idQuestionnaire") String idQuestionnaire, + public ResponseEntity getLatestByInterrogationOneObject(@RequestParam("interrogationId") String interrogationId, + @RequestParam("questionnaireId") String questionnaireId, @RequestParam("mode") Mode mode) { - List responses = surveyUnitService.findLatestByIdAndByIdQuestionnaire(idUE, idQuestionnaire); + List responses = surveyUnitService.findLatestByIdAndByQuestionnaireId(interrogationId, questionnaireId); List outputVariables = new ArrayList<>(); List outputExternalVariables = new ArrayList<>(); responses.stream().filter(rep -> rep.getMode().equals(mode)).forEach(response -> { @@ -325,9 +324,9 @@ public ResponseEntity getLatestByUEOneObject(@RequestParam outputExternalVariables.addAll(response.getExternalVariables()); }); return ResponseEntity.ok(SurveyUnitSimplified.builder() - .idQuest(responses.getFirst().getIdQuest()) - .idCampaign(responses.getFirst().getIdCampaign()) - .idUE(responses.getFirst().getIdUE()) + .questionnaireId(responses.getFirst().getQuestionnaireId()) + .campaignId(responses.getFirst().getCampaignId()) + .interrogationId(responses.getFirst().getInterrogationId()) .variablesUpdate(outputVariables) .externalVariables(outputExternalVariables) .build()); @@ -337,13 +336,13 @@ public ResponseEntity getLatestByUEOneObject(@RequestParam @Operation(summary = "Retrieve all responses for a questionnaire and a list of UE", description = "Return the latest state for each variable for the given ids and a given questionnaire.
" + "For a given id, the endpoint returns a document by collection mode (if there is more than one).") - @PostMapping(path = "/simplified/by-list-ue-and-questionnaire/latest") - public ResponseEntity> getLatestForUEList(@RequestParam("idQuestionnaire") String idQuestionnaire, - @RequestBody List idUEs) { + @PostMapping(path = "/simplified/by-list-interrogation-and-questionnaire/latest") + public ResponseEntity> getLatestForInterrogationList(@RequestParam("questionnaireId") String questionnaireId, + @RequestBody List interrogationIds) { List results = new ArrayList<>(); - List modes = surveyUnitService.findModesByIdQuestionnaire(idQuestionnaire); - idUEs.forEach(idUE -> { - List responses = surveyUnitService.findLatestByIdAndByIdQuestionnaire(idUE.getIdUE(), idQuestionnaire); + List modes = surveyUnitService.findModesByQuestionnaireId(questionnaireId); + interrogationIds.forEach(interrogationId -> { + List responses = surveyUnitService.findLatestByIdAndByQuestionnaireId(interrogationId.getInterrogationId(), questionnaireId); modes.forEach(mode -> { List outputVariables = new ArrayList<>(); List outputExternalVariables = new ArrayList<>(); @@ -353,9 +352,9 @@ public ResponseEntity> getLatestForUEList(@RequestPar }); if (!outputVariables.isEmpty() || !outputExternalVariables.isEmpty()) { results.add(SurveyUnitSimplified.builder() - .idQuest(responses.getFirst().getIdQuest()) - .idCampaign(responses.getFirst().getIdCampaign()) - .idUE(responses.getFirst().getIdUE()) + .questionnaireId(responses.getFirst().getQuestionnaireId()) + .campaignId(responses.getFirst().getCampaignId()) + .interrogationId(responses.getFirst().getInterrogationId()) .mode(mode) .variablesUpdate(outputVariables) .externalVariables(outputExternalVariables) @@ -583,17 +582,17 @@ private ResponseEntity processXmlFileWithMemory(Path filepath, Mode mode return ResponseEntity.status(e.getStatus()).body(e.getMessage()); } - List suDtos = new ArrayList<>(); + List surveyUnitModels = new ArrayList<>(); for (LunaticXmlSurveyUnit su : campaign.getSurveyUnits()) { - suDtos.addAll(LunaticXmlAdapter.convert(su, variablesMap, campaign.getIdCampaign(), modeSpecified)); + surveyUnitModels.addAll(LunaticXmlAdapter.convert(su, variablesMap, campaign.getCampaignId(), modeSpecified)); } - surveyUnitQualityService.verifySurveyUnits(suDtos, variablesMap); + surveyUnitQualityService.verifySurveyUnits(surveyUnitModels, variablesMap); - log.debug("Saving {} survey units updates", suDtos.size()); - surveyUnitService.saveSurveyUnits(suDtos); + log.debug("Saving {} survey units updates", surveyUnitModels.size()); + surveyUnitService.saveSurveyUnits(surveyUnitModels); log.debug("Survey units updates saved"); - log.info("File {} processed with {} survey units", filepath.getFileName(), suDtos.size()); + log.info("File {} processed with {} survey units", filepath.getFileName(), surveyUnitModels.size()); return ResponseEntity.ok().build(); } @@ -613,6 +612,7 @@ private ResponseEntity processRawXmlFile(Path filepath, Mode modeSpecifi lunaticXmlRawDataApiPort.saveData(campaign, modeSpecified); log.debug(SUCCESS_MESSAGE); + log.info("File {} processed" , filepath.getFileName()); return ResponseEntity.ok().build(); } @@ -629,10 +629,10 @@ private ResponseEntity processXmlFileSequentially(Path filepath, Mode mo LunaticXmlSurveyUnit su = parser.readNextSurveyUnit(); while (su != null) { - List suDtos = new ArrayList<>(LunaticXmlAdapter.convert(su, variablesMap, campaign.getIdCampaign(), modeSpecified)); + List surveyUnitModels = new ArrayList<>(LunaticXmlAdapter.convert(su, variablesMap, campaign.getCampaignId(), modeSpecified)); - surveyUnitQualityService.verifySurveyUnits(suDtos, variablesMap); - surveyUnitService.saveSurveyUnits(suDtos); + surveyUnitQualityService.verifySurveyUnits(surveyUnitModels, variablesMap); + surveyUnitService.saveSurveyUnits(surveyUnitModels); suCount++; su = parser.readNextSurveyUnit(); diff --git a/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonDataFile.java b/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonDataFile.java index 9ffc07c8..7ca26137 100644 --- a/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonDataFile.java +++ b/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonDataFile.java @@ -9,7 +9,7 @@ public class LunaticJsonDataFile { @JsonProperty("Id") - String idQuest; + String questionnaireId; @JsonProperty("Label") String labelQuest; diff --git a/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonSurveyUnit.java b/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonSurveyUnit.java index d9d7da98..a1d595b7 100644 --- a/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonSurveyUnit.java +++ b/src/main/java/fr/insee/genesis/controller/sources/json/LunaticJsonSurveyUnit.java @@ -7,10 +7,10 @@ public class LunaticJsonSurveyUnit { @JsonProperty("Id") - private String idUE; + private String interrogationId; @JsonProperty("QuestionnaireModelId") - private String idQuest; + private String questionnaireId; @JsonProperty("Data") private LunaticJsonDataResponse dataResponse; diff --git a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlCampaign.java b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlCampaign.java index a9803eec..e54a0ebd 100644 --- a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlCampaign.java +++ b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlCampaign.java @@ -9,7 +9,7 @@ @Setter public class LunaticXmlCampaign { - private String idCampaign; + private String campaignId; private String label; private List surveyUnits; diff --git a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParser.java b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParser.java index a76e9b0d..1f303467 100644 --- a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParser.java +++ b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParser.java @@ -56,7 +56,7 @@ public LunaticXmlCampaign parseDataFile(Path filePath) throws GenesisException, log.debug("Begin to parse {} ", filePath); LunaticXmlCampaign campaign = new LunaticXmlCampaign(); Element root = document.getDocumentElement(); - campaign.setIdCampaign(root.getElementsByTagName("Id").item(0).getFirstChild().getNodeValue()); + campaign.setCampaignId(root.getElementsByTagName("Id").item(0).getFirstChild().getNodeValue()); if(root.getElementsByTagName("Label").item(0).getFirstChild() != null) campaign.setLabel(root.getElementsByTagName("Label").item(0).getFirstChild().getNodeValue()); else campaign.setLabel(""); diff --git a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParser.java b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParser.java index ee39e086..96413312 100644 --- a/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParser.java +++ b/src/main/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParser.java @@ -57,7 +57,7 @@ public LunaticXmlCampaign getCampaign() throws XMLStreamException { switch (elementName){ case Constants.CAMPAIGN_ID_ELEMENT_NAME: - campaign.setIdCampaign(reader.getElementText()); + campaign.setCampaignId(reader.getElementText()); break; case Constants.CAMPAIGN_LABEL_ELEMENT_NAME: campaign.setLabel(reader.getElementText()); diff --git a/src/main/java/fr/insee/genesis/domain/model/surveyunit/IdLoopTuple.java b/src/main/java/fr/insee/genesis/domain/model/surveyunit/IdLoopTuple.java index 6d1ac551..c004dc98 100644 --- a/src/main/java/fr/insee/genesis/domain/model/surveyunit/IdLoopTuple.java +++ b/src/main/java/fr/insee/genesis/domain/model/surveyunit/IdLoopTuple.java @@ -1,4 +1,4 @@ package fr.insee.genesis.domain.model.surveyunit; -public record IdLoopTuple(String idVar, String idLoop) { +public record IdLoopTuple(String varId, String loopId) { } diff --git a/src/main/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModel.java b/src/main/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModel.java index 13d1b389..7f019fd1 100644 --- a/src/main/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModel.java +++ b/src/main/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModel.java @@ -16,9 +16,9 @@ @AllArgsConstructor public class SurveyUnitModel { - private String idQuest; - private String idCampaign; - private String idUE; + private String questionnaireId; + private String campaignId; + private String interrogationId; private DataState state; private Mode mode; @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd'T'hh:mm") @@ -31,8 +31,8 @@ public class SurveyUnitModel { private String modifiedBy; - public SurveyUnitModel(String idUE, Mode mode) { - this.idUE = idUE; + public SurveyUnitModel(String interrogationId, Mode mode) { + this.interrogationId = interrogationId; this.mode = mode; } @@ -45,11 +45,11 @@ public boolean equals(Object o) { return false; } SurveyUnitModel that = (SurveyUnitModel) o; - return Objects.equals(idUE, that.idUE) && Objects.equals(mode, that.mode); + return Objects.equals(interrogationId, that.interrogationId) && Objects.equals(mode, that.mode); } @Override public int hashCode() { - return Objects.hash(idUE) + Objects.hash(mode); + return Objects.hash(interrogationId) + Objects.hash(mode); } } diff --git a/src/main/java/fr/insee/genesis/domain/model/surveyunit/VariableModel.java b/src/main/java/fr/insee/genesis/domain/model/surveyunit/VariableModel.java index c78e7e75..0814fc7f 100644 --- a/src/main/java/fr/insee/genesis/domain/model/surveyunit/VariableModel.java +++ b/src/main/java/fr/insee/genesis/domain/model/surveyunit/VariableModel.java @@ -6,8 +6,8 @@ @Builder public record VariableModel( - String idVar, + String varId, List values, - String idLoop, - String idParent + String loopId, + String parentId ) {} diff --git a/src/main/java/fr/insee/genesis/domain/ports/api/SurveyUnitApiPort.java b/src/main/java/fr/insee/genesis/domain/ports/api/SurveyUnitApiPort.java index f7a4d79e..63a8dab0 100644 --- a/src/main/java/fr/insee/genesis/domain/ports/api/SurveyUnitApiPort.java +++ b/src/main/java/fr/insee/genesis/domain/ports/api/SurveyUnitApiPort.java @@ -7,7 +7,7 @@ import fr.insee.genesis.domain.model.surveyunit.Mode; import fr.insee.genesis.controller.dto.QuestionnaireWithCampaign; import fr.insee.genesis.domain.model.surveyunit.SurveyUnitModel; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.exceptions.GenesisException; import java.util.List; @@ -19,34 +19,34 @@ public interface SurveyUnitApiPort { void saveSurveyUnits(List suList); - List findByIdsUEAndQuestionnaire(String idUE, String idQuest); + List findByIdsInterrogationAndQuestionnaire(String interrogationId, String questionnaireId); - List findByIdUE(String idUE); + List findByInterrogationId(String interrogationId); - Stream findByIdQuestionnaire(String idQuestionnaire); + Stream findByQuestionnaireId(String questionnaireId); - List findLatestByIdAndByIdQuestionnaire(String idUE, String idQuest); - SurveyUnitDto findLatestValuesByStateByIdAndByIdQuestionnaire(String idUE, String idQuest); + List findLatestByIdAndByQuestionnaireId(String interrogationId, String questionnaireId); + SurveyUnitDto findLatestValuesByStateByIdAndByQuestionnaireId(String interrogationId, String questionnaireId); - List findIdUEsAndModesByIdQuestionnaire(String idQuestionnaire); + List findInterrogationIdsAndModesByQuestionnaireId(String questionnaireId); - List findDistinctIdUEsByIdQuestionnaire(String idQuestionnaire); + List findDistinctInterrogationIdsByQuestionnaireId(String questionnaireId); - List findModesByIdQuestionnaire(String idQuestionnaire); + List findModesByQuestionnaireId(String questionnaireId); - List findModesByIdCampaign(String idCampaign); + List findModesByCampaignId(String campaignId); - Long deleteByIdQuestionnaire(String idQuestionnaire); + Long deleteByQuestionnaireId(String questionnaireId); long countResponses(); - Set findIdQuestionnairesByIdCampaign(String idCampaign); + Set findQuestionnaireIdsByCampaignId(String campaignId); - Set findDistinctIdCampaigns(); + Set findDistinctCampaignIds(); - long countResponsesByIdCampaign(String idCampaign); + long countResponsesByCampaignId(String campaignId); - Set findDistinctIdQuestionnaires(); + Set findDistinctQuestionnaireIds(); List findCampaignsWithQuestionnaires(); diff --git a/src/main/java/fr/insee/genesis/domain/ports/spi/SurveyUnitPersistencePort.java b/src/main/java/fr/insee/genesis/domain/ports/spi/SurveyUnitPersistencePort.java index af4aa319..093a1867 100644 --- a/src/main/java/fr/insee/genesis/domain/ports/spi/SurveyUnitPersistencePort.java +++ b/src/main/java/fr/insee/genesis/domain/ports/spi/SurveyUnitPersistencePort.java @@ -10,29 +10,29 @@ public interface SurveyUnitPersistencePort { void saveAll(List suList); - List findByIds(String idUE, String idQuest); + List findByIds(String interrogationId, String questionnaireId); - List findByIdUE(String idUE); + List findByInterrogationId(String interrogationId); - List findByIdUEsAndIdQuestionnaire(List idUEs, String idQuestionnaire); + List findByInterrogationIdsAndQuestionnaireId(List interrogationIds, String questionnaireId); - Stream findByIdQuestionnaire(String idQuestionnaire); + Stream findByQuestionnaireId(String questionnaireId); - List findIdUEsByIdQuestionnaire(String idQuestionnaire); + List findInterrogationIdsByQuestionnaireId(String questionnaireId); - List findIdUEsByIdCampaign(String idCampaign); + List findInterrogationIdsByCampaignId(String campaignId); - Long deleteByIdQuestionnaire(String idQuestionnaire); + Long deleteByQuestionnaireId(String questionnaireId); long count(); - Set findIdQuestionnairesByIdCampaign(String idCampaign); + Set findQuestionnaireIdsByCampaignId(String campaignId); - Set findDistinctIdCampaigns(); + Set findDistinctCampaignIds(); - long countByIdCampaign(String idCampaign); + long countByCampaignId(String campaignId); - Set findDistinctIdQuestionnaires(); + Set findDistinctQuestionnaireIds(); - Set findIdCampaignsByIdQuestionnaire(String idQuestionnaire); + Set findCampaignIdsByQuestionnaireId(String questionnaireId); } diff --git a/src/main/java/fr/insee/genesis/domain/service/surveyunit/SurveyUnitService.java b/src/main/java/fr/insee/genesis/domain/service/surveyunit/SurveyUnitService.java index b4d8125f..0c144814 100644 --- a/src/main/java/fr/insee/genesis/domain/service/surveyunit/SurveyUnitService.java +++ b/src/main/java/fr/insee/genesis/domain/service/surveyunit/SurveyUnitService.java @@ -2,9 +2,9 @@ import fr.insee.bpm.metadata.model.VariablesMap; import fr.insee.genesis.controller.dto.CampaignWithQuestionnaire; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.controller.dto.QuestionnaireWithCampaign; import fr.insee.genesis.controller.dto.SurveyUnitDto; -import fr.insee.genesis.controller.dto.SurveyUnitId; import fr.insee.genesis.controller.dto.SurveyUnitInputDto; import fr.insee.genesis.controller.dto.VariableDto; import fr.insee.genesis.controller.dto.VariableInputDto; @@ -47,36 +47,36 @@ public void saveSurveyUnits(List surveyUnitModels) { } @Override - public List findByIdsUEAndQuestionnaire(String idUE, String idQuest) { - return surveyUnitPersistencePort.findByIds(idUE, idQuest); + public List findByIdsInterrogationAndQuestionnaire(String interrogationId, String questionnaireId) { + return surveyUnitPersistencePort.findByIds(interrogationId, questionnaireId); } @Override - public List findByIdUE(String idUE) { - return surveyUnitPersistencePort.findByIdUE(idUE); + public List findByInterrogationId(String interrogationId) { + return surveyUnitPersistencePort.findByInterrogationId(interrogationId); } @Override - public Stream findByIdQuestionnaire(String idQuestionnaire) { - return surveyUnitPersistencePort.findByIdQuestionnaire(idQuestionnaire); + public Stream findByQuestionnaireId(String questionnaireId) { + return surveyUnitPersistencePort.findByQuestionnaireId(questionnaireId); } /** * In this method we want to get the latest update for each variable of a survey unit * But we need to separate the updates by mode * So we will calculate the latest state for a given collection mode - * @param idUE : Survey unit id - * @param idQuest : Questionnaire id + * @param interrogationId : Survey unit id + * @param questionnaireId : Questionnaire id * @return the latest update for each variable of a survey unit */ @Override - public List findLatestByIdAndByIdQuestionnaire(String idUE, String idQuest) { + public List findLatestByIdAndByQuestionnaireId(String interrogationId, String questionnaireId) { List latestUpdatesbyVariables = new ArrayList<>(); - List surveyUnitModels = surveyUnitPersistencePort.findByIds(idUE, idQuest); + List surveyUnitModels = surveyUnitPersistencePort.findByIds(interrogationId, questionnaireId); List modes = getDistinctsModes(surveyUnitModels); modes.forEach(mode ->{ List suByMode = surveyUnitModels.stream() - .filter(surveyUnitDto -> surveyUnitDto.getMode().equals(mode)) + .filter(surveyUnitModel -> surveyUnitModel.getMode().equals(mode)) .sorted((o1, o2) -> o2.getRecordDate().compareTo(o1.getRecordDate())) //Sorting update by date (latest updates first by date of upload in database) .toList(); @@ -92,26 +92,26 @@ public List findLatestByIdAndByIdQuestionnaire(String idUE, Str if(latestUpdate.getExternalVariables() == null){ latestUpdate.setExternalVariables(new ArrayList<>()); } - latestUpdate.getCollectedVariables().forEach(colVar -> addedVariables.add(new IdLoopTuple(colVar.idVar(), - colVar.idLoop()))); - latestUpdate.getExternalVariables().forEach(extVar -> addedVariables.add(new IdLoopTuple(extVar.idVar(), ""))); + latestUpdate.getCollectedVariables().forEach(colVar -> addedVariables.add(new IdLoopTuple(colVar.varId(), + colVar.loopId()))); + latestUpdate.getExternalVariables().forEach(extVar -> addedVariables.add(new IdLoopTuple(extVar.varId(), ""))); suByMode.forEach(surveyUnitModel -> { List collectedVariablesToKeep = new ArrayList<>(); List externalVariablesToKeep = new ArrayList<>(); // We iterate over the variables of the update and add them to the list if they are not already added surveyUnitModel.getCollectedVariables().stream() - .filter(colVar -> !addedVariables.contains(new IdLoopTuple(colVar.idVar(), colVar.idLoop()))) + .filter(colVar -> !addedVariables.contains(new IdLoopTuple(colVar.varId(), colVar.loopId()))) .forEach(colVar -> { collectedVariablesToKeep.add(colVar); - addedVariables.add(new IdLoopTuple(colVar.idVar(), colVar.idLoop())); + addedVariables.add(new IdLoopTuple(colVar.varId(), colVar.loopId())); }); if (surveyUnitModel.getExternalVariables() != null){ surveyUnitModel.getExternalVariables().stream() - .filter(extVar -> !addedVariables.contains(new IdLoopTuple(extVar.idVar(), ""))) + .filter(extVar -> !addedVariables.contains(new IdLoopTuple(extVar.varId(), ""))) .forEach(extVar -> { externalVariablesToKeep.add(extVar); - addedVariables.add(new IdLoopTuple(extVar.idVar(), "")); + addedVariables.add(new IdLoopTuple(extVar.varId(), "")); }); } @@ -127,17 +127,17 @@ public List findLatestByIdAndByIdQuestionnaire(String idUE, Str } @Override - public SurveyUnitDto findLatestValuesByStateByIdAndByIdQuestionnaire(String idUE, String idQuest) { + public SurveyUnitDto findLatestValuesByStateByIdAndByQuestionnaireId(String interrogationId, String questionnaireId) { SurveyUnitDto surveyUnitDto = SurveyUnitDto.builder() - .surveyUnitId(idUE) + .interrogationId(interrogationId) .collectedVariables(new ArrayList<>()) .externalVariables(new ArrayList<>()) .build(); //Extract variables Map collectedVariableMap = new HashMap<>(); + List surveyUnitModels = surveyUnitPersistencePort.findByIds(interrogationId, questionnaireId); Map externalVariableMap = new HashMap<>(); - List surveyUnitModels = surveyUnitPersistencePort.findByIds(idUE, idQuest); List modes = getDistinctsModes(surveyUnitModels); modes.forEach(mode -> { List suByMode = surveyUnitModels.stream() @@ -152,38 +152,38 @@ public SurveyUnitDto findLatestValuesByStateByIdAndByIdQuestionnaire(String idUE } @Override - public List findDistinctIdUEsByIdQuestionnaire(String idQuestionnaire) { - List surveyUnitModels = surveyUnitPersistencePort.findIdUEsByIdQuestionnaire(idQuestionnaire); - List suIds = new ArrayList<>(); - surveyUnitModels.forEach(surveyUnitDto -> suIds.add(new SurveyUnitId(surveyUnitDto.getIdUE()))); + public List findDistinctInterrogationIdsByQuestionnaireId(String questionnaireId) { + List surveyUnitModels = surveyUnitPersistencePort.findInterrogationIdsByQuestionnaireId(questionnaireId); + List suIds = new ArrayList<>(); + surveyUnitModels.forEach(surveyUnitModel -> suIds.add(new InterrogationId(surveyUnitModel.getInterrogationId()))); return suIds.stream().distinct().toList(); } @Override - public List findIdUEsAndModesByIdQuestionnaire(String idQuestionnaire) { - List surveyUnitModels = surveyUnitPersistencePort.findIdUEsByIdQuestionnaire(idQuestionnaire); + public List findInterrogationIdsAndModesByQuestionnaireId(String questionnaireId) { + List surveyUnitModels = surveyUnitPersistencePort.findInterrogationIdsByQuestionnaireId(questionnaireId); return surveyUnitModels.stream().distinct().toList(); } @Override - public List findModesByIdQuestionnaire(String idQuestionnaire) { - List surveyUnitModels = surveyUnitPersistencePort.findIdUEsByIdQuestionnaire(idQuestionnaire); + public List findModesByQuestionnaireId(String questionnaireId) { + List surveyUnitModels = surveyUnitPersistencePort.findInterrogationIdsByQuestionnaireId(questionnaireId); List sources = new ArrayList<>(); - surveyUnitModels.forEach(surveyUnitDto -> sources.add(surveyUnitDto.getMode())); + surveyUnitModels.forEach(surveyUnitModel -> sources.add(surveyUnitModel.getMode())); return sources.stream().distinct().toList(); } @Override - public List findModesByIdCampaign(String idCampaign) { - List surveyUnitModels = surveyUnitPersistencePort.findIdUEsByIdCampaign(idCampaign); + public List findModesByCampaignId(String campaignId) { + List surveyUnitModels = surveyUnitPersistencePort.findInterrogationIdsByCampaignId(campaignId); List sources = new ArrayList<>(); - surveyUnitModels.forEach(surveyUnitDto -> sources.add(surveyUnitDto.getMode())); + surveyUnitModels.forEach(surveyUnitModel -> sources.add(surveyUnitModel.getMode())); return sources.stream().distinct().toList(); } @Override - public Long deleteByIdQuestionnaire(String idQuestionnaire) { - return surveyUnitPersistencePort.deleteByIdQuestionnaire(idQuestionnaire); + public Long deleteByQuestionnaireId(String questionnaireId) { + return surveyUnitPersistencePort.deleteByQuestionnaireId(questionnaireId); } @Override @@ -192,42 +192,42 @@ public long countResponses() { } @Override - public Set findIdQuestionnairesByIdCampaign(String idCampaign) { - return surveyUnitPersistencePort.findIdQuestionnairesByIdCampaign(idCampaign); + public Set findQuestionnaireIdsByCampaignId(String campaignId) { + return surveyUnitPersistencePort.findQuestionnaireIdsByCampaignId(campaignId); } @Override - public Set findDistinctIdCampaigns() { - return surveyUnitPersistencePort.findDistinctIdCampaigns(); + public Set findDistinctCampaignIds() { + return surveyUnitPersistencePort.findDistinctCampaignIds(); } @Override public List findCampaignsWithQuestionnaires() { List campaignsWithQuestionnaireList = new ArrayList<>(); - for(String idCampaign : findDistinctIdCampaigns()){ - Set questionnaires = findIdQuestionnairesByIdCampaign(idCampaign); - campaignsWithQuestionnaireList.add(new CampaignWithQuestionnaire(idCampaign,questionnaires)); + for(String campaignId : findDistinctCampaignIds()){ + Set questionnaires = findQuestionnaireIdsByCampaignId(campaignId); + campaignsWithQuestionnaireList.add(new CampaignWithQuestionnaire(campaignId,questionnaires)); } return campaignsWithQuestionnaireList; } @Override - public long countResponsesByIdCampaign(String idCampaign){ - return surveyUnitPersistencePort.countByIdCampaign(idCampaign); + public long countResponsesByCampaignId(String campaignId){ + return surveyUnitPersistencePort.countByCampaignId(campaignId); } @Override - public Set findDistinctIdQuestionnaires() { - return surveyUnitPersistencePort.findDistinctIdQuestionnaires(); + public Set findDistinctQuestionnaireIds() { + return surveyUnitPersistencePort.findDistinctQuestionnaireIds(); } @Override public List findQuestionnairesWithCampaigns() { List questionnaireWithCampaignList = new ArrayList<>(); - for(String idQuestionnaire : findDistinctIdQuestionnaires()){ - Set campaigns = surveyUnitPersistencePort.findIdCampaignsByIdQuestionnaire(idQuestionnaire); + for(String questionnaireId : findDistinctQuestionnaireIds()){ + Set campaigns = surveyUnitPersistencePort.findCampaignIdsByQuestionnaireId(questionnaireId); questionnaireWithCampaignList.add(new QuestionnaireWithCampaign( - idQuestionnaire, + questionnaireId, campaigns) ); @@ -255,10 +255,10 @@ public List parseEditedVariables( for (DataState state : statesReceived){ SurveyUnitModel surveyUnitModel = SurveyUnitModel.builder() - .idCampaign(surveyUnitInputDto.getCampaignId()) + .campaignId(surveyUnitInputDto.getCampaignId()) .mode(surveyUnitInputDto.getMode()) - .idQuest(surveyUnitInputDto.getIdQuestionnaire()) - .idUE(surveyUnitInputDto.getSurveyUnitId()) + .questionnaireId(surveyUnitInputDto.getIdQuestionnaire()) + .interrogationId(surveyUnitInputDto.getSurveyUnitId()) .state(state) .recordDate(LocalDateTime.now()) .collectedVariables(new ArrayList<>()) @@ -273,10 +273,10 @@ public List parseEditedVariables( //Collected variables management for(VariableInputDto editedVariableDto : editedCollectedVariables){ VariableModel collectedVariable = VariableModel.builder() - .idVar(editedVariableDto.getVariableName()) + .varId(editedVariableDto.getVariableName()) .values(new ArrayList<>()) - .idParent(LoopIdentifier.getRelatedVariableName(editedVariableDto.getVariableName(), variablesMap)) - .idLoop(editedVariableDto.getIdLoop()) + .parentId(LoopIdentifier.getRelatedVariableName(editedVariableDto.getVariableName(), variablesMap)) + .loopId(editedVariableDto.getIdLoop()) .build(); collectedVariable.values().add(editedVariableDto.getVariableStateInputDto().getValue()); @@ -293,7 +293,7 @@ public List parseEditedVariables( //Utils private static List getDistinctsModes(List surveyUnitModels) { List sources = new ArrayList<>(); - surveyUnitModels.forEach(surveyUnitDto -> sources.add(surveyUnitDto.getMode())); + surveyUnitModels.forEach(surveyUnitModel -> sources.add(surveyUnitModel.getMode())); return sources.stream().distinct().toList(); } @@ -311,14 +311,14 @@ private void extractVariables(SurveyUnitModel surveyUnitModel, surveyUnitModel.setCollectedVariables(new ArrayList<>()); } for (VariableModel collectedVariable : surveyUnitModel.getCollectedVariables()) { - IdLoopTuple idLoopTuple = new IdLoopTuple(collectedVariable.idVar(), collectedVariable.idLoop()); + IdLoopTuple idLoopTuple = new IdLoopTuple(collectedVariable.varId(), collectedVariable.loopId()); VariableDto variableDto = collectedVariableMap.get(idLoopTuple); //Create variable into map if not exists if (variableDto == null) { variableDto = VariableDto.builder() - .variableName(collectedVariable.idVar()) - .idLoop(collectedVariable.idLoop()) + .variableName(collectedVariable.varId()) + .idLoop(collectedVariable.loopId()) .variableStateDtoList(new ArrayList<>()) .build(); collectedVariableMap.put(idLoopTuple, variableDto); @@ -340,14 +340,14 @@ private void extractVariables(SurveyUnitModel surveyUnitModel, surveyUnitModel.setExternalVariables(new ArrayList<>()); } for(VariableModel externalVariable : surveyUnitModel.getExternalVariables()){ - IdLoopTuple idLoopTuple = new IdLoopTuple(externalVariable.idVar(), externalVariable.idLoop()); + IdLoopTuple idLoopTuple = new IdLoopTuple(externalVariable.varId(), externalVariable.loopId()); VariableDto variableDto = externalVariableMap.get(idLoopTuple); //Create variable into map if not exists if(variableDto == null){ variableDto = VariableDto.builder() - .variableName(externalVariable.idVar()) - .idLoop(externalVariable.idLoop()) + .variableName(externalVariable.varId()) + .idLoop(externalVariable.loopId()) .variableStateDtoList(new ArrayList<>()) .build(); externalVariableMap.put(idLoopTuple, variableDto); diff --git a/src/main/java/fr/insee/genesis/domain/service/volumetry/VolumetryLogService.java b/src/main/java/fr/insee/genesis/domain/service/volumetry/VolumetryLogService.java index a4600ca0..e324e61b 100644 --- a/src/main/java/fr/insee/genesis/domain/service/volumetry/VolumetryLogService.java +++ b/src/main/java/fr/insee/genesis/domain/service/volumetry/VolumetryLogService.java @@ -39,9 +39,9 @@ public void writeVolumetries(SurveyUnitApiPort surveyUnitApiPort) throws IOExcep Files.writeString(logFilePath, "campaign;volumetry\n"); //Write lines - Set campaigns = surveyUnitApiPort.findDistinctIdCampaigns(); + Set campaigns = surveyUnitApiPort.findDistinctCampaignIds(); for (String campaignId : campaigns) { - long countResult = surveyUnitApiPort.countResponsesByIdCampaign(campaignId); + long countResult = surveyUnitApiPort.countResponsesByCampaignId(campaignId); String line = campaignId + ";" + countResult + "\n"; diff --git a/src/main/java/fr/insee/genesis/domain/utils/DataVerifier.java b/src/main/java/fr/insee/genesis/domain/utils/DataVerifier.java index 72953946..87ef14a6 100644 --- a/src/main/java/fr/insee/genesis/domain/utils/DataVerifier.java +++ b/src/main/java/fr/insee/genesis/domain/utils/DataVerifier.java @@ -40,106 +40,106 @@ private DataVerifier() { * Verify data format in all surveyUnits DTOs * If there is at least 1 incorrect variable for a survey unit, a new SurveyUnitDto is created with "FORCED" status * The new surveyUnits are added to the list - * @param suDtosList list of SurveyUnitDtos to verify + * @param surveyUnitModelsList list of SurveyUnitDtos to verify * @param variablesMap VariablesMap containing definitions of each variable */ - public static void verifySurveyUnits(List suDtosList, VariablesMap variablesMap){ - List suDtosListForced = new ArrayList<>(); // Created FORCED SU DTOs + public static void verifySurveyUnits(List surveyUnitModelsList, VariablesMap variablesMap){ + List surveyUnitModelsListForced = new ArrayList<>(); // Created FORCED SU DTOs - for(String idUE : getIdUEs(suDtosList)) { // For each id of the list - List srcSuDtosOfIdUE = suDtosList.stream().filter(element -> element.getIdUE().equals(idUE)).toList(); + for(String interrogationId : getInterrogationIds(surveyUnitModelsList)) { // For each id of the list + List srcSurveyUnitModelsOfInterrogationId = surveyUnitModelsList.stream().filter(element -> element.getInterrogationId().equals(interrogationId)).toList(); List correctedCollectedVariables = new ArrayList<>(); List correctedExternalVariables = new ArrayList<>(); //Get corrected variables - collectedVariablesManagement(srcSuDtosOfIdUE, variablesMap, correctedCollectedVariables); - externalVariablesManagement(srcSuDtosOfIdUE, variablesMap, correctedExternalVariables); + collectedVariablesManagement(srcSurveyUnitModelsOfInterrogationId, variablesMap, correctedCollectedVariables); + externalVariablesManagement(srcSurveyUnitModelsOfInterrogationId, variablesMap, correctedExternalVariables); //Create FORCED if any corrected variable if(!correctedCollectedVariables.isEmpty() || !correctedExternalVariables.isEmpty()){ - SurveyUnitModel newForcedSuDto = createForcedDto(suDtosList, idUE, correctedCollectedVariables, correctedExternalVariables); - suDtosListForced.add(newForcedSuDto); + SurveyUnitModel newForcedSurveyUnitModel = createForcedDto(surveyUnitModelsList, interrogationId, correctedCollectedVariables, correctedExternalVariables); + surveyUnitModelsListForced.add(newForcedSurveyUnitModel); } } - suDtosList.addAll(suDtosListForced); + surveyUnitModelsList.addAll(surveyUnitModelsListForced); } private static SurveyUnitModel createForcedDto( - List suDtosList, - String idUE, + List surveyUnitModelsList, + String interrogationId, List correctedCollectedVariables, List correctedExternalVariables ) { - SurveyUnitModel sampleSuDto = suDtosList.stream().filter(element -> element.getIdUE().equals(idUE)).toList().getFirst(); - SurveyUnitModel newForcedSuDto = SurveyUnitModel.builder() - .idQuest(sampleSuDto.getIdQuest()) - .idCampaign(sampleSuDto.getIdCampaign()) - .idUE(idUE) + SurveyUnitModel sampleSurveyUnitModel = surveyUnitModelsList.stream().filter(element -> element.getInterrogationId().equals(interrogationId)).toList().getFirst(); + SurveyUnitModel newForcedSurveyUnitModel = SurveyUnitModel.builder() + .questionnaireId(sampleSurveyUnitModel.getQuestionnaireId()) + .campaignId(sampleSurveyUnitModel.getCampaignId()) + .interrogationId(interrogationId) .state(DataState.FORCED) - .mode(sampleSuDto.getMode()) + .mode(sampleSurveyUnitModel.getMode()) .recordDate(LocalDateTime.now()) - .fileDate(sampleSuDto.getFileDate()) + .fileDate(sampleSurveyUnitModel.getFileDate()) .collectedVariables(new ArrayList<>()) .externalVariables(new ArrayList<>()) .build(); for(VariableModel correctedCollectedVariable : correctedCollectedVariables){ - newForcedSuDto.getCollectedVariables().add( + newForcedSurveyUnitModel.getCollectedVariables().add( VariableModel.builder() - .idVar(correctedCollectedVariable.idVar()) + .varId(correctedCollectedVariable.varId()) .values(correctedCollectedVariable.values()) - .idLoop(correctedCollectedVariable.idLoop()) - .idParent(correctedCollectedVariable.idParent()) + .loopId(correctedCollectedVariable.loopId()) + .parentId(correctedCollectedVariable.parentId()) .build() ); } for(VariableModel correctedExternalVariable : correctedExternalVariables){ - newForcedSuDto.getExternalVariables().add( + newForcedSurveyUnitModel.getExternalVariables().add( VariableModel.builder() - .idVar(correctedExternalVariable.idVar()) + .varId(correctedExternalVariable.varId()) .values(correctedExternalVariable.values()) - .idLoop(correctedExternalVariable.idLoop()) - .idParent(correctedExternalVariable.idParent()) + .loopId(correctedExternalVariable.loopId()) + .parentId(correctedExternalVariable.parentId()) .build() ); } - return newForcedSuDto; + return newForcedSurveyUnitModel; } /** - * Fetch individual IdUEs of variable from the list - * @param suDtosList source list - * @return a set of IdUEs + * Fetch individual interrogationIds of variable from the list + * @param surveyUnitModelsList source list + * @return a set of interrogationIds */ - private static Set getIdUEs(List suDtosList) { - Set idUEs = new HashSet<>(); - for(SurveyUnitModel surveyUnitModel : suDtosList){ - idUEs.add(surveyUnitModel.getIdUE()); + private static Set getInterrogationIds(List surveyUnitModelsList) { + Set interrogationIds = new HashSet<>(); + for(SurveyUnitModel surveyUnitModel : surveyUnitModelsList){ + interrogationIds.add(surveyUnitModel.getInterrogationId()); } - return idUEs; + return interrogationIds; } /** * Adds the collected variables for the FORCED document - * @param srcSuDtosOfIdUE source Survey Unit documents associated with IdUE + * @param srcSurveyUnitModelsOfInterrogationId source Survey Unit documents associated with InterrogationId * @param variablesMap variables definitions * @param correctedCollectedVariables FORCED document variables */ - private static void collectedVariablesManagement(List srcSuDtosOfIdUE, VariablesMap variablesMap, List correctedCollectedVariables){ + private static void collectedVariablesManagement(List srcSurveyUnitModelsOfInterrogationId, VariablesMap variablesMap, List correctedCollectedVariables){ Set variableNames = new HashSet<>(); List variablesToVerify = new ArrayList<>(); //Sort from more priority to less - List sortedSuDtos = srcSuDtosOfIdUE.stream().sorted(Comparator.comparing(surveyUnitDto -> dataStatesPriority.get(surveyUnitDto.getState()))).toList(); + List sortedSurveyUnitModels = srcSurveyUnitModelsOfInterrogationId.stream().sorted(Comparator.comparing(surveyUnitModel -> dataStatesPriority.get(surveyUnitModel.getState()))).toList(); //Get more priority variables to verify - for(SurveyUnitModel srcSuDto : sortedSuDtos){ - for(VariableModel collectedVariable : srcSuDto.getCollectedVariables()){ - if(!variableNames.contains(collectedVariable.idVar())){ - variableNames.add(collectedVariable.idVar()); + for(SurveyUnitModel srcSurveyUnitModel : sortedSurveyUnitModels){ + for(VariableModel collectedVariable : srcSurveyUnitModel.getCollectedVariables()){ + if(!variableNames.contains(collectedVariable.varId())){ + variableNames.add(collectedVariable.varId()); variablesToVerify.add(collectedVariable); } } @@ -147,11 +147,11 @@ private static void collectedVariablesManagement(List srcSuDtos //Verify variables for(VariableModel collectedVariableToVerify : variablesToVerify){ - if(variablesMap.hasVariable(collectedVariableToVerify.idVar())) + if(variablesMap.hasVariable(collectedVariableToVerify.varId())) { VariableModel correctedCollectedVariable = verifyCollectedVariable( collectedVariableToVerify, - variablesMap.getVariable(collectedVariableToVerify.idVar()) + variablesMap.getVariable(collectedVariableToVerify.varId()) ); if(correctedCollectedVariable != null){ @@ -174,28 +174,27 @@ private static VariableModel verifyCollectedVariable(VariableModel collectedVari } } - return isInvalid ? VariableModel.builder() - .idVar(collectedVariable.idVar()) + .varId(collectedVariable.varId()) .values(newValues) - .idLoop(collectedVariable.idLoop()) - .idParent(collectedVariable.idParent()) + .loopId(collectedVariable.loopId()) + .parentId(collectedVariable.parentId()) .build() : null; } - private static void externalVariablesManagement(List srcSuDtosOfIdUE, VariablesMap variablesMap, List correctedExternalVariables) { + private static void externalVariablesManagement(List srcSurveyUnitModelsOfInterrogationId, VariablesMap variablesMap, List correctedExternalVariables) { //COLLECTED only - Optional collectedSuDtoOpt = srcSuDtosOfIdUE.stream().filter( - suDto -> suDto.getState().equals(DataState.COLLECTED) + Optional collectedSurveyUnitModelOpt = srcSurveyUnitModelsOfInterrogationId.stream().filter( + surveyUnitModel -> surveyUnitModel.getState().equals(DataState.COLLECTED) ).findFirst(); //Verify variables - if(collectedSuDtoOpt.isPresent()){ - for(VariableModel variable: collectedSuDtoOpt.get().getExternalVariables()){ - if(variablesMap.hasVariable(variable.idVar())) { + if(collectedSurveyUnitModelOpt.isPresent()){ + for(VariableModel variable: collectedSurveyUnitModelOpt.get().getExternalVariables()){ + if(variablesMap.hasVariable(variable.varId())) { VariableModel correctedExternalVariable = verifyExternalVariable( variable, - variablesMap.getVariable(variable.idVar()) + variablesMap.getVariable(variable.varId()) ); if (correctedExternalVariable != null) { correctedExternalVariables.add(correctedExternalVariable); @@ -225,7 +224,7 @@ private static VariableModel verifyExternalVariable(VariableModel externalVariab } return isInvalid ? VariableModel.builder() - .idVar(externalVariable.idVar()) + .varId(externalVariable.varId()) .values(newValues) .build() : null; } diff --git a/src/main/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitMongoAdapter.java b/src/main/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitMongoAdapter.java index d8a64d10..57c5ab1a 100644 --- a/src/main/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitMongoAdapter.java +++ b/src/main/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitMongoAdapter.java @@ -43,23 +43,23 @@ public void saveAll(List suListDto) { } @Override - public List findByIds(String idUE, String idQuest) { - List surveyUnits = mongoRepository.findByIdUEAndIdQuestionnaire(idUE, idQuest); + public List findByIds(String interrogationId, String questionnaireId) { + List surveyUnits = mongoRepository.findByInterrogationIdAndQuestionnaireId(interrogationId, questionnaireId); return surveyUnits.isEmpty() ? Collections.emptyList() : SurveyUnitDocumentMapper.INSTANCE.listDocumentToListModel(surveyUnits); } @Override - public List findByIdUE(String idUE) { - List surveyUnits = mongoRepository.findByIdUE(idUE); + public List findByInterrogationId(String questionnaireId) { + List surveyUnits = mongoRepository.findByInterrogationId(questionnaireId); return surveyUnits.isEmpty() ? Collections.emptyList() : SurveyUnitDocumentMapper.INSTANCE.listDocumentToListModel(surveyUnits); } @Override - public List findByIdUEsAndIdQuestionnaire(List idUEs, String idQuestionnaire) { + public List findByInterrogationIdsAndQuestionnaireId(List questionnaireIds, String questionnaireId) { List surveyUnits= new ArrayList<>(); // TODO: 18-10-2023 : find a way to do this in one query - idUEs.forEach(su -> { - List docs = mongoRepository.findByIdUEAndIdQuestionnaire(su.getIdUE(), idQuestionnaire); + questionnaireIds.forEach(su -> { + List docs = mongoRepository.findByInterrogationIdAndQuestionnaireId(su.getInterrogationId(), questionnaireId); surveyUnits.addAll(docs); }); return surveyUnits.isEmpty() ? Collections.emptyList() : SurveyUnitDocumentMapper.INSTANCE.listDocumentToListModel(surveyUnits); @@ -67,14 +67,14 @@ public List findByIdUEsAndIdQuestionnaire(List @Override - public Stream findByIdQuestionnaire(String idQuestionnaire) { - Stream surveyUnits = mongoRepository.findByIdQuestionnaire(idQuestionnaire); + public Stream findByQuestionnaireId(String questionnaireId) { + Stream surveyUnits = mongoRepository.findByQuestionnaireId(questionnaireId); return surveyUnits.map(SurveyUnitDocumentMapper.INSTANCE::documentToModel); } @Override - public Long deleteByIdQuestionnaire(String idQuestionnaire) { - return mongoRepository.deleteByIdQuestionnaire(idQuestionnaire); + public Long deleteByQuestionnaireId(String questionnaireId) { + return mongoRepository.deleteByQuestionnaireId(questionnaireId); } @Override @@ -83,80 +83,80 @@ public long count() { } @Override - public Set findIdQuestionnairesByIdCampaign(String idCampaign){ + public Set findQuestionnaireIdsByCampaignId(String campaignId){ Set mongoResponse = - mongoRepository.findIdQuestionnairesByIdCampaign(idCampaign); + mongoRepository.findQuestionnaireIdsByCampaignId(campaignId); - //Extract idQuestionnaires from JSON response - Set idQuestionnaires = new HashSet<>(); + //Extract questionnaireIds from JSON response + Set questionnaireIds = new HashSet<>(); for(String line : mongoResponse){ ObjectMapper objectMapper = new ObjectMapper(); try{ JsonNode jsonNode = objectMapper.readTree(line); - idQuestionnaires.add(jsonNode.get("idQuestionnaire").asText()); + questionnaireIds.add(jsonNode.get("questionnaireId").asText()); }catch (JsonProcessingException e){ log.error(e.getMessage()); } } - return idQuestionnaires; + return questionnaireIds; } @Override - public Set findDistinctIdCampaigns() { - Set idCampaigns = new HashSet<>(); - for(String idCampaign : mongoTemplate.getCollection(Constants.MONGODB_RESPONSE_COLLECTION_NAME).distinct("idCampaign", + public Set findDistinctCampaignIds() { + Set campaignIds = new HashSet<>(); + for(String campaignId : mongoTemplate.getCollection(Constants.MONGODB_RESPONSE_COLLECTION_NAME).distinct("campaignId", String.class)){ - idCampaigns.add(idCampaign); + campaignIds.add(campaignId); } - return idCampaigns; + return campaignIds; } @Override - public List findIdUEsByIdQuestionnaire(String idQuestionnaire) { - List surveyUnits = mongoRepository.findIdUEsByIdQuestionnaire(idQuestionnaire); + public List findInterrogationIdsByQuestionnaireId(String questionnaireId) { + List surveyUnits = mongoRepository.findInterrogationIdsByQuestionnaireId(questionnaireId); return surveyUnits.isEmpty() ? Collections.emptyList() : SurveyUnitDocumentMapper.INSTANCE.listDocumentToListModel(surveyUnits); } @Override - public List findIdUEsByIdCampaign(String idCampaign) { - List surveyUnits = mongoRepository.findIdUEsByIdCampaign(idCampaign); + public List findInterrogationIdsByCampaignId(String campaignId) { + List surveyUnits = mongoRepository.findInterrogationIdsByCampaignId(campaignId); return surveyUnits.isEmpty() ? Collections.emptyList() : SurveyUnitDocumentMapper.INSTANCE.listDocumentToListModel(surveyUnits); } - public long countByIdCampaign(String idCampaign){ - return mongoRepository.countByIdCampaign(idCampaign); + public long countByCampaignId(String campaignId){ + return mongoRepository.countByCampaignId(campaignId); } @Override - public Set findDistinctIdQuestionnaires() { - Set idQuestionnaires = new HashSet<>(); - for(String idQuestionnaire : mongoTemplate.getCollection(Constants.MONGODB_RESPONSE_COLLECTION_NAME).distinct( - "idQuestionnaire", + public Set findDistinctQuestionnaireIds() { + Set questionnaireIds = new HashSet<>(); + for(String questionnaireId : mongoTemplate.getCollection(Constants.MONGODB_RESPONSE_COLLECTION_NAME).distinct( + "questionnaireId", String.class)){ - idQuestionnaires.add(idQuestionnaire); + questionnaireIds.add(questionnaireId); } - return idQuestionnaires; + return questionnaireIds; } @Override - public Set findIdCampaignsByIdQuestionnaire(String idQuestionnaire) { + public Set findCampaignIdsByQuestionnaireId(String questionnaireId) { List mongoResponse = - mongoRepository.findIdCampaignsByIdQuestionnaire(idQuestionnaire).stream().distinct().toList(); + mongoRepository.findCampaignIdsByQuestionnaireId(questionnaireId).stream().distinct().toList(); //Extract idCampagigns from JSON response - Set idCampaigns = new HashSet<>(); + Set campaignIds = new HashSet<>(); for(String line : mongoResponse){ ObjectMapper objectMapper = new ObjectMapper(); try{ JsonNode jsonNode = objectMapper.readTree(line); - idCampaigns.add(jsonNode.get("idCampaign").asText()); + campaignIds.add(jsonNode.get("campaignId").asText()); }catch (JsonProcessingException e){ log.error(e.getMessage()); } } - return idCampaigns; + return campaignIds; } } diff --git a/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/SurveyUnitDocument.java b/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/SurveyUnitDocument.java index 2f7e47e1..70f8bc68 100644 --- a/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/SurveyUnitDocument.java +++ b/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/SurveyUnitDocument.java @@ -10,13 +10,13 @@ @Data @Document(collection = Constants.MONGODB_RESPONSE_COLLECTION_NAME) -@CompoundIndex(name = "idCampaign_1_idQuestionnaire_1", def = "{'idCampaign': 1, 'idQuestionnaire': 1}") //1 = ascending, -1 = descending -@CompoundIndex(name = "idQuestionnaire_1_idCampaign_1", def = "{'idQuestionnaire': 1, 'idCampaign': 1}") -@CompoundIndex(name = "idQuestionnaire_1_idUE_1", def = "{'idQuestionnaire': 1, 'idUE': 1}") +@CompoundIndex(name = "campaignId_1_questionnaireId_1", def = "{'campaignId': 1, 'questionnaireId': 1}") //1 = ascending, -1 = descending +@CompoundIndex(name = "questionnaireId_1_campaignId_1", def = "{'questionnaireId': 1, 'campaignId': 1}") +@CompoundIndex(name = "questionnaireId_1_interrogationId_1", def = "{'questionnaireId': 1, 'interrogationId': 1}") public class SurveyUnitDocument { - private String idCampaign; - private String idUE; - private String idQuestionnaire; + private String campaignId; + private String interrogationId; + private String questionnaireId; private String state; private String mode; private LocalDateTime recordDate; diff --git a/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/VariableDocument.java b/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/VariableDocument.java index cb562581..c093d41d 100644 --- a/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/VariableDocument.java +++ b/src/main/java/fr/insee/genesis/infrastructure/document/surveyunit/VariableDocument.java @@ -11,8 +11,8 @@ public class VariableDocument implements Serializable { @Serial private static final long serialVersionUID = -1576556180669134053L; - private String idVar; - private String idLoop; - private String idParent; + private String varId; + private String loopId; + private String parentId; private List values; } \ No newline at end of file diff --git a/src/main/java/fr/insee/genesis/infrastructure/mappers/SurveyUnitDocumentMapper.java b/src/main/java/fr/insee/genesis/infrastructure/mappers/SurveyUnitDocumentMapper.java index db0ada3b..f89ba3e1 100644 --- a/src/main/java/fr/insee/genesis/infrastructure/mappers/SurveyUnitDocumentMapper.java +++ b/src/main/java/fr/insee/genesis/infrastructure/mappers/SurveyUnitDocumentMapper.java @@ -12,10 +12,8 @@ public interface SurveyUnitDocumentMapper { SurveyUnitDocumentMapper INSTANCE = Mappers.getMapper(SurveyUnitDocumentMapper.class); - @Mapping(source = "idQuestionnaire", target = "idQuest") SurveyUnitModel documentToModel(SurveyUnitDocument surveyUnit); - @Mapping(source = "idQuest", target = "idQuestionnaire") SurveyUnitDocument modelToDocument(SurveyUnitModel surveyUnitModel); List listDocumentToListModel(List surveyUnits); diff --git a/src/main/java/fr/insee/genesis/infrastructure/repository/SurveyUnitMongoDBRepository.java b/src/main/java/fr/insee/genesis/infrastructure/repository/SurveyUnitMongoDBRepository.java index 18c2152b..88624365 100644 --- a/src/main/java/fr/insee/genesis/infrastructure/repository/SurveyUnitMongoDBRepository.java +++ b/src/main/java/fr/insee/genesis/infrastructure/repository/SurveyUnitMongoDBRepository.java @@ -13,28 +13,28 @@ @Repository public interface SurveyUnitMongoDBRepository extends MongoRepository { - List findByIdUE(String idUE); + List findByInterrogationId(String interrogationId); - List findByIdUEAndIdQuestionnaire(String idUE, String idQuestionnaire); + List findByInterrogationIdAndQuestionnaireId(String interrogationId, String questionnaireId); - @Query(value = "{ 'idQuestionnaire' : ?0 }", fields = "{ 'idUE' : 1, 'mode' : 1 }") - List findIdUEsByIdQuestionnaire(String idQuestionnaire); + @Query(value = "{ 'questionnaireId' : ?0 }", fields = "{ 'interrogationId' : 1, 'mode' : 1 }") + List findInterrogationIdsByQuestionnaireId(String questionnaireId); - @Query(value = "{ 'idCampaign' : ?0 }", fields = "{ 'idUE' : 1, 'mode' : 1 }") - List findIdUEsByIdCampaign(String idCampaign); + @Query(value = "{ 'campaignId' : ?0 }", fields = "{ 'interrogationId' : 1, 'mode' : 1 }") + List findInterrogationIdsByCampaignId(String campaignId); - Long deleteByIdQuestionnaire(String idQuestionnaire); + Long deleteByQuestionnaireId(String questionnaireId); @Meta(cursorBatchSize = 20) - Stream findByIdQuestionnaire(String idQuestionnaire); + Stream findByQuestionnaireId(String questionnaireId); long count(); - @Query(value = "{ 'idCampaign' : ?0 }", fields = "{ _id : 0, 'idQuestionnaire' : 1 }") - Set findIdQuestionnairesByIdCampaign(String idCampaign); + @Query(value = "{ 'campaignId' : ?0 }", fields = "{ _id : 0, 'questionnaireId' : 1 }") + Set findQuestionnaireIdsByCampaignId(String campaignId); - long countByIdCampaign(String idCampaign); + long countByCampaignId(String campaignId); - @Query(value = "{ 'idQuestionnaire' : ?0 }", fields = "{ _id : 0, 'idCampaign' : 1 }") - Set findIdCampaignsByIdQuestionnaire(String idQuestionnaire); + @Query(value = "{ 'questionnaireId' : ?0 }", fields = "{ _id : 0, 'campaignId' : 1 }") + Set findCampaignIdsByQuestionnaireId(String questionnaireId); } \ No newline at end of file diff --git a/src/test/java/cucumber/functional_tests/MainDefinitions.java b/src/test/java/cucumber/functional_tests/MainDefinitions.java index 11e8f5c3..e773f0fe 100644 --- a/src/test/java/cucumber/functional_tests/MainDefinitions.java +++ b/src/test/java/cucumber/functional_tests/MainDefinitions.java @@ -87,7 +87,7 @@ public void copy_data_file(String dataFile) throws IOException { StandardCopyOption.REPLACE_EXISTING); } - @When("We create DTOs from file {string} with DDI {string}") + @When("We create survey unit models from file {string} with DDI {string}") public void get_dtos(String fileName, String ddiName) throws IOException, ParserConfigurationException, SAXException, GenesisException, MetadataParserException { Path filePath = inDirectory.resolve(fileName); @@ -103,7 +103,7 @@ public void get_dtos(String fileName, String ddiName) throws IOException, Parser ).getVariables(); List surveyUnitModels1 = new ArrayList<>(); for (LunaticXmlSurveyUnit su : campaign.getSurveyUnits()) { - surveyUnitModels1.addAll(LunaticXmlAdapter.convert(su, variablesMap, campaign.getIdCampaign(), Mode.WEB)); + surveyUnitModels1.addAll(LunaticXmlAdapter.convert(su, variablesMap, campaign.getCampaignId(), Mode.WEB)); } surveyUnitQualityService.verifySurveyUnits(surveyUnitModels1,variablesMap); surveyUnitModels = surveyUnitModels1; @@ -111,7 +111,7 @@ public void get_dtos(String fileName, String ddiName) throws IOException, Parser } @When("We save data from that directory") - public void get_dtos_from_folder() throws Exception { + public void get_su_models_from_folder() throws Exception { responseController.saveResponsesFromXmlCampaignFolder(this.inDirectory.getFileName().toString(), null, true); } @@ -129,33 +129,33 @@ public void check_surveyunits_by_state(int expectedCount, String expectedStatus) - @Then("For SurveyUnit {string} there should be at least one {string} SurveyUnit DTO") - public void check_expected_datastate_dto(String surveyUnitId, String expectedDataState) { - Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().toString().equals(expectedDataState) - && surveyUnitDto.getIdUE().equals(surveyUnitId) + @Then("For SurveyUnit {string} there should be at least one {string} SurveyUnit Model") + public void check_expected_datastate_model(String interrogationId, String expectedDataState) { + Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().toString().equals(expectedDataState) + && surveyUnitModel.getInterrogationId().equals(interrogationId) ).isNotEmpty(); } - @Then("For SurveyUnit {string} there shouldn't be a {string} SurveyUnit DTO") - public void check_unexpected_datastate_dto(String surveyUnitId, String unexpectedDataState) { - Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().toString().equals(unexpectedDataState) - && surveyUnitDto.getIdUE().equals(surveyUnitId) + @Then("For SurveyUnit {string} there shouldn't be a {string} SurveyUnit Model") + public void check_unexpected_datastate_model(String interrogationId, String unexpectedDataState) { + Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().toString().equals(unexpectedDataState) + && surveyUnitModel.getInterrogationId().equals(interrogationId) ).isEmpty(); } - @Then("We should have a {string} DTO for survey unit {string} with {string} filled with {string} at index {int}") - public void check_survey_unit_dto_content(String dataState, String surveyUnitId, String variableName, String expectedValue, int expectedIndex) { + @Then("We should have a {string} Survey Unit model with id {string} with {string} filled with {string} at index {int}") + public void check_survey_unit_model_content(String dataState, String interrogationId, String variableName, String expectedValue, int expectedIndex) { //Get DTO - Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().toString().equals(dataState) - && surveyUnitDto.getIdUE().equals(surveyUnitId) + Assertions.assertThat(this.surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().toString().equals(dataState) + && surveyUnitModel.getInterrogationId().equals(interrogationId) ).isNotEmpty(); Optional concernedSurveyUnitModelOptional = this.surveyUnitModels.stream().filter(dto -> dto.getState().toString().equals(dataState) - && dto.getIdUE().equals(surveyUnitId) + && dto.getInterrogationId().equals(interrogationId) ).findFirst(); Assertions.assertThat(concernedSurveyUnitModelOptional).isPresent(); @@ -164,10 +164,10 @@ public void check_survey_unit_dto_content(String dataState, String surveyUnitId, //Get Variable Assertions.assertThat(concernedSurveyUnitModel.getCollectedVariables()).filteredOn(collectedvariableModel -> - collectedvariableModel.idVar().equals(variableName)).isNotEmpty(); + collectedvariableModel.varId().equals(variableName)).isNotEmpty(); Optional concernedCollectedVariableOptional = concernedSurveyUnitModel.getCollectedVariables().stream().filter(variable -> - variable.idVar().equals(variableName) + variable.varId().equals(variableName) ).findFirst(); Assertions.assertThat(concernedCollectedVariableOptional).isPresent(); @@ -186,20 +186,20 @@ public void external_variable_volumetric_check(int expectedNumberOfValues, Strin //Get SurveyUnitModel Assertions.assertThat(surveyUnitPersistence.getMongoStub()).filteredOn(surveyUnitModel -> surveyUnitModel.getState().equals(DataState.COLLECTED) - && surveyUnitModel.getIdUE().equals(interrogationId) + && surveyUnitModel.getInterrogationId().equals(interrogationId) ).isNotEmpty().hasSize(1); SurveyUnitModel surveyUnitModel = surveyUnitPersistence.getMongoStub().stream().filter(surveyUnitModel1 -> surveyUnitModel1.getState().equals(DataState.COLLECTED) - && surveyUnitModel1.getIdUE().equals(interrogationId) + && surveyUnitModel1.getInterrogationId().equals(interrogationId) ).toList().getFirst(); //Get Variable Assertions.assertThat(surveyUnitModel.getExternalVariables()).filteredOn(variableModel -> - variableModel.idVar().equals(externalVariableName)).isNotEmpty(); + variableModel.varId().equals(externalVariableName)).isNotEmpty(); Optional concernedVariableOptional = surveyUnitModel.getExternalVariables().stream().filter(variable -> - variable.idVar().equals(externalVariableName) + variable.varId().equals(externalVariableName) ).findFirst(); Assertions.assertThat(concernedVariableOptional).isPresent(); @@ -219,29 +219,29 @@ public void forExternalVariableInSurveyUnitWeShouldHaveForLoop(String externalVa //Get SurveyUnitModel Assertions.assertThat(surveyUnitPersistence.getMongoStub()).filteredOn(surveyUnitDto -> surveyUnitDto.getState().equals(DataState.COLLECTED) - && surveyUnitDto.getIdUE().equals(interrogationId) + && surveyUnitDto.getInterrogationId().equals(interrogationId) ).isNotEmpty().hasSize(1); SurveyUnitModel surveyUnitModel = surveyUnitPersistence.getMongoStub().stream().filter(dto -> dto.getState().equals(DataState.COLLECTED) - && dto.getIdUE().equals(interrogationId) + && dto.getInterrogationId().equals(interrogationId) ).toList().getFirst(); //Get Variable Assertions.assertThat(surveyUnitModel.getExternalVariables()).filteredOn(variableModel -> - variableModel.idVar().equals(externalVariableName) - && Objects.equals(variableModel.idLoop(), expectedLoopId) + variableModel.varId().equals(externalVariableName) + && Objects.equals(variableModel.loopId(), expectedLoopId) ).isNotEmpty().hasSize(1); VariableModel concernedExternalVariable = surveyUnitModel.getExternalVariables().stream().filter(variableModel -> - variableModel.idVar().equals(externalVariableName) - && Objects.equals(variableModel.idLoop(), expectedLoopId) + variableModel.varId().equals(externalVariableName) + && Objects.equals(variableModel.loopId(), expectedLoopId) ).toList().getFirst(); //Value content assertion Assertions.assertThat(concernedExternalVariable.values()).hasSize(1); Assertions.assertThat(concernedExternalVariable.values().getFirst()).isEqualTo(expectedValue); - Assertions.assertThat(concernedExternalVariable.idLoop()).isNotNull().isEqualTo(expectedLoopId); + Assertions.assertThat(concernedExternalVariable.loopId()).isNotNull().isEqualTo(expectedLoopId); } @Then("For external variable {string} in survey unit {string} we should have {string} as idLoop and {string} as first " + @@ -251,27 +251,27 @@ public void check_idLoop_and_value(String externalVariableName, String interroga //Get SurveyUnitModel Assertions.assertThat(surveyUnitPersistence.getMongoStub()).filteredOn(surveyUnitDto -> surveyUnitDto.getState().equals(DataState.COLLECTED) - && surveyUnitDto.getIdUE().equals(interrogationId) + && surveyUnitDto.getInterrogationId().equals(interrogationId) ).isNotEmpty().hasSize(1); SurveyUnitModel surveyUnitModel = surveyUnitPersistence.getMongoStub().stream().filter(dto -> dto.getState().equals(DataState.COLLECTED) - && dto.getIdUE().equals(interrogationId) + && dto.getInterrogationId().equals(interrogationId) ).toList().getFirst(); //Get Variable Assertions.assertThat(surveyUnitModel.getExternalVariables()).filteredOn(variableModel -> - variableModel.idVar().equals(externalVariableName) - && Objects.equals(variableModel.idLoop(), expectedLoopId) + variableModel.varId().equals(externalVariableName) + && Objects.equals(variableModel.loopId(), expectedLoopId) ).isNotEmpty().hasSize(1); VariableModel concernedExternalVariable = surveyUnitModel.getExternalVariables().stream().filter(variableModel -> - variableModel.idVar().equals(externalVariableName) - && Objects.equals(variableModel.idLoop(), expectedLoopId) + variableModel.varId().equals(externalVariableName) + && Objects.equals(variableModel.loopId(), expectedLoopId) ).toList().getFirst(); //Value content assertion - Assertions.assertThat(concernedExternalVariable.idLoop()).isNotNull().isEqualTo(expectedLoopId); + Assertions.assertThat(concernedExternalVariable.loopId()).isNotNull().isEqualTo(expectedLoopId); Assertions.assertThat(concernedExternalVariable.values()).hasSize(1); Assertions.assertThat(concernedExternalVariable.values().getFirst()).isEqualTo(expectedValue); } diff --git a/src/test/java/fr/insee/genesis/TestConstants.java b/src/test/java/fr/insee/genesis/TestConstants.java index f0d78aee..4faa5407 100644 --- a/src/test/java/fr/insee/genesis/TestConstants.java +++ b/src/test/java/fr/insee/genesis/TestConstants.java @@ -7,4 +7,7 @@ public class TestConstants { public static final String TEST_RESOURCES_DIRECTORY = "src/test/resources"; public static final String UNIT_TESTS_DDI_DIRECTORY = TEST_RESOURCES_DIRECTORY + "/specs"; + + public static final String DEFAULT_INTERROGATION_ID = "TESTINTERROGATIONID"; + public static final String DEFAULT_QUESTIONNAIRE_ID = "TESTQUESTIONNAIREID"; } diff --git a/src/test/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapterTest.java b/src/test/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapterTest.java index 3d84db65..7b7840a5 100644 --- a/src/test/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapterTest.java +++ b/src/test/java/fr/insee/genesis/controller/adapter/LunaticXmlAdapterTest.java @@ -53,8 +53,8 @@ void setUp() { lunaticXmlData.setCollected(collected); List external = List.of(); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit1.setId("idUE1"); - lunaticXmlSurveyUnit1.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit1.setId("interrogationId1"); + lunaticXmlSurveyUnit1.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit1.setData(lunaticXmlData); //SurveyUnit 2 : COLLECTED + EDITED @@ -72,8 +72,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit2.setId("idUE1"); - lunaticXmlSurveyUnit2.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit2.setId("interrogationId1"); + lunaticXmlSurveyUnit2.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit2.setData(lunaticXmlData); //SurveyUnit 3 : COLLECTED + EDITED + FORCED @@ -92,8 +92,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit3.setId("idUE1"); - lunaticXmlSurveyUnit3.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit3.setId("interrogationId1"); + lunaticXmlSurveyUnit3.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit3.setData(lunaticXmlData); //SurveyUnit 4 : COLLECTED + EDITED + PREVIOUS @@ -112,8 +112,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit4.setId("idUE1"); - lunaticXmlSurveyUnit4.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit4.setId("interrogationId1"); + lunaticXmlSurveyUnit4.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit4.setData(lunaticXmlData); //SurveyUnit 5 : COLLECTED + EDITED + PREVIOUS + INPUTED @@ -133,8 +133,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit5.setId("idUE1"); - lunaticXmlSurveyUnit5.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit5.setId("interrogationId1"); + lunaticXmlSurveyUnit5.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit5.setData(lunaticXmlData); //SurveyUnit 6 : COLLECTED only, has one unknown variable @@ -153,8 +153,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit6.setId("idUE1"); - lunaticXmlSurveyUnit6.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit6.setId("interrogationId1"); + lunaticXmlSurveyUnit6.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit6.setData(lunaticXmlData); //SurveyUnit 7 : COLLECTED only, has one unknown variable with known variable prefix @@ -172,8 +172,8 @@ void setUp() { lunaticXmlData.setCollected(collected); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit7.setId("idUE1"); - lunaticXmlSurveyUnit7.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit7.setId("interrogationId1"); + lunaticXmlSurveyUnit7.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit7.setData(lunaticXmlData); //SurveyUnit 8 : Only collected data @@ -188,8 +188,8 @@ void setUp() { lunaticXmlData.setCollected(collected); external = List.of(); lunaticXmlData.setExternal(external); - lunaticXmlSurveyUnit8.setId("idUE1"); - lunaticXmlSurveyUnit8.setQuestionnaireModelId("idQuest1"); + lunaticXmlSurveyUnit8.setId("interrogationId1"); + lunaticXmlSurveyUnit8.setQuestionnaireModelId("questionnaireId1"); lunaticXmlSurveyUnit8.setData(lunaticXmlData); //SurveyUnit 9 : External data only @@ -219,51 +219,51 @@ void setUp() { @DisplayName("SurveyUnitDto should not be null") void test01() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).isNotNull().isNotEmpty(); + Assertions.assertThat(surveyUnitModels).isNotNull().isNotEmpty(); } @Test - @DisplayName("SurveyUnitDto should have the right idQuest") + @DisplayName("SurveyUnitDto should have the right questionnaireId") void test02() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos.getFirst().getIdQuest()).isEqualTo("idQuest1"); + Assertions.assertThat(surveyUnitModels.getFirst().getQuestionnaireId()).isEqualTo("questionnaireId1"); } @Test @DisplayName("SurveyUnitDto should have the right id") void test03() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos.getFirst().getIdUE()).isEqualTo("idUE1"); + Assertions.assertThat(surveyUnitModels.getFirst().getInterrogationId()).isEqualTo("interrogationId1"); } @Test @DisplayName("SurveyUnitDto should contains 4 variable state updates") void test04() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit1, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).hasSize(4); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).hasSize(4); } @Test @DisplayName("There should be a EDITED DTO with EDITED data") void test05() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit2, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit2, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(2); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.EDITED) + Assertions.assertThat(surveyUnitModels).hasSize(2); + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.EDITED) ).isNotEmpty(); - Optional editedDTO = suDtos.stream().filter(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.EDITED) + Optional editedDTO = surveyUnitModels.stream().filter(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.EDITED) ).findFirst(); Assertions.assertThat(editedDTO).isPresent(); @@ -277,14 +277,14 @@ void test05() { @DisplayName("There should be both EDITED DTO and FORCED DTO if there is EDITED and FORCED data") void test06() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit3, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit3, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(3); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.EDITED) + Assertions.assertThat(surveyUnitModels).hasSize(3); + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.EDITED) ).isNotEmpty(); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.FORCED) + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.FORCED) ).isNotEmpty(); } @@ -292,14 +292,14 @@ void test06() { @DisplayName("There should be a EDITED DTO and PREVIOUS DTO if there is EDITED and PREVIOUS data") void test07() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit4, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit4, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(3); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.EDITED) + Assertions.assertThat(surveyUnitModels).hasSize(3); + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.EDITED) ).isNotEmpty(); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.PREVIOUS) + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.PREVIOUS) ).isNotEmpty(); } @@ -307,17 +307,17 @@ void test07() { @DisplayName("There should be multiple DTOs if there is different data states (all 4)") void test08() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit5, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit5, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(4); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.EDITED) + Assertions.assertThat(surveyUnitModels).hasSize(4); + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.EDITED) ).isNotEmpty(); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.PREVIOUS) + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.PREVIOUS) ).isNotEmpty(); - Assertions.assertThat(suDtos).filteredOn(surveyUnitDto -> - surveyUnitDto.getState().equals(DataState.INPUTED) + Assertions.assertThat(surveyUnitModels).filteredOn(surveyUnitModel -> + surveyUnitModel.getState().equals(DataState.INPUTED) ).isNotEmpty(); } @@ -325,52 +325,52 @@ void test08() { @DisplayName("If a variable not present in DDI then he is in the root group") void test09() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit6, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit6, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(1); - - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> - collectedVariableDto.idVar().equals("var3")).isNotEmpty(); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals("var3")).toList().getFirst().idParent()).isNull(); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals("var3")).toList().getFirst().idLoop()).isEqualTo(Constants.ROOT_GROUP_NAME); + Assertions.assertThat(surveyUnitModels).hasSize(1); + + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> + collectedVariableDto.varId().equals("var3")).isNotEmpty(); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals("var3")).toList().getFirst().parentId()).isNull(); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals("var3")).toList().getFirst().loopId()).isEqualTo(Constants.ROOT_GROUP_NAME); } @Test @DisplayName("If a variable A not present in DDI and is the extension of a known variable B, then the variable A has B as related and is in the same group") void test10() { // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit7, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit7, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(1); - - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> - collectedVariableDto.idVar().equals("var1_MISSING")).isNotEmpty(); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals("var1_MISSING")).toList().getFirst().idParent()).isNotNull().isEqualTo("var1"); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals("var1_MISSING")).toList().getFirst().idLoop()).isNotEqualTo(Constants.ROOT_GROUP_NAME).isEqualTo(LOOP_NAME); + Assertions.assertThat(surveyUnitModels).hasSize(1); + + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> + collectedVariableDto.varId().equals("var1_MISSING")).isNotEmpty(); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals("var1_MISSING")).toList().getFirst().parentId()).isNotNull().isEqualTo("var1"); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals("var1_MISSING")).toList().getFirst().loopId()).isNotEqualTo(Constants.ROOT_GROUP_NAME).isEqualTo(LOOP_NAME); } @Test @DisplayName("Value should be affected in the good loop iteration") void test11(){ // When - List suDtos = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit8, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); + List surveyUnitModels = LunaticXmlAdapter.convert(lunaticXmlSurveyUnit8, metadataModel.getVariables(), ID_CAMPAIGN, Mode.WEB); // Then - Assertions.assertThat(suDtos).hasSize(1); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).hasSize(3); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> - collectedVariableDto.idVar().equals("var1")).isNotEmpty(); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> - collectedVariableDto.idLoop().equals("BOUCLE1_1")).isEmpty(); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idLoop().equals("BOUCLE1_2")).toList().getFirst().values().getFirst()).isEqualTo("1"); - Assertions.assertThat(suDtos.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idLoop().equals("BOUCLE1_3")).toList().getFirst().values().getFirst()).isEqualTo("2"); + Assertions.assertThat(surveyUnitModels).hasSize(1); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).hasSize(3); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> + collectedVariableDto.varId().equals("var1")).isNotEmpty(); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables()).filteredOn(collectedVariableDto -> + collectedVariableDto.loopId().equals("BOUCLE1_1")).isEmpty(); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.loopId().equals("BOUCLE1_2")).toList().getFirst().values().getFirst()).isEqualTo("1"); + Assertions.assertThat(surveyUnitModels.getFirst().getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.loopId().equals("BOUCLE1_3")).toList().getFirst().values().getFirst()).isEqualTo("2"); } @@ -383,10 +383,10 @@ void test12(){ Assertions.assertThat(suDtos).hasSize(1); Assertions.assertThat(suDtos.getFirst().getCollectedVariables()).isEmpty(); Assertions.assertThat(suDtos.getFirst().getExternalVariables()).hasSize(1); - Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().idVar()).isEqualTo("extvar1"); + Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().varId()).isEqualTo("extvar1"); Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().values()).hasSize(1); Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().values().getFirst()).isEqualTo("ext"); - Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().idLoop()).isEqualTo(LOOP_NAME+"_1"); - Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().idParent()).isEqualTo(Constants.ROOT_GROUP_NAME); + Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().loopId()).isEqualTo(LOOP_NAME+"_1"); + Assertions.assertThat(suDtos.getFirst().getExternalVariables().getFirst().parentId()).isEqualTo(Constants.ROOT_GROUP_NAME); } } diff --git a/src/test/java/fr/insee/genesis/controller/rest/HealthCheckControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/HealthCheckControllerTest.java index 1dd54314..04847a89 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/HealthCheckControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/HealthCheckControllerTest.java @@ -30,24 +30,25 @@ static void init() { surveyUnitPersistencePortStub = new SurveyUnitPersistencePortStub(); SurveyUnitApiPort surveyUnitApiPort = new SurveyUnitService(surveyUnitPersistencePortStub); List externalVariableList = new ArrayList<>(); - VariableModel variable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel variable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + externalVariableList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); surveyUnitPersistencePortStub.getMongoStub().add(SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE("TESTIDUE") - .idQuest("TESTIDQUESTIONNAIRE") + .interrogationId("TESTINTERROGATIONID") + .questionnaireId("TESTQUESTIONNAIREID") .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023, 1, 1, 0, 0, 0)) .recordDate(LocalDateTime.of(2024, 1, 1, 0, 0, 0)) diff --git a/src/test/java/fr/insee/genesis/controller/rest/UtilsControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/UtilsControllerTest.java index 74e1a571..87d6f503 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/UtilsControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/UtilsControllerTest.java @@ -2,7 +2,7 @@ import cucumber.TestConstants; import fr.insee.genesis.Constants; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.domain.model.surveyunit.DataState; import fr.insee.genesis.domain.model.surveyunit.Mode; import fr.insee.genesis.domain.model.surveyunit.SurveyUnitModel; @@ -35,10 +35,10 @@ class UtilsControllerTest { static UtilsController utilsControllerStatic; static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; - static List surveyUnitIdList; + static List interrogationIdList; //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; + static final String defaultInterrogationId = "TESTINTERROGATIONID"; + static final String defaultQuestionnaireId = "TESTQUESTIONNAIREID"; @BeforeAll static void init() { @@ -50,8 +50,8 @@ static void init() { , new VolumetryLogService(new ConfigStub()) ); - surveyUnitIdList = new ArrayList<>(); - surveyUnitIdList.add(new SurveyUnitId(defaultIdUE)); + interrogationIdList = new ArrayList<>(); + interrogationIdList.add(new InterrogationId(defaultInterrogationId)); } @BeforeEach @@ -60,23 +60,23 @@ void reset() throws IOException { surveyUnitPersistencePortStub.getMongoStub().clear(); List externalVariableList = new ArrayList<>(); - VariableModel variable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel variable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); surveyUnitPersistencePortStub.getMongoStub().add(SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(defaultIdUE) - .idQuest(defaultIdQuest) + .interrogationId(defaultInterrogationId) + .questionnaireId(defaultQuestionnaireId) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023, 1, 1, 0, 0, 0)) .recordDate(LocalDateTime.of(2024, 1, 1, 0, 0, 0)) @@ -248,7 +248,7 @@ void saveVolumetryTest() throws IOException { .resolve(LocalDate.now().format(DateTimeFormatter.ofPattern(Constants.VOLUMETRY_FILE_DATE_FORMAT)) + Constants.VOLUMETRY_FILE_SUFFIX + ".csv"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTIDCAMPAIGN;1"); + Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTCAMPAIGNID;1"); //CLEAN Files.deleteIfExists(logFilePath); @@ -267,7 +267,7 @@ void saveVolumetryTest_overwrite() throws IOException { .resolve(LocalDate.now().format(DateTimeFormatter.ofPattern(Constants.VOLUMETRY_FILE_DATE_FORMAT)) + Constants.VOLUMETRY_FILE_SUFFIX + ".csv"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTIDCAMPAIGN;1").doesNotContain("TESTIDCAMPAIGN;1\nTESTIDCAMPAIGN;1"); + Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTCAMPAIGNID;1").doesNotContain("TESTCAMPAIGNID;1\nTESTCAMPAIGNID;1"); //CLEAN Files.deleteIfExists(logFilePath); @@ -276,7 +276,7 @@ void saveVolumetryTest_overwrite() throws IOException { @Test void saveVolumetryTest_additionnal_campaign() throws IOException { //Given - addAdditionalDtoToMongoStub("TESTIDCAMPAIGN2","TESTQUEST2"); + addAdditionalDtoToMongoStub("TESTCAMPAIGNID2","TESTQUEST2"); //WHEN ResponseEntity response = utilsControllerStatic.saveVolumetry(); @@ -288,7 +288,7 @@ void saveVolumetryTest_additionnal_campaign() throws IOException { .resolve(LocalDate.now().format(DateTimeFormatter.ofPattern(Constants.VOLUMETRY_FILE_DATE_FORMAT)) + Constants.VOLUMETRY_FILE_SUFFIX + ".csv"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTIDCAMPAIGN;1").contains("TESTIDCAMPAIGN2;1"); + Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTCAMPAIGNID;1").contains("TESTCAMPAIGNID2;1"); //CLEAN Files.deleteIfExists(logFilePath); @@ -297,7 +297,7 @@ void saveVolumetryTest_additionnal_campaign() throws IOException { void saveVolumetryTest_additionnal_campaign_and_document() throws IOException { //Given addAdditionalDtoToMongoStub("TESTQUEST"); - addAdditionalDtoToMongoStub("TESTIDCAMPAIGN2","TESTQUEST2"); + addAdditionalDtoToMongoStub("TESTCAMPAIGNID2","TESTQUEST2"); //WHEN ResponseEntity response = utilsControllerStatic.saveVolumetry(); @@ -309,7 +309,7 @@ void saveVolumetryTest_additionnal_campaign_and_document() throws IOException { .resolve(LocalDate.now().format(DateTimeFormatter.ofPattern(Constants.VOLUMETRY_FILE_DATE_FORMAT)) + Constants.VOLUMETRY_FILE_SUFFIX + ".csv"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTIDCAMPAIGN;2").contains("TESTIDCAMPAIGN2;1"); + Assertions.assertThat(logFilePath).exists().content().isNotEmpty().contains("TESTCAMPAIGNID;2").contains("TESTCAMPAIGNID2;1"); //CLEAN Files.deleteIfExists(logFilePath); @@ -347,29 +347,29 @@ void cleanOldVolumetryLogFiles() throws IOException { // Utilities - private void addAdditionalDtoToMongoStub(String idQuestionnaire) { - addAdditionalDtoToMongoStub("TESTIDCAMPAIGN",idQuestionnaire); + private void addAdditionalDtoToMongoStub(String questionnaireId) { + addAdditionalDtoToMongoStub("TESTCAMPAIGNID",questionnaireId); } - private void addAdditionalDtoToMongoStub(String idCampaign, String idQuestionnaire) { + private void addAdditionalDtoToMongoStub(String campaignId, String questionnaireId) { List externalVariableList = new ArrayList<>(); - VariableModel variable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel variable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign(idCampaign) + .campaignId(campaignId) .mode(Mode.WEB) - .idUE(defaultIdUE) - .idQuest(idQuestionnaire) + .interrogationId(defaultInterrogationId) + .questionnaireId(questionnaireId) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023, 2, 2, 0, 0, 0)) .recordDate(LocalDateTime.of(2024, 2, 2, 0, 0, 0)) diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/CampaignControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/responses/CampaignControllerTest.java index 94180008..c16366a8 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/CampaignControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/CampaignControllerTest.java @@ -20,8 +20,8 @@ class CampaignControllerTest { static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; + static final String DEFAULT_INTERROGATION_ID = "TESTINTERROGATIONID"; + static final String DEFAULT_ID_QUEST = "TESTQUESTIONNAIREID"; @BeforeAll static void init() { @@ -47,8 +47,8 @@ void getCampaignsTest() { ResponseEntity> response = campaignControllerStatic.getCampaigns(); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsExactly( - "TESTIDCAMPAIGN","TESTCAMPAIGN2"); + Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsOnly( + "TESTCAMPAIGNID","TESTCAMPAIGN2"); } @Test @@ -61,17 +61,17 @@ void getCampaignsWithQuestionnairesTest() { Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); Assertions.assertThat(response.getBody().stream().filter(campaignWithQuestionnaire -> - campaignWithQuestionnaire.getIdCampaign().equals("TESTIDCAMPAIGN") - || campaignWithQuestionnaire.getIdCampaign().equals("TESTCAMPAIGN2") + campaignWithQuestionnaire.getCampaignId().equals("TESTCAMPAIGNID") + || campaignWithQuestionnaire.getCampaignId().equals("TESTCAMPAIGN2") )).isNotNull().isNotEmpty().hasSize(2); Assertions.assertThat(response.getBody().stream().filter( - campaignWithQuestionnaire -> campaignWithQuestionnaire.getIdCampaign().equals("TESTIDCAMPAIGN") - ).findFirst().get().getQuestionnaires()).containsExactly(defaultIdQuest, "TESTQUESTIONNAIRE2"); + campaignWithQuestionnaire -> campaignWithQuestionnaire.getCampaignId().equals("TESTCAMPAIGNID") + ).findFirst().get().getQuestionnaires()).containsOnly(DEFAULT_ID_QUEST, "TESTQUESTIONNAIRE2"); Assertions.assertThat(response.getBody().stream().filter( - campaignWithQuestionnaire -> campaignWithQuestionnaire.getIdCampaign().equals("TESTCAMPAIGN2") - ).findFirst().get().getQuestionnaires()).containsExactly("TESTQUESTIONNAIRE2"); + campaignWithQuestionnaire -> campaignWithQuestionnaire.getCampaignId().equals("TESTCAMPAIGN2") + ).findFirst().get().getQuestionnaires()).containsOnly("TESTQUESTIONNAIRE2"); } } diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/IdUEControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/responses/InterrogationControllerTest.java similarity index 64% rename from src/test/java/fr/insee/genesis/controller/rest/responses/IdUEControllerTest.java rename to src/test/java/fr/insee/genesis/controller/rest/responses/InterrogationControllerTest.java index c750f9d1..bd75dbab 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/IdUEControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/InterrogationControllerTest.java @@ -1,6 +1,6 @@ package fr.insee.genesis.controller.rest.responses; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.domain.ports.api.SurveyUnitApiPort; import fr.insee.genesis.domain.service.surveyunit.SurveyUnitService; import fr.insee.genesis.stubs.SurveyUnitPersistencePortStub; @@ -13,20 +13,20 @@ import java.io.IOException; import java.util.List; -class IdUEControllerTest { +import static fr.insee.genesis.TestConstants.DEFAULT_INTERROGATION_ID; +import static fr.insee.genesis.TestConstants.DEFAULT_QUESTIONNAIRE_ID; + +class InterrogationControllerTest { //Given - static IdUEController idUeControllerStatic; + static InterrogationController interrogationControllerStatic; static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; - //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; @BeforeAll static void init() { surveyUnitPersistencePortStub = new SurveyUnitPersistencePortStub(); SurveyUnitApiPort surveyUnitApiPort = new SurveyUnitService(surveyUnitPersistencePortStub); - idUeControllerStatic = new IdUEController( surveyUnitApiPort ); + interrogationControllerStatic = new InterrogationController( surveyUnitApiPort ); } @@ -38,12 +38,12 @@ void reset() throws IOException { //When + Then @Test - void getAllIdUEsByQuestionnaireTest() { - ResponseEntity> response = idUeControllerStatic.getAllIdUEsByQuestionnaire(defaultIdQuest); + void getAllInterrogationIdsByQuestionnaireTest() { + ResponseEntity> response = interrogationControllerStatic.getAllInterrogationIdsByQuestionnaire(DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); - Assertions.assertThat(response.getBody().getFirst().getIdUE()).isEqualTo(defaultIdUE); + Assertions.assertThat(response.getBody().getFirst().getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); } diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/ModeControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/responses/ModeControllerTest.java index 06376ccd..aca7ca46 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/ModeControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/ModeControllerTest.java @@ -13,15 +13,13 @@ import java.io.IOException; import java.util.List; +import static fr.insee.genesis.TestConstants.DEFAULT_QUESTIONNAIRE_ID; + class ModeControllerTest { //Given static ModeController modeControllerStatic; static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; - //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; - @BeforeAll static void init() { surveyUnitPersistencePortStub = new SurveyUnitPersistencePortStub(); @@ -40,7 +38,7 @@ void reset() throws IOException { //When + Then @Test void getModesByQuestionnaireTest() { - ResponseEntity> response = modeControllerStatic.getModesByQuestionnaire(defaultIdQuest); + ResponseEntity> response = modeControllerStatic.getModesByQuestionnaire(DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().hasSize(1); @@ -49,7 +47,7 @@ void getModesByQuestionnaireTest() { @Test void getModesByCampaignTest() { - ResponseEntity> response = modeControllerStatic.getModesByCampaign("TESTIDCAMPAIGN"); + ResponseEntity> response = modeControllerStatic.getModesByCampaign("TESTCAMPAIGNID"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().hasSize(1); diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/QuestionnaireControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/responses/QuestionnaireControllerTest.java index 17caeb3b..88790b58 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/QuestionnaireControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/QuestionnaireControllerTest.java @@ -14,21 +14,19 @@ import java.util.List; import java.util.Set; +import static fr.insee.genesis.TestConstants.DEFAULT_QUESTIONNAIRE_ID; + class QuestionnaireControllerTest { //Given static QuestionnaireController questionnaireControllerStatic; static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; - //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; - @BeforeAll static void init() { surveyUnitPersistencePortStub = new SurveyUnitPersistencePortStub(); SurveyUnitApiPort surveyUnitApiPort = new SurveyUnitService(surveyUnitPersistencePortStub); - questionnaireControllerStatic = new QuestionnaireController(surveyUnitApiPort ); + questionnaireControllerStatic = new QuestionnaireController(surveyUnitApiPort); } @@ -47,19 +45,19 @@ void getQuestionnairesTest() { ResponseEntity> response = questionnaireControllerStatic.getQuestionnaires(); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsExactly( - defaultIdQuest,"TESTQUESTIONNAIRE2"); + Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsOnly( + DEFAULT_QUESTIONNAIRE_ID,"TESTQUESTIONNAIRE2"); } @Test void getQuestionnairesByCampaignTest() { Utils.addAdditionalDtoToMongoStub("TESTQUESTIONNAIRE2", surveyUnitPersistencePortStub); - ResponseEntity> response = questionnaireControllerStatic.getQuestionnairesByCampaign("TESTIDCAMPAIGN"); + ResponseEntity> response = questionnaireControllerStatic.getQuestionnairesByCampaign("TESTCAMPAIGNID"); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); - Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsExactly( - defaultIdQuest,"TESTQUESTIONNAIRE2"); + Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty().containsOnly( + DEFAULT_QUESTIONNAIRE_ID,"TESTQUESTIONNAIRE2"); } @@ -75,17 +73,17 @@ void getQuestionnairesWithCampaignsTest() { Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); Assertions.assertThat(response.getBody().stream().filter(questionnaireWithCampaign -> - questionnaireWithCampaign.getIdQuestionnaire().equals(defaultIdQuest) - || questionnaireWithCampaign.getIdQuestionnaire().equals("TESTQUESTIONNAIRE2") + questionnaireWithCampaign.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) + || questionnaireWithCampaign.getQuestionnaireId().equals("TESTQUESTIONNAIRE2") )).isNotNull().isNotEmpty().hasSize(2); Assertions.assertThat(response.getBody().stream().filter( - questionnaireWithCampaign -> questionnaireWithCampaign.getIdQuestionnaire().equals(defaultIdQuest) - ).findFirst().get().getCampaigns()).containsExactly("TESTIDCAMPAIGN"); + questionnaireWithCampaign -> questionnaireWithCampaign.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) + ).findFirst().get().getCampaigns()).containsExactly("TESTCAMPAIGNID"); Assertions.assertThat(response.getBody().stream().filter( - questionnaireWithCampaign -> questionnaireWithCampaign.getIdQuestionnaire().equals("TESTQUESTIONNAIRE2") - ).findFirst().get().getCampaigns()).containsExactly("TESTIDCAMPAIGN", "TESTCAMPAIGN2"); + questionnaireWithCampaign -> questionnaireWithCampaign.getQuestionnaireId().equals("TESTQUESTIONNAIRE2") + ).findFirst().get().getCampaigns()).containsExactly("TESTCAMPAIGNID", "TESTCAMPAIGN2"); } } diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/ResponseControllerTest.java b/src/test/java/fr/insee/genesis/controller/rest/responses/ResponseControllerTest.java index 3745e026..2367e7a3 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/ResponseControllerTest.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/ResponseControllerTest.java @@ -4,7 +4,7 @@ import fr.insee.genesis.Constants; import fr.insee.genesis.configuration.Config; import fr.insee.genesis.controller.dto.SurveyUnitDto; -import fr.insee.genesis.controller.dto.SurveyUnitId; +import fr.insee.genesis.controller.dto.InterrogationId; import fr.insee.genesis.controller.dto.SurveyUnitInputDto; import fr.insee.genesis.controller.dto.SurveyUnitSimplified; import fr.insee.genesis.controller.dto.VariableInputDto; @@ -43,14 +43,16 @@ import java.util.ArrayList; import java.util.List; +import static fr.insee.genesis.TestConstants.DEFAULT_INTERROGATION_ID; +import static fr.insee.genesis.TestConstants.DEFAULT_QUESTIONNAIRE_ID; + class ResponseControllerTest { //Given static ResponseController responseControllerStatic; static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; static LunaticXmlPersistanceStub lunaticXmlPersistanceStub; static LunaticJsonPersistanceStub lunaticJsonPersistanceStub; - - static List surveyUnitIdList; + static List interrogationIdList; //Constants static final String DEFAULT_ID_UE = "TESTIDUE"; static final String DEFAULT_ID_QUEST = "TESTIDQUESTIONNAIRE"; @@ -80,8 +82,8 @@ static void init() { , new AuthUtils(config) ); - surveyUnitIdList = new ArrayList<>(); - surveyUnitIdList.add(new SurveyUnitId(DEFAULT_ID_UE)); + interrogationIdList = new ArrayList<>(); + interrogationIdList.add(new InterrogationId(DEFAULT_INTERROGATION_ID)); } @BeforeEach @@ -208,21 +210,21 @@ void saveResponsesFromAllCampaignFoldersTests(){ //Gets @Test void findResponsesByUEAndQuestionnaireTest() { - ResponseEntity> response = responseControllerStatic.findResponsesByUEAndQuestionnaire(DEFAULT_ID_UE, DEFAULT_ID_QUEST); + ResponseEntity> response = responseControllerStatic.findResponsesByInterrogationAndQuestionnaire(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); - Assertions.assertThat(response.getBody().getFirst().getIdUE()).isEqualTo(DEFAULT_ID_UE); - Assertions.assertThat(response.getBody().getFirst().getIdQuest()).isEqualTo(DEFAULT_ID_QUEST); + Assertions.assertThat(response.getBody().getFirst().getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); + Assertions.assertThat(response.getBody().getFirst().getQuestionnaireId()).isEqualTo(DEFAULT_QUESTIONNAIRE_ID); } @Test void findAllResponsesByQuestionnaireTest() { - Path path = Path.of(TestConstants.TEST_RESOURCES_DIRECTORY, "OUT", DEFAULT_ID_QUEST); + Path path = Path.of(TestConstants.TEST_RESOURCES_DIRECTORY, "OUT", DEFAULT_QUESTIONNAIRE_ID); File dir = new File(String.valueOf(path)); FileSystemUtils.deleteRecursively(dir); - ResponseEntity response = responseControllerStatic.findAllResponsesByQuestionnaire(DEFAULT_ID_QUEST); + ResponseEntity response = responseControllerStatic.findAllResponsesByQuestionnaire(DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull(); @@ -240,14 +242,14 @@ void getAllResponsesByQuestionnaireTestSequential() throws IOException { surveyUnitPersistencePortStub.getMongoStub().clear(); for (int i = 0; i < Constants.BATCH_SIZE + 2; i++) { - Utils.addAdditionalDtoToMongoStub("TESTIDCAMPAIGN", DEFAULT_ID_UE + i, + Utils.addAdditionalDtoToMongoStub("TESTCAMPAIGNID", DEFAULT_INTERROGATION_ID + i, LocalDateTime.of(2023, 1, 1, 0, 0, 0), LocalDateTime.of(2024, 1, 1, 0, 0, 0), surveyUnitPersistencePortStub); } //When - ResponseEntity response = responseControllerStatic.findAllResponsesByQuestionnaire(DEFAULT_ID_QUEST); + ResponseEntity response = responseControllerStatic.findAllResponsesByQuestionnaire(DEFAULT_QUESTIONNAIRE_ID); //Then Assertions.assertThat(response).isNotNull(); @@ -264,32 +266,32 @@ void getAllResponsesByQuestionnaireTestSequential() throws IOException { void getLatestByUETest() { Utils.addAdditionalDtoToMongoStub(surveyUnitPersistencePortStub); - ResponseEntity> response = responseControllerStatic.getLatestByUE(DEFAULT_ID_UE, DEFAULT_ID_QUEST); + ResponseEntity> response = responseControllerStatic.getLatestByInterrogation(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); - Assertions.assertThat(response.getBody().getFirst().getIdUE()).isEqualTo(DEFAULT_ID_UE); - Assertions.assertThat(response.getBody().getFirst().getIdQuest()).isEqualTo(DEFAULT_ID_QUEST); + Assertions.assertThat(response.getBody().getFirst().getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); + Assertions.assertThat(response.getBody().getFirst().getQuestionnaireId()).isEqualTo(DEFAULT_QUESTIONNAIRE_ID); Assertions.assertThat(response.getBody().getFirst().getFileDate()).hasMonth(Month.FEBRUARY); } @Test void getLatestByUEOneObjectTest() { - ResponseEntity response = responseControllerStatic.getLatestByUEOneObject(DEFAULT_ID_UE, DEFAULT_ID_QUEST, Mode.WEB); + ResponseEntity response = responseControllerStatic.getLatestByInterrogationOneObject(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID, Mode.WEB); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull(); - Assertions.assertThat(response.getBody().getIdUE()).isEqualTo(DEFAULT_ID_UE); - Assertions.assertThat(response.getBody().getIdQuest()).isEqualTo(DEFAULT_ID_QUEST); + Assertions.assertThat(response.getBody().getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); + Assertions.assertThat(response.getBody().getQuestionnaireId()).isEqualTo(DEFAULT_QUESTIONNAIRE_ID); } @Test void getLatestForUEListTest() { - ResponseEntity> response = responseControllerStatic.getLatestForUEList(DEFAULT_ID_QUEST, surveyUnitIdList); + ResponseEntity> response = responseControllerStatic.getLatestForInterrogationList(DEFAULT_QUESTIONNAIRE_ID, interrogationIdList); Assertions.assertThat(response.getStatusCode().is2xxSuccessful()).isTrue(); Assertions.assertThat(response.getBody()).isNotNull().isNotEmpty(); - Assertions.assertThat(response.getBody().getFirst().getIdUE()).isEqualTo(DEFAULT_ID_UE); + Assertions.assertThat(response.getBody().getFirst().getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); } // Perret tests @@ -326,9 +328,9 @@ void getLatestByStatesSurveyDataTest(){ //WHEN - ResponseEntity response = responseControllerStatic.findResponsesByUEAndQuestionnaireLatestStates( - DEFAULT_ID_UE, - DEFAULT_ID_QUEST + ResponseEntity response = responseControllerStatic.findResponsesByInterrogationAndQuestionnaireLatestStates( + DEFAULT_INTERROGATION_ID, + DEFAULT_QUESTIONNAIRE_ID ); @@ -336,7 +338,7 @@ void getLatestByStatesSurveyDataTest(){ SurveyUnitDto surveyUnitDto = response.getBody(); Assertions.assertThat(surveyUnitDto).isNotNull(); - Assertions.assertThat(surveyUnitDto.getSurveyUnitId()).isEqualTo(DEFAULT_ID_UE); + Assertions.assertThat(surveyUnitDto.getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); Assertions.assertThat(surveyUnitDto.getCollectedVariables().getFirst().getVariableName()) .isEqualTo("TESTIDVAR"); @@ -423,8 +425,8 @@ void saveEditedTest() { //THEN Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub()).hasSize(1); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getIdCampaign()).isEqualTo(campaignId); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getIdQuest()).isEqualTo(idQuest); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCampaignId()).isEqualTo(campaignId); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getQuestionnaireId()).isEqualTo(idQuest); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getMode()).isEqualTo(Mode.WEB); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getState()).isEqualTo(DataState.EDITED); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getFileDate()).isNull(); @@ -432,9 +434,9 @@ void saveEditedTest() { Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getExternalVariables()).isEmpty(); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables()).hasSize(1); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idVar()).isEqualTo(idVar); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idLoop()).isEqualTo(idLoop); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idParent()).isEqualTo(Constants.ROOT_GROUP_NAME); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().varId()).isEqualTo(idVar); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().loopId()).isEqualTo(idLoop); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().parentId()).isEqualTo(Constants.ROOT_GROUP_NAME); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().values()).hasSize(1); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().values().getFirst()).isEqualTo(editedValue); @@ -489,8 +491,8 @@ void saveEditedTest_DocumentEdited() { //THEN //EDITED document assertions Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub()).hasSize(2); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getIdCampaign()).isEqualTo(campaignId); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getIdQuest()).isEqualTo(idQuest); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCampaignId()).isEqualTo(campaignId); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getQuestionnaireId()).isEqualTo(idQuest); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getState()).isEqualTo(DataState.EDITED); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getMode()).isEqualTo(Mode.WEB); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getFileDate()).isNull(); @@ -498,9 +500,9 @@ void saveEditedTest_DocumentEdited() { Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getExternalVariables()).isEmpty(); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables()).hasSize(2); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idVar()).isEqualTo(idVar); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idLoop()).isEqualTo(idLoop); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().idParent()).isEqualTo(Constants.ROOT_GROUP_NAME); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().varId()).isEqualTo(idVar); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().loopId()).isEqualTo(idLoop); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().parentId()).isEqualTo(Constants.ROOT_GROUP_NAME); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().values()).hasSize(1); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getCollectedVariables().getFirst().values().getFirst()).isEqualTo(editedValue); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getFirst().getModifiedBy()).isNull(); @@ -555,8 +557,8 @@ void saveEditedTest_DocumentForced() { Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub()).hasSize(2); //FORCED document assertions - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getIdCampaign()).isEqualTo(campaignId); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getIdQuest()).isEqualTo(idQuest); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCampaignId()).isEqualTo(campaignId); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getQuestionnaireId()).isEqualTo(idQuest); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getState()).isEqualTo(DataState.FORCED); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getMode()).isEqualTo(Mode.WEB); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getFileDate()).isNull(); @@ -564,9 +566,9 @@ void saveEditedTest_DocumentForced() { Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getExternalVariables()).isEmpty(); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables()).hasSize(1); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().idVar()).isEqualTo(idVar2); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().idLoop()).isEqualTo(idLoop); - Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().idParent()).isEqualTo(Constants.ROOT_GROUP_NAME); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().varId()).isEqualTo(idVar2); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().loopId()).isEqualTo(idLoop); + Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().parentId()).isEqualTo(Constants.ROOT_GROUP_NAME); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().values()).hasSize(1); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getCollectedVariables().getFirst().values().getFirst()).isNotNull().isEmpty(); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub().getLast().getModifiedBy()).isNull(); diff --git a/src/test/java/fr/insee/genesis/controller/rest/responses/Utils.java b/src/test/java/fr/insee/genesis/controller/rest/responses/Utils.java index e3681493..c199fb36 100644 --- a/src/test/java/fr/insee/genesis/controller/rest/responses/Utils.java +++ b/src/test/java/fr/insee/genesis/controller/rest/responses/Utils.java @@ -20,14 +20,14 @@ class Utils { //Constants - static final String defaultIdUE = "TESTIDUE"; - static final String defaultIdQuest = "TESTIDQUESTIONNAIRE"; + static final String DEFAULT_INTERROGATION_ID = "TESTINTERROGATIONID"; + static final String DEFAULT_ID_QUEST = "TESTQUESTIONNAIREID"; static void reset(SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) throws IOException { //MongoDB stub management surveyUnitPersistencePortStub.getMongoStub().clear(); - addAdditionalDtoToMongoStub("TESTIDCAMPAIGN", defaultIdQuest , + addAdditionalDtoToMongoStub("TESTCAMPAIGNID", DEFAULT_ID_QUEST, LocalDateTime.of(2023, 1, 1, 0, 0, 0), LocalDateTime.of(2024, 1, 1, 0, 0, 0), surveyUnitPersistencePortStub); @@ -166,43 +166,42 @@ static void reset(SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) t } static void addAdditionalDtoToMongoStub(SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { - addAdditionalDtoToMongoStub(defaultIdQuest, surveyUnitPersistencePortStub); + addAdditionalDtoToMongoStub(DEFAULT_ID_QUEST, surveyUnitPersistencePortStub); } - static void addAdditionalDtoToMongoStub(String idQuestionnaire, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { - addAdditionalDtoToMongoStub("TESTIDCAMPAIGN",idQuestionnaire, surveyUnitPersistencePortStub); + static void addAdditionalDtoToMongoStub(String questionnaireId, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { + addAdditionalDtoToMongoStub("TESTCAMPAIGNID",questionnaireId, surveyUnitPersistencePortStub); } - static void addAdditionalDtoToMongoStub(String idCampaign, String idQuestionnaire, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { - addAdditionalDtoToMongoStub(idCampaign,idQuestionnaire, + static void addAdditionalDtoToMongoStub(String campaignId, String questionnaireId, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { + addAdditionalDtoToMongoStub(campaignId,questionnaireId, LocalDateTime.of(2023, 2, 2, 0, 0, 0), LocalDateTime.of(2024, 2, 2, 0, 0, 0), surveyUnitPersistencePortStub); } - static void addAdditionalDtoToMongoStub(String idCampaign, String idQuestionnaire, + static void addAdditionalDtoToMongoStub(String campaignId, String questionnaireId, LocalDateTime fileDate, LocalDateTime recordDate, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { List externalVariableList = new ArrayList<>(); - VariableModel externalVariable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel externalVariable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(externalVariable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); - collectedVariableList.add(collectedVariable); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign(idCampaign) + .campaignId(campaignId) .mode(Mode.WEB) - .idUE(defaultIdUE) - .idQuest(idQuestionnaire) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(questionnaireId) .state(DataState.COLLECTED) .fileDate(fileDate) .recordDate(recordDate) @@ -221,24 +220,24 @@ static void addAdditionalDtoToMongoStub(DataState state, LocalDateTime recordDate, SurveyUnitPersistencePortStub surveyUnitPersistencePortStub) { List externalVariableList = new ArrayList<>(); - VariableModel variable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{externalVariableValue})).build(); + VariableModel variable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{externalVariableValue})).build(); externalVariableList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{collectedVariableValue})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(defaultIdUE) - .idQuest(defaultIdQuest) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(DEFAULT_ID_QUEST) .state(state) .fileDate(fileDate) .recordDate(recordDate) diff --git a/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParserTest.java b/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParserTest.java index 33775160..352f62ae 100644 --- a/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParserTest.java +++ b/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataParserTest.java @@ -20,7 +20,7 @@ static void setUp() throws Exception { @Test void campaignHasGoodLabelAndId() { Assertions.assertThat(campaign.getLabel()).isEqualTo("Enquête Test Unitaire"); - Assertions.assertThat(campaign.getIdCampaign()).isEqualTo("TEST2023X01"); + Assertions.assertThat(campaign.getCampaignId()).isEqualTo("TEST2023X01"); } @Test diff --git a/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParserTest.java b/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParserTest.java index 74d6b37d..9cf741be 100644 --- a/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParserTest.java +++ b/src/test/java/fr/insee/genesis/controller/sources/xml/LunaticXmlDataSequentialParserTest.java @@ -36,7 +36,7 @@ void setUp() throws Exception { @Test void campaignHasGoodLabelAndId() { Assertions.assertThat(campaign.getLabel()).isEqualTo("Enquête Test Unitaire"); - Assertions.assertThat(campaign.getIdCampaign()).isEqualTo("TEST2023X01"); + Assertions.assertThat(campaign.getCampaignId()).isEqualTo("TEST2023X01"); } @Test diff --git a/src/test/java/fr/insee/genesis/controller/utils/DataVerifierTest.java b/src/test/java/fr/insee/genesis/controller/utils/DataVerifierTest.java index de6f3006..5abf3ffa 100644 --- a/src/test/java/fr/insee/genesis/controller/utils/DataVerifierTest.java +++ b/src/test/java/fr/insee/genesis/controller/utils/DataVerifierTest.java @@ -87,49 +87,48 @@ static void setUp() { //Manual modifications //Valid 2nd variable on 5th and 13th case - SurveyUnitModel suDto = testSurveyUnitModels.stream().filter(surveyUnitDto -> - surveyUnitDto.getIdUE().equals("TestUE5") + SurveyUnitModel surveyUnitModel = testSurveyUnitModels.stream().filter(surveyUnitModel1 -> + surveyUnitModel1.getInterrogationId().equals("TestUE5") ).toList().getFirst(); - suDto.getCollectedVariables().get(1).values().set(0,"1"); + surveyUnitModel.getCollectedVariables().get(1).values().set(0,"1"); - suDto = testSurveyUnitModels.stream().filter(surveyUnitDto -> - surveyUnitDto.getIdUE().equals("TestUE13") + surveyUnitModel = testSurveyUnitModels.stream().filter(surveyUnitModel1 -> + surveyUnitModel1.getInterrogationId().equals("TestUE13") ).toList().getFirst(); - suDto.getCollectedVariables().get(1).values().set(0,"1"); - suDto.getExternalVariables().get(1).values().set(0,"1"); - + surveyUnitModel.getCollectedVariables().get(1).values().set(0,"1"); + surveyUnitModel.getExternalVariables().get(1).values().set(0,"1"); //Valid EDITED variables on 3rd and 7th case for priority test - SurveyUnitModel suDtoEdited = testSurveyUnitModels.stream().filter(surveyUnitDto -> - surveyUnitDto.getIdUE().equals("TestUE3") - && surveyUnitDto.getState().equals(DataState.EDITED) + SurveyUnitModel surveyUnitModelEdited = testSurveyUnitModels.stream().filter(surveyUnitModel1 -> + surveyUnitModel1.getInterrogationId().equals("TestUE3") + && surveyUnitModel1.getState().equals(DataState.EDITED) ).toList().getFirst(); - suDtoEdited.getCollectedVariables().getFirst().values().set(0,"1"); + surveyUnitModelEdited.getCollectedVariables().getFirst().values().set(0,"1"); - suDtoEdited = testSurveyUnitModels.stream().filter(surveyUnitDto -> - surveyUnitDto.getIdUE().equals("TestUE7") - && surveyUnitDto.getState().equals(DataState.EDITED) + surveyUnitModelEdited = testSurveyUnitModels.stream().filter(surveyUnitModel1 -> + surveyUnitModel1.getInterrogationId().equals("TestUE7") + && surveyUnitModel1.getState().equals(DataState.EDITED) ).toList().getFirst(); - suDtoEdited.getCollectedVariables().get(0).values().set(0,"1"); - suDtoEdited.getCollectedVariables().get(1).values().set(0,"1"); + surveyUnitModelEdited.getCollectedVariables().get(0).values().set(0,"1"); + surveyUnitModelEdited.getCollectedVariables().get(1).values().set(0,"1"); //Remove EDITED variable on 8th case - suDtoEdited = testSurveyUnitModels.stream().filter(surveyUnitDto -> - surveyUnitDto.getIdUE().equals("TestUE8") - && surveyUnitDto.getState().equals(DataState.EDITED) + surveyUnitModelEdited = testSurveyUnitModels.stream().filter(surveyUnitModel1 -> + surveyUnitModel1.getInterrogationId().equals("TestUE8") + && surveyUnitModel1.getState().equals(DataState.EDITED) ).toList().getFirst(); - suDtoEdited.getCollectedVariables().remove(1); + surveyUnitModelEdited.getCollectedVariables().remove(1); //When DataVerifier.verifySurveyUnits(testSurveyUnitModels,metadataModel.getVariables()); } - private static void createCase(int variableNumber, int stateNumber, int valueNumber, boolean hasIncorrectValues, boolean hasExternalVariables, String idUE, List testSurveyUnitModels, VariablesMap variablesMap) { + private static void createCase(int variableNumber, int stateNumber, int valueNumber, boolean hasIncorrectValues, boolean hasExternalVariables, String interrogationId, List testSurveyUnitModels, VariablesMap variablesMap) { for(int stateIndex = 0; stateIndex < stateNumber; stateIndex++){ List collectedVariables = new ArrayList<>(); List externalVariables = new ArrayList<>(); @@ -149,14 +148,14 @@ private static void createCase(int variableNumber, int stateNumber, int valueNum } collectedVariables.add(VariableModel.builder() - .idVar("testInteger" + variableIndex) + .varId("testInteger" + variableIndex) .values(new ArrayList<>(values)) .build() ); if(hasExternalVariables){ externalVariables.add(VariableModel.builder() - .idVar("testInteger" + variableIndex) + .varId("testInteger" + variableIndex) .values(new ArrayList<>(values)) .build() ); @@ -164,9 +163,9 @@ private static void createCase(int variableNumber, int stateNumber, int valueNum } SurveyUnitModel surveyUnitModel = SurveyUnitModel.builder() - .idQuest("IdQuest1") - .idCampaign("IdCampaign1") - .idUE(idUE) + .questionnaireId("QuestionnaireId1") + .campaignId("IdCampaign1") + .interrogationId(interrogationId) .state(stateIndex % 2 == 0 ? DataState.COLLECTED : DataState.EDITED) .mode(Mode.WEB) .recordDate(LocalDateTime.now()) @@ -183,96 +182,97 @@ private static void createCase(int variableNumber, int stateNumber, int valueNum //Then //Assertions - private void assertForcedExistence(List testSurveyUnitModels, String idUE, boolean hasToExist) { + private void assertForcedExistence(List testSurveyUnitModels, String interrogationId, boolean hasToExist) { if(hasToExist) assertThat(testSurveyUnitModels).filteredOn(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED) .hasSize(1); else assertThat(testSurveyUnitModels).filteredOn(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED) .isEmpty(); } - private void assertCollectedVariableContent(List testSurveyUnitModels, String idUE, String variableName, int valueIndex, String expectedContent) { - assertForcedExistence(testSurveyUnitModels,idUE,true); + private void assertCollectedVariableContent(List testSurveyUnitModels, String interrogationId, String variableName, int valueIndex, String expectedContent) { + assertForcedExistence(testSurveyUnitModels,interrogationId,true); - Optional suDtoOpt = testSurveyUnitModels.stream().filter(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + Optional surveyUnitModelOpt = testSurveyUnitModels.stream().filter(surveyUnit -> + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED).findFirst(); - assertThat(suDtoOpt).isPresent(); + assertThat(surveyUnitModelOpt).isPresent(); - SurveyUnitModel suDto = suDtoOpt.get(); + SurveyUnitModel surveyUnitModel = surveyUnitModelOpt.get(); - assertThat(suDto.getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals(variableName) )).isNotEmpty(); - assertThat(suDto.getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals(variableName) ).findFirst()).isPresent(); - assertThat(suDto.getCollectedVariables().stream().filter(collectedVariableDto -> - collectedVariableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getCollectedVariables().stream().filter(collectedVariableDto -> + collectedVariableDto.varId().equals(variableName) ).findFirst().get().values().get(valueIndex)).isEqualTo(expectedContent); } - private void assertForcedCollectedVariableExistence(List testSurveyUnitModels, String idUE, String variableName, boolean hasToExist) { - assertForcedExistence(testSurveyUnitModels,idUE, true); - Optional suDtoOpt = testSurveyUnitModels.stream().filter(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + private void assertForcedCollectedVariableExistence(List testSurveyUnitModels, String interrogationId, String variableName, boolean hasToExist) { + assertForcedExistence(testSurveyUnitModels,interrogationId, true); + Optional surveyUnitModelOpt = testSurveyUnitModels.stream().filter(surveyUnit -> + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED).findFirst(); - assertThat(suDtoOpt).isPresent(); + assertThat(surveyUnitModelOpt).isPresent(); - SurveyUnitModel suDto = suDtoOpt.get(); + SurveyUnitModel surveyUnitModel = surveyUnitModelOpt.get(); - if(hasToExist) - assertThat(suDto.getCollectedVariables().stream().filter(collectedVariableDto -> collectedVariableDto.idVar().equals(variableName)).toList()).isNotEmpty(); - else - assertThat(suDto.getCollectedVariables().stream().filter(collectedVariableDto -> collectedVariableDto.idVar().equals(variableName)).toList()).isEmpty(); + if(hasToExist) { + assertThat(surveyUnitModel.getCollectedVariables().stream().filter(collectedVariableDto -> collectedVariableDto.varId().equals(variableName)).toList()).isNotEmpty(); + }else { + assertThat(surveyUnitModel.getCollectedVariables().stream().filter(collectedVariableDto -> collectedVariableDto.varId().equals(variableName)).toList()).isEmpty(); + } } - private void assertForcedExternalVariableExistence(List testSurveyUnitModels, String idUE, String variableName, boolean hasToExist) { - assertForcedExistence(testSurveyUnitModels,idUE, true); - Optional suDtoOpt = testSurveyUnitModels.stream().filter(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + private void assertForcedExternalVariableExistence(List testSurveyUnitModels, String interrogationId, String variableName, boolean hasToExist) { + assertForcedExistence(testSurveyUnitModels,interrogationId, true); + Optional surveyUnitModelOpt = testSurveyUnitModels.stream().filter(surveyUnit -> + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED).findFirst(); - assertThat(suDtoOpt).isPresent(); + assertThat(surveyUnitModelOpt).isPresent(); - SurveyUnitModel suDto = suDtoOpt.get(); + SurveyUnitModel surveyUnitModel = surveyUnitModelOpt.get(); if(hasToExist) - assertThat(suDto.getExternalVariables().stream().filter(variableDto -> variableDto.idVar().equals(variableName)).toList()).isNotEmpty(); + assertThat(surveyUnitModel.getExternalVariables().stream().filter(variableModel -> variableModel.varId().equals(variableName)).toList()).isNotEmpty(); else - assertThat(suDto.getExternalVariables().stream().filter(variableDto -> variableDto.idVar().equals(variableName)).toList()).isEmpty(); + assertThat(surveyUnitModel.getExternalVariables().stream().filter(variableModel -> variableModel.varId().equals(variableName)).toList()).isEmpty(); } - private void assertExternalVariableContent(List testSurveyUnitModels, String idUE, String variableName, int valueIndex, String expectedContent) { - assertForcedExistence(testSurveyUnitModels,idUE,true); + private void assertExternalVariableContent(List testSurveyUnitModels, String interrogationId, String variableName, int valueIndex, String expectedContent) { + assertForcedExistence(testSurveyUnitModels,interrogationId,true); - Optional suDtoOpt = testSurveyUnitModels.stream().filter(surveyUnit -> - surveyUnit.getIdUE().equals(idUE) + Optional surveyUnitModelOpt = testSurveyUnitModels.stream().filter(surveyUnit -> + surveyUnit.getInterrogationId().equals(interrogationId) && surveyUnit.getState() == DataState.FORCED).findFirst(); - assertThat(suDtoOpt).isPresent(); + assertThat(surveyUnitModelOpt).isPresent(); - SurveyUnitModel suDto = suDtoOpt.get(); + SurveyUnitModel surveyUnitModel = surveyUnitModelOpt.get(); - assertThat(suDto.getExternalVariables().stream().filter(variableDto -> - variableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getExternalVariables().stream().filter(variableModel -> + variableModel.varId().equals(variableName) )).isNotEmpty(); - assertThat(suDto.getExternalVariables().stream().filter(variableDto -> - variableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getExternalVariables().stream().filter(variableModel -> + variableModel.varId().equals(variableName) ).findFirst()).isPresent(); - assertThat(suDto.getExternalVariables().stream().filter(variableDto -> - variableDto.idVar().equals(variableName) + assertThat(surveyUnitModel.getExternalVariables().stream().filter(variableModel -> + variableModel.varId().equals(variableName) ).findFirst().get().values().get(valueIndex)).isEqualTo(expectedContent); } diff --git a/src/test/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModelTest.java b/src/test/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModelTest.java index 6daa991f..4ed67048 100644 --- a/src/test/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModelTest.java +++ b/src/test/java/fr/insee/genesis/domain/model/surveyunit/SurveyUnitModelTest.java @@ -14,26 +14,26 @@ public class SurveyUnitModelTest { public void toJSONTest() throws JsonProcessingException { List externalVariableList = new ArrayList<>(); VariableModel externalVariable = VariableModel.builder() - .idVar("TESTIDVAREXT") + .varId("TESTIDVAREXT") .values(List.of(new String[]{"V1","V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); externalVariableList.add(externalVariable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1","V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); SurveyUnitModel surveyUnitModel = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") - .idQuest("TESTIDQUEST") - .idUE("TESTIDUE") + .campaignId("TESTCAMPAIGNID") + .questionnaireId("TESTQUESTIONNAIREID") + .interrogationId("TESTINTERROGATIONID") .mode(Mode.WEB) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2000,1,1,0,0,0)) @@ -46,7 +46,7 @@ public void toJSONTest() throws JsonProcessingException { ObjectMapper objectMapper = new ObjectMapper(); objectMapper.findAndRegisterModules(); Assertions.assertEquals( - objectMapper.readTree("{\"idQuest\":\"TESTIDQUEST\",\"idCampaign\":\"TESTIDCAMPAIGN\",\"idUE\":\"TESTIDUE\",\"state\":\"COLLECTED\",\"mode\":\"WEB\",\"recordDate\":\"2000-01-01T12:00\",\"fileDate\":\"2000-01-01T12:00\",\"collectedVariables\":[{\"idVar\":\"TESTIDVAR\",\"values\":[\"V1\",\"V2\"],\"idLoop\":\"TESTIDLOOP\",\"idParent\":\"TESTIDPARENT\"}],\"externalVariables\":[{\"idVar\":\"TESTIDVAREXT\",\"values\":[\"V1\",\"V2\"],\"idLoop\":\"TESTIDLOOP\",\"idParent\":\"TESTIDPARENT\"}],\"modifiedBy\": null}"), + objectMapper.readTree("{\"questionnaireId\":\"TESTQUESTIONNAIREID\",\"campaignId\":\"TESTCAMPAIGNID\",\"interrogationId\":\"TESTINTERROGATIONID\",\"state\":\"COLLECTED\",\"mode\":\"WEB\",\"recordDate\":\"2000-01-01T12:00\",\"fileDate\":\"2000-01-01T12:00\",\"collectedVariables\":[{\"varId\":\"TESTIDVAR\",\"values\":[\"V1\",\"V2\"],\"loopId\":\"TESTIDLOOP\",\"parentId\":\"TESTIDPARENT\"}],\"externalVariables\":[{\"varId\":\"TESTIDVAREXT\",\"values\":[\"V1\",\"V2\"],\"loopId\":\"TESTIDLOOP\",\"parentId\":\"TESTIDPARENT\"}],\"modifiedBy\": null}"), objectMapper.readTree(objectMapper.writeValueAsString(surveyUnitModel)) ); } diff --git a/src/test/java/fr/insee/genesis/domain/model/surveyunit/VariableModelTest.java b/src/test/java/fr/insee/genesis/domain/model/surveyunit/VariableModelTest.java index 37c67adb..12aca1f5 100644 --- a/src/test/java/fr/insee/genesis/domain/model/surveyunit/VariableModelTest.java +++ b/src/test/java/fr/insee/genesis/domain/model/surveyunit/VariableModelTest.java @@ -12,9 +12,9 @@ class VariableModelTest { @Test void toJSONTest() throws JsonProcessingException { VariableModel variable = VariableModel.builder() - .idVar("TESTIDVAR") - .idParent("TESTIDPARENT") - .idLoop("TESTIDLOOP") + .varId("TESTIDVAR") + .parentId("TESTIDPARENT") + .loopId("TESTIDLOOP") .values(new ArrayList<>(List.of(new String[]{"V1", "V2"}))) .build(); @@ -22,6 +22,6 @@ void toJSONTest() throws JsonProcessingException { objectMapper.findAndRegisterModules(); Assertions.assertEquals(objectMapper.readTree(objectMapper.writeValueAsString(variable)), - objectMapper.readTree("{\"values\":[\"V1\",\"V2\"],\"idVar\":\"TESTIDVAR\",\"idLoop\":\"TESTIDLOOP\",\"idParent\":\"TESTIDPARENT\"}")); + objectMapper.readTree("{\"values\":[\"V1\",\"V2\"],\"varId\":\"TESTIDVAR\",\"loopId\":\"TESTIDLOOP\",\"parentId\":\"TESTIDPARENT\"}")); } } \ No newline at end of file diff --git a/src/test/java/fr/insee/genesis/domain/service/SurveyUnitServiceTest.java b/src/test/java/fr/insee/genesis/domain/service/SurveyUnitServiceTest.java index 82fa9cf4..b0171fc2 100644 --- a/src/test/java/fr/insee/genesis/domain/service/SurveyUnitServiceTest.java +++ b/src/test/java/fr/insee/genesis/domain/service/SurveyUnitServiceTest.java @@ -28,8 +28,8 @@ class SurveyUnitServiceTest { static SurveyUnitPersistencePortStub surveyUnitPersistencePortStub; //Constants - static final String DEFAULT_ID_UE = "TESTIDUE"; - static final String DEFAULT_ID_QUEST = "TESTIDQUESTIONNAIRE"; + static final String DEFAULT_INTERROGATION_ID = "TESTINTERROGATIONID"; + static final String DEFAULT_QUESTIONNAIRE_ID = "TESTQUESTIONNAIREID"; @BeforeAll static void init(){ @@ -43,25 +43,25 @@ void reset(){ surveyUnitPersistencePortStub.getMongoStub().clear(); List externalVariableList = new ArrayList<>(); VariableModel variable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); surveyUnitPersistencePortStub.getMongoStub().add(SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(DEFAULT_ID_UE) - .idQuest(DEFAULT_ID_QUEST) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(DEFAULT_QUESTIONNAIRE_ID) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023,1,1,0,0,0)) .recordDate(LocalDateTime.of(2024,1,1,0,0,0)) @@ -78,26 +78,26 @@ void saveAllTest(){ List externalVariableList = new ArrayList<>(); VariableModel externalVariableModel = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(externalVariableModel); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariableModel = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariableModel); newSurveyUnitModelList.add( SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE("TESTIDUE2") - .idQuest(DEFAULT_ID_QUEST) + .interrogationId("TESTINTERROGATIONID2") + .questionnaireId(DEFAULT_QUESTIONNAIRE_ID) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023,1,1,0,0,0)) .recordDate(LocalDateTime.of(2024,1,1,0,0,0)) @@ -109,19 +109,19 @@ void saveAllTest(){ surveyUnitServiceStatic.saveSurveyUnits(newSurveyUnitModelList); Assertions.assertThat(surveyUnitPersistencePortStub.getMongoStub()).filteredOn(surveyUnitModel -> - surveyUnitModel.getIdCampaign().equals("TESTIDCAMPAIGN") + surveyUnitModel.getCampaignId().equals("TESTCAMPAIGNID") && surveyUnitModel.getMode().equals(Mode.WEB) - && surveyUnitModel.getIdUE().equals("TESTIDUE2") - && surveyUnitModel.getIdQuest().equals(DEFAULT_ID_QUEST) + && surveyUnitModel.getInterrogationId().equals("TESTINTERROGATIONID2") + && surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) && surveyUnitModel.getState().equals(DataState.COLLECTED) && surveyUnitModel.getFileDate().equals(LocalDateTime.of(2023,1,1,0,0,0)) && surveyUnitModel.getRecordDate().equals(LocalDateTime.of(2024,1,1,0,0,0)) && !surveyUnitModel.getExternalVariables().stream().filter( - externalVariable -> externalVariable.idVar().equals("TESTIDVAR") + externalVariable -> externalVariable.varId().equals("TESTIDVAR") && externalVariable.values().containsAll(List.of(new String[]{"V1", "V2"})) ).toList().isEmpty() && !surveyUnitModel.getCollectedVariables().stream().filter( - collectedVariable -> collectedVariable.idVar().equals("TESTIDVAR") + collectedVariable -> collectedVariable.varId().equals("TESTIDVAR") && collectedVariable.values().containsAll(List.of(new String[]{"V1", "V2"})) ).toList().isEmpty() ).isNotEmpty(); @@ -129,25 +129,25 @@ void saveAllTest(){ @Test void findByIdsUEAndQuestionnaireTest(){ - Assertions.assertThat(surveyUnitServiceStatic.findByIdsUEAndQuestionnaire(DEFAULT_ID_UE, DEFAULT_ID_QUEST)).filteredOn( - surveyUnitDto -> - surveyUnitDto.getIdUE().equals(DEFAULT_ID_UE) - && surveyUnitDto.getIdQuest().equals(DEFAULT_ID_QUEST) + Assertions.assertThat(surveyUnitServiceStatic.findByIdsInterrogationAndQuestionnaire(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + surveyUnitModel -> + surveyUnitModel.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) + && surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) ).isNotEmpty(); } @Test - void findByIdUETest(){ - Assertions.assertThat(surveyUnitServiceStatic.findByIdUE(DEFAULT_ID_UE)).filteredOn( - surveyUnitDto -> - surveyUnitDto.getIdUE().equals(DEFAULT_ID_UE) + void findByInterrogationIdTest(){ + Assertions.assertThat(surveyUnitServiceStatic.findByInterrogationId(DEFAULT_INTERROGATION_ID)).filteredOn( + surveyUnitModel -> + surveyUnitModel.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) ).isNotEmpty(); } @Test - void findByIdQuestionnaireTest(){ - Assertions.assertThat(surveyUnitServiceStatic.findByIdQuestionnaire(DEFAULT_ID_QUEST)).filteredOn( - surveyUnitDto -> surveyUnitDto.getIdQuest().equals(DEFAULT_ID_QUEST) + void findByQuestionnaireIdTest(){ + Assertions.assertThat(surveyUnitServiceStatic.findByQuestionnaireId(DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + surveyUnitModel -> surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) ).isNotEmpty(); } @@ -155,10 +155,10 @@ void findByIdQuestionnaireTest(){ void findLatestByIdAndByModeTest(){ addAdditionnalDtoToMongoStub(); - Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByIdQuestionnaire(DEFAULT_ID_UE, DEFAULT_ID_QUEST)).filteredOn( - surveyUnitDto -> surveyUnitDto.getIdUE().equals(DEFAULT_ID_UE) - && surveyUnitDto.getIdQuest().equals(DEFAULT_ID_QUEST) - && surveyUnitDto.getFileDate().getMonth().equals(Month.FEBRUARY) + Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByQuestionnaireId(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + surveyUnitModel -> surveyUnitModel.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) + && surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) + && surveyUnitModel.getFileDate().getMonth().equals(Month.FEBRUARY) ).isNotEmpty(); } @@ -172,10 +172,10 @@ void findResponsesByUEAndQuestionnaireTest_null_collectedVariables() { ); surveyUnitPersistencePortStub.getMongoStub().getLast().setCollectedVariables(null); - Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByIdQuestionnaire(DEFAULT_ID_UE, DEFAULT_ID_QUEST)).filteredOn( - surveyUnitDto -> surveyUnitDto.getIdUE().equals(DEFAULT_ID_UE) - && surveyUnitDto.getIdQuest().equals(DEFAULT_ID_QUEST) - && surveyUnitDto.getFileDate().getMonth().equals(Month.FEBRUARY) + Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByQuestionnaireId(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + surveyUnitModel -> surveyUnitModel.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) + && surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) + && surveyUnitModel.getFileDate().getMonth().equals(Month.FEBRUARY) ).isNotEmpty(); } @Test @@ -188,25 +188,25 @@ void findResponsesByUEAndQuestionnaireTest_null_externalVariables() { ); surveyUnitPersistencePortStub.getMongoStub().getLast().setExternalVariables(null); - Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByIdQuestionnaire(DEFAULT_ID_UE, DEFAULT_ID_QUEST)).filteredOn( - surveyUnitDto -> surveyUnitDto.getIdUE().equals(DEFAULT_ID_UE) - && surveyUnitDto.getIdQuest().equals(DEFAULT_ID_QUEST) - && surveyUnitDto.getFileDate().getMonth().equals(Month.FEBRUARY) + Assertions.assertThat(surveyUnitServiceStatic.findLatestByIdAndByQuestionnaireId(DEFAULT_INTERROGATION_ID, DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + surveyUnitModel -> surveyUnitModel.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) + && surveyUnitModel.getQuestionnaireId().equals(DEFAULT_QUESTIONNAIRE_ID) + && surveyUnitModel.getFileDate().getMonth().equals(Month.FEBRUARY) ).isNotEmpty(); } @Test - void findDistinctIdUEsByIdQuestionnaireTest(){ + void findDistinctInterrogationIdsByQuestionnaireIdTest(){ addAdditionnalDtoToMongoStub(); - Assertions.assertThat(surveyUnitServiceStatic.findDistinctIdUEsByIdQuestionnaire(DEFAULT_ID_QUEST)).filteredOn( - surveyUnitId -> surveyUnitId.getIdUE().equals(DEFAULT_ID_UE) + Assertions.assertThat(surveyUnitServiceStatic.findDistinctInterrogationIdsByQuestionnaireId(DEFAULT_QUESTIONNAIRE_ID)).filteredOn( + interrogationId -> interrogationId.getInterrogationId().equals(DEFAULT_INTERROGATION_ID) ).isNotEmpty().hasSize(1); } @Test - void findIdUEsByIdQuestionnaireTest(){ - Assertions.assertThat(surveyUnitServiceStatic.findModesByIdQuestionnaire(DEFAULT_ID_QUEST)).filteredOn( + void findInterrogationIdsByQuestionnaireIdTest(){ + Assertions.assertThat(surveyUnitServiceStatic.findModesByQuestionnaireId(DEFAULT_QUESTIONNAIRE_ID)).filteredOn( mode -> mode.equals(Mode.WEB) ).isNotEmpty(); } @@ -215,17 +215,17 @@ void findIdUEsByIdQuestionnaireTest(){ void getQuestionnairesByCampaignTest() { addAdditionnalDtoToMongoStub("TESTQUESTIONNAIRE2"); - Assertions.assertThat(surveyUnitServiceStatic.findIdQuestionnairesByIdCampaign("TESTIDCAMPAIGN")).isNotEmpty().hasSize(2); + Assertions.assertThat(surveyUnitServiceStatic.findQuestionnaireIdsByCampaignId("TESTCAMPAIGNID")).isNotEmpty().hasSize(2); } @Test void getAllCampaignsTest() { - Assertions.assertThat(surveyUnitServiceStatic.findDistinctIdCampaigns()).contains("TESTIDCAMPAIGN"); + Assertions.assertThat(surveyUnitServiceStatic.findDistinctCampaignIds()).contains("TESTCAMPAIGNID"); } @Test - void findLatestByIdAndByIdQuestionnairePerretTest(){ + void findLatestByIdAndByQuestionnaireIdPerretTest(){ //Given //Recent Collected already in stub //Old Collected @@ -254,14 +254,14 @@ void findLatestByIdAndByIdQuestionnairePerretTest(){ //When - SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByIdQuestionnaire( - DEFAULT_ID_UE, - DEFAULT_ID_QUEST + SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByQuestionnaireId( + DEFAULT_INTERROGATION_ID, + DEFAULT_QUESTIONNAIRE_ID ); //Then - Assertions.assertThat(surveyUnitDto.getSurveyUnitId()).isEqualTo(DEFAULT_ID_UE); + Assertions.assertThat(surveyUnitDto.getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); Assertions.assertThat(surveyUnitDto.getCollectedVariables().getFirst().getVariableName()) .isEqualTo("TESTIDVAR"); @@ -311,7 +311,7 @@ void findLatestByIdAndByIdQuestionnairePerretTest(){ } @Test - void findLatestByIdAndByIdQuestionnairePerretTest_null_collectedVariables(){ + void findLatestByIdAndByQuestionnaireIdPerretTest_null_collectedVariables(){ //Given addAdditionnalDtoToMongoStub(DataState.EDITED, "C NEW E", @@ -323,14 +323,14 @@ void findLatestByIdAndByIdQuestionnairePerretTest_null_collectedVariables(){ //When - SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByIdQuestionnaire( - DEFAULT_ID_UE, - DEFAULT_ID_QUEST + SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByQuestionnaireId( + DEFAULT_INTERROGATION_ID, + DEFAULT_QUESTIONNAIRE_ID ); //Then - Assertions.assertThat(surveyUnitDto.getSurveyUnitId()).isEqualTo(DEFAULT_ID_UE); + Assertions.assertThat(surveyUnitDto.getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); Assertions.assertThat(surveyUnitDto.getCollectedVariables().getFirst().getVariableName()) .isEqualTo("TESTIDVAR"); @@ -370,7 +370,7 @@ void findLatestByIdAndByIdQuestionnairePerretTest_null_collectedVariables(){ } @Test - void findLatestByIdAndByIdQuestionnairePerretTest_null_externalVariables(){ + void findLatestByIdAndByQuestionnaireIdPerretTest_null_externalVariables(){ //Given addAdditionnalDtoToMongoStub(DataState.EDITED, "C NEW E", @@ -382,14 +382,14 @@ void findLatestByIdAndByIdQuestionnairePerretTest_null_externalVariables(){ //When - SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByIdQuestionnaire( - DEFAULT_ID_UE, - DEFAULT_ID_QUEST + SurveyUnitDto surveyUnitDto = surveyUnitServiceStatic.findLatestValuesByStateByIdAndByQuestionnaireId( + DEFAULT_INTERROGATION_ID, + DEFAULT_QUESTIONNAIRE_ID ); //Then - Assertions.assertThat(surveyUnitDto.getSurveyUnitId()).isEqualTo(DEFAULT_ID_UE); + Assertions.assertThat(surveyUnitDto.getInterrogationId()).isEqualTo(DEFAULT_INTERROGATION_ID); Assertions.assertThat(surveyUnitDto.getCollectedVariables().getFirst().getVariableName()) .isEqualTo("TESTIDVAR"); @@ -430,23 +430,23 @@ void findLatestByIdAndByIdQuestionnairePerretTest_null_externalVariables(){ private void addAdditionnalDtoToMongoStub(){ List externalVariableList = new ArrayList<>(); - VariableModel externalVariableModel = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel externalVariableModel = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(externalVariableModel); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariableModel = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariableModel); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(DEFAULT_ID_UE) - .idQuest(DEFAULT_ID_QUEST) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(DEFAULT_QUESTIONNAIRE_ID) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023,2,2,0,0,0)) .recordDate(LocalDateTime.of(2024,2,2,0,0,0)) @@ -456,25 +456,25 @@ private void addAdditionnalDtoToMongoStub(){ surveyUnitPersistencePortStub.getMongoStub().add(recentDTO); } - private void addAdditionnalDtoToMongoStub(String idQuestionnaire) { + private void addAdditionnalDtoToMongoStub(String questionnaireId) { List externalVariableList = new ArrayList<>(); - VariableModel externalVariableModel = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel externalVariableModel = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableList.add(externalVariableModel); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariableModel = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariableModel); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(DEFAULT_ID_UE) - .idQuest(idQuestionnaire) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(questionnaireId) .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023, 2, 2, 0, 0, 0)) .recordDate(LocalDateTime.of(2024, 2, 2, 0, 0, 0)) @@ -490,23 +490,23 @@ private void addAdditionnalDtoToMongoStub(DataState state, LocalDateTime fileDate, LocalDateTime recordDate) { List externalVariableList = new ArrayList<>(); - VariableModel externalVariableModel = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{externalVariableValue})).build(); + VariableModel externalVariableModel = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{externalVariableValue})).build(); externalVariableList.add(externalVariableModel); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{collectedVariableValue})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); SurveyUnitModel recentDTO = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE(DEFAULT_ID_UE) - .idQuest(DEFAULT_ID_QUEST) + .interrogationId(DEFAULT_INTERROGATION_ID) + .questionnaireId(DEFAULT_QUESTIONNAIRE_ID) .state(state) .fileDate(fileDate) .recordDate(recordDate) diff --git a/src/test/java/fr/insee/genesis/domain/utils/DataVerifierTest.java b/src/test/java/fr/insee/genesis/domain/utils/DataVerifierTest.java index 87acbb92..bd4dd3a0 100644 --- a/src/test/java/fr/insee/genesis/domain/utils/DataVerifierTest.java +++ b/src/test/java/fr/insee/genesis/domain/utils/DataVerifierTest.java @@ -36,23 +36,23 @@ void setUp() { // Setup survey units VariableModel collectedVariable1 = VariableModel.builder() - .idVar("var1") + .varId("var1") .values(List.of("123")) - .idLoop("loop1") - .idParent("parent1") + .loopId("loop1") + .parentId("parent1") .build(); VariableModel collectedVariable2 = VariableModel.builder() - .idVar("var2") + .varId("var2") .values(List.of("true")) - .idLoop("loop2") - .idParent("parent2") + .loopId("loop2") + .parentId("parent2") .build(); SurveyUnitModel surveyUnit = SurveyUnitModel.builder() - .idUE("UE1100000001") - .idQuest("Quest1") - .idCampaign("Camp1") + .interrogationId("UE1100000001") + .questionnaireId("Quest1") + .campaignId("Camp1") .state(DataState.COLLECTED) .collectedVariables(List.of(collectedVariable1, collectedVariable2)) .externalVariables(List.of()) @@ -63,13 +63,13 @@ void setUp() { @Test void shouldHandleEmptyValuesList() { - List suDtosList = new ArrayList<>(); + List surveyUnitModelsList = new ArrayList<>(); // When - DataVerifier.verifySurveyUnits(suDtosList, null); + DataVerifier.verifySurveyUnits(surveyUnitModelsList, null); // Then - assertTrue(suDtosList.isEmpty()); // Empty list, nothing invalid, so should return null + assertTrue(surveyUnitModelsList.isEmpty()); // Empty list, nothing invalid, so should return null } @Test @@ -78,21 +78,21 @@ void shouldAddForcedSurveyUnit_WhenInvalidCollectedVariable() { // Add invalid value surveyUnits.clear(); VariableModel collectedVariable1 = VariableModel.builder() - .idVar("var1") + .varId("var1") .values(List.of("invalid")) - .idLoop("loop1") - .idParent("parent1") + .loopId("loop1") + .parentId("parent1") .build(); VariableModel collectedVariable2 = VariableModel.builder() - .idVar("var2") + .varId("var2") .values(List.of("true")) - .idLoop("loop2") - .idParent("parent2") + .loopId("loop2") + .parentId("parent2") .build(); SurveyUnitModel surveyUnit = SurveyUnitModel.builder() - .idUE("UE1100000001") - .idQuest("Quest1") - .idCampaign("Camp1") + .interrogationId("UE1100000001") + .questionnaireId("Quest1") + .campaignId("Camp1") .state(DataState.COLLECTED) .collectedVariables(List.of(collectedVariable1, collectedVariable2)) .externalVariables(List.of()) @@ -120,15 +120,15 @@ void shouldNotAddForcedSurveyUnit_WhenAllVariablesAreValid() { @Test void shouldAddForcedSurveyUnit_WhenInvalidExternalVariable() { //Add surveyUnit with invalid external Variable - VariableModel extVar = VariableModel.builder().idVar("var2").values(List.of( + VariableModel extVar = VariableModel.builder().varId("var2").values(List.of( "notBoolean")).build(); List listVarExt = new ArrayList<>(); listVarExt.add(extVar); SurveyUnitModel surveyUnitWithInvalidExt = SurveyUnitModel.builder() - .idUE("UE1100000002") - .idQuest("Quest1") - .idCampaign("Camp1") + .interrogationId("UE1100000002") + .questionnaireId("Quest1") + .campaignId("Camp1") .state(DataState.COLLECTED) .collectedVariables(List.of()) .externalVariables(listVarExt) @@ -151,21 +151,21 @@ void shouldCorrectInvalidValuesInForcedSurveyUnit() { // ADD invalid values surveyUnits.clear(); VariableModel collectedVariable1 = VariableModel.builder() - .idVar("var1") + .varId("var1") .values(List.of("invalid", "456")) - .idLoop("loop1") - .idParent("parent1") + .loopId("loop1") + .parentId("parent1") .build(); VariableModel collectedVariable2 = VariableModel.builder() - .idVar("var2") + .varId("var2") .values(List.of("false")) - .idLoop("loop2") - .idParent("parent2") + .loopId("loop2") + .parentId("parent2") .build(); SurveyUnitModel surveyUnit = SurveyUnitModel.builder() - .idUE("UE1100000001") - .idQuest("Quest1") - .idCampaign("Camp1") + .interrogationId("UE1100000001") + .questionnaireId("Quest1") + .campaignId("Camp1") .state(DataState.COLLECTED) .collectedVariables(List.of(collectedVariable1, collectedVariable2)) .externalVariables(List.of()) diff --git a/src/test/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitModelMongoAdapterTest.java b/src/test/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitModelMongoAdapterTest.java index 9d65034f..0eec87b0 100644 --- a/src/test/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitModelMongoAdapterTest.java +++ b/src/test/java/fr/insee/genesis/infrastructure/adapter/SurveyUnitModelMongoAdapterTest.java @@ -38,9 +38,9 @@ class SurveyUnitModelMongoAdapterTest { static void setUp() { surveyUnitMongoAdapter = new SurveyUnitMongoAdapter(mongoRepository, null); suDoc= new SurveyUnitDocument(); - suDoc.setIdCampaign("idCampaign"); - suDoc.setIdUE("UE1100000001"); - suDoc.setIdQuestionnaire("TEST2023X01"); + suDoc.setCampaignId("campaignId"); + suDoc.setInterrogationId("UE1100000001"); + suDoc.setQuestionnaireId("TEST2023X01"); suDoc.setState("COLLECTED"); suDoc.setMode("WEB"); suDoc.setRecordDate(LocalDateTime.now()); @@ -48,9 +48,9 @@ static void setUp() { suDoc.setExternalVariables(List.of(new VariableDocument())); suDoc2= new SurveyUnitDocument(); - suDoc2.setIdCampaign("idCampaign"); - suDoc2.setIdUE("UE1100000001"); - suDoc2.setIdQuestionnaire("TEST2023X01"); + suDoc2.setCampaignId("campaignId"); + suDoc2.setInterrogationId("UE1100000001"); + suDoc2.setQuestionnaireId("TEST2023X01"); suDoc2.setState("COLLECTED"); suDoc2.setMode("TEL"); suDoc2.setRecordDate(LocalDateTime.now()); @@ -58,9 +58,9 @@ static void setUp() { suDoc2.setExternalVariables(List.of(new VariableDocument())); suDoc3= new SurveyUnitDocument(); - suDoc3.setIdCampaign("idCampaign"); - suDoc3.setIdUE("UE1100000002"); - suDoc3.setIdQuestionnaire("TEST2023X01"); + suDoc3.setCampaignId("campaignId"); + suDoc3.setInterrogationId("UE1100000002"); + suDoc3.setQuestionnaireId("TEST2023X01"); suDoc3.setState("COLLECTED"); suDoc3.setMode("WEB"); suDoc3.setRecordDate(LocalDateTime.now()); @@ -74,7 +74,7 @@ void shouldReturnListOfSurveyUnitDto_IfIdsFoundInDataBase() { List responses = new ArrayList<>(); responses.add(suDoc); responses.add(suDoc2); - when(mongoRepository.findByIdUEAndIdQuestionnaire(any(String.class), any(String.class))).thenReturn(responses); + when(mongoRepository.findByInterrogationIdAndQuestionnaireId(any(String.class), any(String.class))).thenReturn(responses); // When List updates = surveyUnitMongoAdapter.findByIds("UE1100000001", "TEST2023X01"); // Then @@ -85,7 +85,7 @@ void shouldReturnListOfSurveyUnitDto_IfIdsFoundInDataBase() { @Test void shouldReturnEmptyList_IfIdsNotFoundInDataBase() { //Given - when(mongoRepository.findByIdUEAndIdQuestionnaire(any(String.class), any(String.class))).thenReturn(List.of()); + when(mongoRepository.findByInterrogationIdAndQuestionnaireId(any(String.class), any(String.class))).thenReturn(List.of()); // When List updates = surveyUnitMongoAdapter.findByIds("UE1100000001", "TEST2023X01"); // Then @@ -93,83 +93,83 @@ void shouldReturnEmptyList_IfIdsNotFoundInDataBase() { } @Test - void shouldReturnListOfSurveyUnitDto_IfIdUEFoundInDataBase() { + void shouldReturnListOfSurveyUnitDto_IfInterrogationIdFoundInDataBase() { //Given List responses = new ArrayList<>(); responses.add(suDoc); responses.add(suDoc2); - when(mongoRepository.findByIdUE(any(String.class))).thenReturn(responses); + when(mongoRepository.findByInterrogationId(any(String.class))).thenReturn(responses); // When - List updates = surveyUnitMongoAdapter.findByIdUE("UE1100000001"); + List updates = surveyUnitMongoAdapter.findByInterrogationId("UE1100000001"); // Then Assertions.assertThat(updates).isNotNull().hasSize(2); Assertions.assertThat(updates.getFirst().getMode()).isEqualTo(Mode.WEB); } @Test - void shouldReturnEmptyList_IfIdUENotFoundInDataBase() { + void shouldReturnEmptyList_IfInterrogationIdNotFoundInDataBase() { //Given - when(mongoRepository.findByIdUE(any(String.class))).thenReturn(List.of()); + when(mongoRepository.findByInterrogationId(any(String.class))).thenReturn(List.of()); // When - List updates = surveyUnitMongoAdapter.findByIdUE("UE1100000001"); + List updates = surveyUnitMongoAdapter.findByInterrogationId("UE1100000001"); // Then Assertions.assertThat(updates).isEmpty(); } @Test - void shouldReturnListOfSurveyUnitDto_IfIdQuestionnaireFoundInDataBase() { + void shouldReturnListOfSurveyUnitDto_IfQuestionnaireIdFoundInDataBase() { //Given List responses = new ArrayList<>(); responses.add(suDoc); responses.add(suDoc2); responses.add(suDoc3); - when(mongoRepository.findByIdQuestionnaire(any(String.class))).thenReturn(responses.stream()); + when(mongoRepository.findByQuestionnaireId(any(String.class))).thenReturn(responses.stream()); // When - Stream updates = surveyUnitMongoAdapter.findByIdQuestionnaire("TEST2023X01"); + Stream updates = surveyUnitMongoAdapter.findByQuestionnaireId("TEST2023X01"); // Then Assertions.assertThat(updates).isNotNull().hasSize(3); //Assertions.assertThat(updates.get(2).getMode()).isEqualTo(Mode.WEB); } @Test - void shouldReturnEmptyList_IfIdQuestionnaireNotFoundInDataBase() { + void shouldReturnEmptyList_IfQuestionnaireIdNotFoundInDataBase() { //Given - when(mongoRepository.findByIdQuestionnaire(any(String.class))).thenReturn(Stream.empty()); + when(mongoRepository.findByQuestionnaireId(any(String.class))).thenReturn(Stream.empty()); // When - List updates = surveyUnitMongoAdapter.findByIdQuestionnaire("TEST2023X01").toList(); + List updates = surveyUnitMongoAdapter.findByQuestionnaireId("TEST2023X01").toList(); // Then Assertions.assertThat(updates).isEmpty(); } @Test - void shouldReturnListOfSurveyUnitDto_WhenGivenAListOfIdUEs() { + void shouldReturnListOfSurveyUnitDto_WhenGivenAListOfInterrogationIds() { //Given List responses1 = new ArrayList<>(); responses1.add(suDoc); responses1.add(suDoc2); List responses2 = new ArrayList<>(); responses2.add(suDoc3); - when(mongoRepository.findByIdUEAndIdQuestionnaire("UE1100000001", "TEST2023X01")).thenReturn(responses1); - when(mongoRepository.findByIdUEAndIdQuestionnaire("UE1100000002", "TEST2023X01")).thenReturn(responses2); - SurveyUnitModel id1 = SurveyUnitModel.builder().idUE("UE1100000001").build(); - SurveyUnitModel id2 = SurveyUnitModel.builder().idUE("UE1100000002").build(); + when(mongoRepository.findByInterrogationIdAndQuestionnaireId("UE1100000001", "TEST2023X01")).thenReturn(responses1); + when(mongoRepository.findByInterrogationIdAndQuestionnaireId("UE1100000002", "TEST2023X01")).thenReturn(responses2); + SurveyUnitModel id1 = SurveyUnitModel.builder().interrogationId("UE1100000001").build(); + SurveyUnitModel id2 = SurveyUnitModel.builder().interrogationId("UE1100000002").build(); List ids = List.of(id1, id2); // When - List updates = surveyUnitMongoAdapter.findByIdUEsAndIdQuestionnaire(ids, "TEST2023X01"); + List updates = surveyUnitMongoAdapter.findByInterrogationIdsAndQuestionnaireId(ids, "TEST2023X01"); // Then Assertions.assertThat(updates).isNotNull().hasSize(3); Assertions.assertThat(updates.getFirst().getMode()).isEqualTo(Mode.WEB); } @Test - void shouldReturnEmptyList_IfIdUEsNotFoundInDataBase() { + void shouldReturnEmptyList_IfInterrogationIdsNotFoundInDataBase() { //Given - when(mongoRepository.findByIdUEAndIdQuestionnaire(any(String.class),any(String.class))).thenReturn(List.of()); - SurveyUnitModel id1 = SurveyUnitModel.builder().idUE("UE1100000001").build(); - SurveyUnitModel id2 = SurveyUnitModel.builder().idUE("UE1100000002").build(); + when(mongoRepository.findByInterrogationIdAndQuestionnaireId(any(String.class),any(String.class))).thenReturn(List.of()); + SurveyUnitModel id1 = SurveyUnitModel.builder().interrogationId("UE1100000001").build(); + SurveyUnitModel id2 = SurveyUnitModel.builder().interrogationId("UE1100000002").build(); List ids = List.of(id1, id2); // When - List updates = surveyUnitMongoAdapter.findByIdUEsAndIdQuestionnaire(ids, "TEST2023X01"); + List updates = surveyUnitMongoAdapter.findByInterrogationIdsAndQuestionnaireId(ids, "TEST2023X01"); // Then Assertions.assertThat(updates).isEmpty(); } diff --git a/src/test/java/fr/insee/genesis/infrastructure/mapper/ScheduleDocumentMapperImplTest.java b/src/test/java/fr/insee/genesis/infrastructure/mapper/ScheduleDocumentMapperImplTest.java index da2b7ba6..66213a20 100644 --- a/src/test/java/fr/insee/genesis/infrastructure/mapper/ScheduleDocumentMapperImplTest.java +++ b/src/test/java/fr/insee/genesis/infrastructure/mapper/ScheduleDocumentMapperImplTest.java @@ -28,11 +28,11 @@ static void init(){ scheduleDocumentMapperImplStatic = new ScheduleDocumentMapperImpl(); scheduleDocument = new ScheduleDocument(); - scheduleDocument.setSurveyName("TESTIDCAMPAIGN"); + scheduleDocument.setSurveyName("TESTCAMPAIGNID"); scheduleDocument.setKraftwerkExecutionScheduleList(new ArrayList<>()); scheduleModel = ScheduleModel.builder() - .surveyName("TESTIDCAMPAIGN") + .surveyName("TESTCAMPAIGNID") .kraftwerkExecutionScheduleList(new ArrayList<>()) .build(); @@ -63,7 +63,7 @@ void shouldReturnNull(){ void shouldReturnDocumentDtoFromDocument(){ ScheduleModel scheduleModel1 = scheduleDocumentMapperImplStatic.documentToModel(scheduleDocument); - Assertions.assertThat(scheduleModel1.getSurveyName()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(scheduleModel1.getSurveyName()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(scheduleModel1.getLastExecution()).isNull(); Assertions.assertThat(scheduleModel1.getKraftwerkExecutionScheduleList()).hasSize(1); @@ -77,7 +77,7 @@ void shouldReturnDocumentDtoFromDocument(){ void shouldReturnDocumentFromDocumentDto(){ ScheduleDocument scheduleDocument1 = scheduleDocumentMapperImplStatic.modelToDocument(scheduleModel); - Assertions.assertThat(scheduleDocument1.getSurveyName()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(scheduleDocument1.getSurveyName()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(scheduleDocument1.getLastExecution()).isNull(); Assertions.assertThat(scheduleDocument1.getKraftwerkExecutionScheduleList()).hasSize(1); @@ -95,7 +95,7 @@ void shouldReturnDocumentLDtoListFromDocumentList(){ List scheduleModelList = scheduleDocumentMapperImplStatic.listDocumentToListModel(scheduleDocumentList); - Assertions.assertThat(scheduleModelList.getFirst().getSurveyName()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(scheduleModelList.getFirst().getSurveyName()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(scheduleModelList.getFirst().getLastExecution()).isNull(); Assertions.assertThat(scheduleModelList.getFirst().getKraftwerkExecutionScheduleList()).hasSize(1); @@ -116,7 +116,7 @@ void shouldReturnDocumentListFromDocumentDtoList(){ List scheduleDocumentList = scheduleDocumentMapperImplStatic.listModelToListDocument(scheduleModelList); - Assertions.assertThat(scheduleDocumentList.getFirst().getSurveyName()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(scheduleDocumentList.getFirst().getSurveyName()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(scheduleDocumentList.getFirst().getLastExecution()).isNull(); Assertions.assertThat(scheduleDocumentList.getFirst().getKraftwerkExecutionScheduleList()).hasSize(1); diff --git a/src/test/java/fr/insee/genesis/infrastructure/mapper/SurveyUnitDocumentMapperImplTest.java b/src/test/java/fr/insee/genesis/infrastructure/mapper/SurveyUnitDocumentMapperImplTest.java index 70d0ce0d..f82fbda9 100644 --- a/src/test/java/fr/insee/genesis/infrastructure/mapper/SurveyUnitDocumentMapperImplTest.java +++ b/src/test/java/fr/insee/genesis/infrastructure/mapper/SurveyUnitDocumentMapperImplTest.java @@ -30,46 +30,46 @@ static void init(){ surveyUnitDocumentMapperImplStatic = new SurveyUnitDocumentMapperImpl(); surveyUnitDocumentStatic = new SurveyUnitDocument(); - surveyUnitDocumentStatic.setIdCampaign("TESTIDCAMPAIGN"); + surveyUnitDocumentStatic.setCampaignId("TESTCAMPAIGNID"); surveyUnitDocumentStatic.setMode("WEB"); - surveyUnitDocumentStatic.setIdUE("TESTIDUE"); - surveyUnitDocumentStatic.setIdQuestionnaire("TESTIDQUESTIONNAIRE"); + surveyUnitDocumentStatic.setInterrogationId("TESTINTERROGATIONID"); + surveyUnitDocumentStatic.setQuestionnaireId("TESTQUESTIONNAIREID"); surveyUnitDocumentStatic.setState("COLLECTED"); surveyUnitDocumentStatic.setFileDate(LocalDateTime.of(2023,1,1,0,0,0)); List documentExternalVariableList = new ArrayList<>(); VariableDocument externalVariable = new VariableDocument(); - externalVariable.setIdVar("TESTIDVAR"); + externalVariable.setVarId("TESTIDVAR"); externalVariable.setValues(List.of(new String[]{"V1", "V2"})); documentExternalVariableList.add(externalVariable); surveyUnitDocumentStatic.setExternalVariables(documentExternalVariableList); List documentCollectedVariableList = new ArrayList<>(); VariableDocument variableDocument = new VariableDocument(); - variableDocument.setIdVar("TESTIDVAR"); + variableDocument.setVarId("TESTIDVAR"); variableDocument.setValues(List.of(new String[]{"V1", "V2"})); documentCollectedVariableList.add(variableDocument); surveyUnitDocumentStatic.setCollectedVariables(documentCollectedVariableList); List externalVariableDtoList = new ArrayList<>(); - VariableModel variable = VariableModel.builder().idVar("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); + VariableModel variable = VariableModel.builder().varId("TESTIDVAR").values(List.of(new String[]{"V1", "V2"})).build(); externalVariableDtoList.add(variable); List collectedVariableList = new ArrayList<>(); VariableModel collectedVariable = VariableModel.builder() - .idVar("TESTIDVAR") + .varId("TESTIDVAR") .values(List.of(new String[]{"V1", "V2"})) - .idLoop("TESTIDLOOP") - .idParent("TESTIDPARENT") + .loopId("TESTIDLOOP") + .parentId("TESTIDPARENT") .build(); collectedVariableList.add(collectedVariable); surveyUnitStatic = SurveyUnitModel.builder() - .idCampaign("TESTIDCAMPAIGN") + .campaignId("TESTCAMPAIGNID") .mode(Mode.WEB) - .idUE("TESTIDUE") - .idQuest("TESTIDQUESTIONNAIRE") + .interrogationId("TESTINTERROGATIONID") + .questionnaireId("TESTQUESTIONNAIREID") .state(DataState.COLLECTED) .fileDate(LocalDateTime.of(2023,1,1,0,0,0)) .recordDate(LocalDateTime.of(2024,1,1,0,0,0)) @@ -94,20 +94,20 @@ void shouldReturnNull(){ void shouldReturnDocumentDtoFromDocument(){ SurveyUnitModel surveyUnit = surveyUnitDocumentMapperImplStatic.documentToModel(surveyUnitDocumentStatic); - Assertions.assertThat(surveyUnit.getIdCampaign()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(surveyUnit.getCampaignId()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(surveyUnit.getMode()).isEqualTo(Mode.WEB); - Assertions.assertThat(surveyUnit.getIdUE()).isEqualTo("TESTIDUE"); - Assertions.assertThat(surveyUnit.getIdQuest()).isEqualTo("TESTIDQUESTIONNAIRE"); + Assertions.assertThat(surveyUnit.getInterrogationId()).isEqualTo("TESTINTERROGATIONID"); + Assertions.assertThat(surveyUnit.getQuestionnaireId()).isEqualTo("TESTQUESTIONNAIREID"); Assertions.assertThat(surveyUnit.getState()).isEqualTo(DataState.COLLECTED); Assertions.assertThat(surveyUnit.getFileDate()).isEqualTo(LocalDateTime.of(2023,1,1,0,0,0)); Assertions.assertThat(surveyUnit.getExternalVariables()).filteredOn(externalVariableDto -> - externalVariableDto.idVar().equals("TESTIDVAR") + externalVariableDto.varId().equals("TESTIDVAR") && externalVariableDto.values().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); Assertions.assertThat(surveyUnit.getCollectedVariables()).filteredOn(variableStateDto -> - variableStateDto.idVar().equals("TESTIDVAR") + variableStateDto.varId().equals("TESTIDVAR") && variableStateDto.values().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); @@ -118,20 +118,20 @@ void shouldReturnDocumentDtoFromDocument(){ void shouldReturnDocumentFromDocumentDto(){ SurveyUnitDocument surveyUnitDocument = surveyUnitDocumentMapperImplStatic.modelToDocument(surveyUnitStatic); - Assertions.assertThat(surveyUnitDocument.getIdCampaign()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(surveyUnitDocument.getCampaignId()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(surveyUnitDocument.getMode()).isEqualTo("WEB"); - Assertions.assertThat(surveyUnitDocument.getIdUE()).isEqualTo("TESTIDUE"); - Assertions.assertThat(surveyUnitDocument.getIdQuestionnaire()).isEqualTo("TESTIDQUESTIONNAIRE"); + Assertions.assertThat(surveyUnitDocument.getInterrogationId()).isEqualTo("TESTINTERROGATIONID"); + Assertions.assertThat(surveyUnitDocument.getQuestionnaireId()).isEqualTo("TESTQUESTIONNAIREID"); Assertions.assertThat(surveyUnitDocument.getState()).isEqualTo("COLLECTED"); Assertions.assertThat(surveyUnitDocument.getFileDate()).isEqualTo(LocalDateTime.of(2023,1,1,0,0,0)); Assertions.assertThat(surveyUnitDocument.getExternalVariables()).filteredOn(externalVariableDto -> - externalVariableDto.getIdVar().equals("TESTIDVAR") + externalVariableDto.getVarId().equals("TESTIDVAR") && externalVariableDto.getValues().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); Assertions.assertThat(surveyUnitDocument.getCollectedVariables()).filteredOn(variableDocument -> - variableDocument.getIdVar().equals("TESTIDVAR") + variableDocument.getVarId().equals("TESTIDVAR") && variableDocument.getValues().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); @@ -146,20 +146,20 @@ void shouldReturnDocumentLDtoListFromDocumentList(){ List surveyUnitList = surveyUnitDocumentMapperImplStatic.listDocumentToListModel(surveyUnitDocumentList); - Assertions.assertThat(surveyUnitList.getFirst().getIdCampaign()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(surveyUnitList.getFirst().getCampaignId()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(surveyUnitList.getFirst().getMode()).isEqualTo(Mode.WEB); - Assertions.assertThat(surveyUnitList.getFirst().getIdUE()).isEqualTo("TESTIDUE"); - Assertions.assertThat(surveyUnitList.getFirst().getIdQuest()).isEqualTo("TESTIDQUESTIONNAIRE"); + Assertions.assertThat(surveyUnitList.getFirst().getInterrogationId()).isEqualTo("TESTINTERROGATIONID"); + Assertions.assertThat(surveyUnitList.getFirst().getQuestionnaireId()).isEqualTo("TESTQUESTIONNAIREID"); Assertions.assertThat(surveyUnitList.getFirst().getState()).isEqualTo(DataState.COLLECTED); Assertions.assertThat(surveyUnitList.getFirst().getFileDate()).isEqualTo(LocalDateTime.of(2023,1,1,0,0,0)); Assertions.assertThat(surveyUnitList.getFirst().getExternalVariables()).filteredOn(externalVariableDto -> - externalVariableDto.idVar().equals("TESTIDVAR") + externalVariableDto.varId().equals("TESTIDVAR") && externalVariableDto.values().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); Assertions.assertThat(surveyUnitList.getFirst().getCollectedVariables()).filteredOn(variableStateDto -> - variableStateDto.idVar().equals("TESTIDVAR") + variableStateDto.varId().equals("TESTIDVAR") && variableStateDto.values().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); } @@ -172,20 +172,20 @@ void shouldReturnDocumentListFromDocumentDtoList(){ List surveyUnitDocumentList = surveyUnitDocumentMapperImplStatic.listModelToListDocument(surveyUnitList); - Assertions.assertThat(surveyUnitDocumentList.getFirst().getIdCampaign()).isEqualTo("TESTIDCAMPAIGN"); + Assertions.assertThat(surveyUnitDocumentList.getFirst().getCampaignId()).isEqualTo("TESTCAMPAIGNID"); Assertions.assertThat(surveyUnitDocumentList.getFirst().getMode()).isEqualTo("WEB"); - Assertions.assertThat(surveyUnitDocumentList.getFirst().getIdUE()).isEqualTo("TESTIDUE"); - Assertions.assertThat(surveyUnitDocumentList.getFirst().getIdQuestionnaire()).isEqualTo("TESTIDQUESTIONNAIRE"); + Assertions.assertThat(surveyUnitDocumentList.getFirst().getInterrogationId()).isEqualTo("TESTINTERROGATIONID"); + Assertions.assertThat(surveyUnitDocumentList.getFirst().getQuestionnaireId()).isEqualTo("TESTQUESTIONNAIREID"); Assertions.assertThat(surveyUnitDocumentList.getFirst().getState()).isEqualTo("COLLECTED"); Assertions.assertThat(surveyUnitDocumentList.getFirst().getFileDate()).isEqualTo(LocalDateTime.of(2023,1,1,0,0,0)); Assertions.assertThat(surveyUnitDocumentList.getFirst().getExternalVariables()).filteredOn(externalVariableDto -> - externalVariableDto.getIdVar().equals("TESTIDVAR") + externalVariableDto.getVarId().equals("TESTIDVAR") && externalVariableDto.getValues().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); Assertions.assertThat(surveyUnitDocumentList.getFirst().getCollectedVariables()).filteredOn(variableDocument -> - variableDocument.getIdVar().equals("TESTIDVAR") + variableDocument.getVarId().equals("TESTIDVAR") && variableDocument.getValues().containsAll(List.of(new String[]{"V1", "V2"})) ).isNotEmpty(); } diff --git a/src/test/java/fr/insee/genesis/stubs/SurveyUnitPersistencePortStub.java b/src/test/java/fr/insee/genesis/stubs/SurveyUnitPersistencePortStub.java index b5852aa9..67a9e042 100644 --- a/src/test/java/fr/insee/genesis/stubs/SurveyUnitPersistencePortStub.java +++ b/src/test/java/fr/insee/genesis/stubs/SurveyUnitPersistencePortStub.java @@ -20,10 +20,10 @@ public void saveAll(List suList) { } @Override - public List findByIds(String idUE, String idQuest) { + public List findByIds(String interrogationId, String questionnaireId) { List surveyUnitModelList = new ArrayList<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdUE().equals(idUE) && SurveyUnitModel.getIdQuest().equals(idQuest)) + if(SurveyUnitModel.getInterrogationId().equals(interrogationId) && SurveyUnitModel.getQuestionnaireId().equals(questionnaireId)) surveyUnitModelList.add(SurveyUnitModel); } @@ -31,10 +31,10 @@ public List findByIds(String idUE, String idQuest) { } @Override - public List findByIdUE(String idUE) { + public List findByInterrogationId(String interrogationId) { List surveyUnitModelList = new ArrayList<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdUE().equals(idUE)) + if(SurveyUnitModel.getInterrogationId().equals(interrogationId)) surveyUnitModelList.add(SurveyUnitModel); } @@ -42,11 +42,11 @@ public List findByIdUE(String idUE) { } @Override - public List findByIdUEsAndIdQuestionnaire(List idUEs, String idQuestionnaire) { + public List findByInterrogationIdsAndQuestionnaireId(List interrogationIds, String questionnaireId) { List surveyUnitModelList = new ArrayList<>(); - for(SurveyUnitModel surveyUnitModel : idUEs) { + for(SurveyUnitModel surveyUnitModel : interrogationIds) { for (SurveyUnitModel document : mongoStub) { - if (surveyUnitModel.getIdUE().equals(document.getIdUE()) && document.getIdQuest().equals(idQuestionnaire)) + if (surveyUnitModel.getInterrogationId().equals(document.getInterrogationId()) && document.getQuestionnaireId().equals(questionnaireId)) surveyUnitModelList.add(document); } } @@ -55,10 +55,10 @@ public List findByIdUEsAndIdQuestionnaire(List } @Override - public Stream findByIdQuestionnaire(String idQuestionnaire) { + public Stream findByQuestionnaireId(String questionnaireId) { List surveyUnitModelList = new ArrayList<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdQuest().equals(idQuestionnaire)) + if(SurveyUnitModel.getQuestionnaireId().equals(questionnaireId)) surveyUnitModelList.add(SurveyUnitModel); } @@ -66,12 +66,12 @@ public Stream findByIdQuestionnaire(String idQuestionnaire) { } @Override - public List findIdUEsByIdQuestionnaire(String idQuestionnaire) { + public List findInterrogationIdsByQuestionnaireId(String questionnaireId) { List surveyUnitModelList = new ArrayList<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdQuest().equals(idQuestionnaire)) + if(SurveyUnitModel.getQuestionnaireId().equals(questionnaireId)) surveyUnitModelList.add( - new SurveyUnitModel(SurveyUnitModel.getIdUE(), SurveyUnitModel.getMode()) + new SurveyUnitModel(SurveyUnitModel.getInterrogationId(), SurveyUnitModel.getMode()) ); } @@ -79,12 +79,12 @@ public List findIdUEsByIdQuestionnaire(String idQuestionnaire) } @Override - public List findIdUEsByIdCampaign(String idCampaign) { + public List findInterrogationIdsByCampaignId(String campaignId) { List surveyUnitModelList = new ArrayList<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdCampaign().equals(idCampaign)) + if(SurveyUnitModel.getCampaignId().equals(campaignId)) surveyUnitModelList.add( - new SurveyUnitModel(SurveyUnitModel.getIdUE(), SurveyUnitModel.getMode()) + new SurveyUnitModel(SurveyUnitModel.getInterrogationId(), SurveyUnitModel.getMode()) ); } @@ -92,7 +92,7 @@ public List findIdUEsByIdCampaign(String idCampaign) { } @Override - public Long deleteByIdQuestionnaire(String idQuestionnaire) { + public Long deleteByQuestionnaireId(String questionnaireId) { return null; } @@ -102,49 +102,49 @@ public long count() { } @Override - public Set findIdQuestionnairesByIdCampaign(String idCampaign) { - Set idQuestionnaireSet = new HashSet<>(); + public Set findQuestionnaireIdsByCampaignId(String campaignId) { + Set questionnaireIdSet = new HashSet<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdCampaign().equals(idCampaign)) - idQuestionnaireSet.add(SurveyUnitModel.getIdQuest()); + if(SurveyUnitModel.getCampaignId().equals(campaignId)) + questionnaireIdSet.add(SurveyUnitModel.getQuestionnaireId()); } - return idQuestionnaireSet; + return questionnaireIdSet; } @Override - public Set findDistinctIdCampaigns() { + public Set findDistinctCampaignIds() { Set campaignIds = new HashSet<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - campaignIds.add(SurveyUnitModel.getIdCampaign()); + campaignIds.add(SurveyUnitModel.getCampaignId()); } return campaignIds; } @Override - public long countByIdCampaign(String idCampaign) { + public long countByCampaignId(String campaignId) { return mongoStub.stream().filter( - SurveyUnitDto -> SurveyUnitDto.getIdCampaign().equals(idCampaign)).toList().size(); + SurveyUnitDto -> SurveyUnitDto.getCampaignId().equals(campaignId)).toList().size(); } @Override - public Set findDistinctIdQuestionnaires() { + public Set findDistinctQuestionnaireIds() { Set questionnaireIds = new HashSet<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - questionnaireIds.add(SurveyUnitModel.getIdQuest()); + questionnaireIds.add(SurveyUnitModel.getQuestionnaireId()); } return questionnaireIds; } @Override - public Set findIdCampaignsByIdQuestionnaire(String idQuestionnaire) { - Set idCampaignSet = new HashSet<>(); + public Set findCampaignIdsByQuestionnaireId(String questionnaireId) { + Set campaignIdSet = new HashSet<>(); for(SurveyUnitModel SurveyUnitModel : mongoStub){ - if(SurveyUnitModel.getIdQuest().equals(idQuestionnaire)) - idCampaignSet.add(SurveyUnitModel.getIdCampaign()); + if(SurveyUnitModel.getQuestionnaireId().equals(questionnaireId)) + campaignIdSet.add(SurveyUnitModel.getCampaignId()); } - return idCampaignSet; + return campaignIdSet; } } diff --git a/src/test/resources/features/do_we_extract_data.feature b/src/test/resources/features/do_we_extract_data.feature index 3dacbb3f..3dc5e075 100644 --- a/src/test/resources/features/do_we_extract_data.feature +++ b/src/test/resources/features/do_we_extract_data.feature @@ -3,18 +3,18 @@ Feature: Do we extract data ? Scenario Outline: Collected data extraction (COLLECTED only) Given We have data in directory "SAMPLETEST-PARADATA-v1" - When We create DTOs from file "reponse-platine/data.complete.validated.STPDv1.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v1.xml" - Then For SurveyUnit "" there should be at least one "" SurveyUnit DTO + When We create survey unit models from file "reponse-platine/data.complete.validated.STPDv1.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v1.xml" + Then For SurveyUnit "" there should be at least one "" SurveyUnit Model Examples: - | SurveyUnitId | ExpectedStatus | + | InterrogationId | ExpectedStatus | | 0000007 | COLLECTED | Scenario Outline: Collected data extraction (COLLECTED only) Given We have data in directory "SAMPLETEST-PARADATA-v1" - When We create DTOs from file "reponse-platine/data.complete.validated.STPDv1.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v1.xml" - Then For SurveyUnit "" there shouldn't be a "" SurveyUnit DTO + When We create survey unit models from file "reponse-platine/data.complete.validated.STPDv1.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v1.xml" + Then For SurveyUnit "" there shouldn't be a "" SurveyUnit Model Examples: - | SurveyUnitId | UnexpectedStatus | + | InterrogationId | UnexpectedStatus | | 0000007 | EDITED | | 0000007 | FORCED | | 0000007 | INPUTED | @@ -24,10 +24,10 @@ Feature: Do we extract data ? Scenario Outline: Collected data extraction (all states) Given We have data in directory "SAMPLETEST-PARADATA-v2" - When We create DTOs from file "reponse-platine/data.complete.validated.STPDv2.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v2.xml" - Then For SurveyUnit "" there should be at least one "" SurveyUnit DTO + When We create survey unit models from file "reponse-platine/data.complete.validated.STPDv2.20231122164209.xml" with DDI "ddi-SAMPLETEST-PARADATA-v2.xml" + Then For SurveyUnit "" there should be at least one "" SurveyUnit Model Examples: - | SurveyUnitId | ExpectedStatus | + | InterrogationId | ExpectedStatus | | 0000007 | COLLECTED | | 0000007 | EDITED | | 0000007 | FORCED | @@ -36,10 +36,10 @@ Feature: Do we extract data ? Scenario Outline: Collected data extraction content Given We have data in directory "" - When We create DTOs from file "" with DDI "" - Then We should have a "" DTO for survey unit "" with "" filled with "" at index + When We create survey unit models from file "" with DDI "" + Then We should have a "" Survey Unit model with id "" with "" filled with "" at index Examples: - | Directory | FileName | DDIFileName | ExpectedDataState | SurveyUnitId | VariableName | ExpectedValue | ExpectedIndex | + | Directory | FileName | DDIFileName | ExpectedDataState | InterrogationId | VariableName | ExpectedValue | ExpectedIndex | | SAMPLETEST-PARADATA-v1 | reponse-platine/data.complete.validated.STPDv1.20231122164209.xml |ddi-SAMPLETEST-PARADATA-v1.xml | COLLECTED | 0000007 | PRENOM_C | TESTPRENOM7 | 0 | | SAMPLETEST-PARADATA-v2 | reponse-platine/data.complete.validated.STPDv2.20231122164209.xml |ddi-SAMPLETEST-PARADATA-v2.xml | COLLECTED | 0000007 | PRENOM_C | TESTPRENOM7 | 0 | | SAMPLETEST-PARADATA-v2 | reponse-platine/data.complete.validated.STPDv2.20231122164209.xml |ddi-SAMPLETEST-PARADATA-v2.xml | EDITED | 0000007 | SANTE_ENFLOG71 | TESTSANTE7E | 0 |