Marmelator/persistence-service
1
0
Fork 0
Code Issues Pull Requests 17 Actions Packages Projects Releases 1.1k Wiki Activity

RED-6770: Fixed lazy init exception in dossier template import by eagerly fetching the needed collection

Browse Source
This commit is contained in:
Viktor Seifert 2023-05-17 15:37:19 +02:00
parent 58fe156920
commit a8a612de55
2 changed files with 25 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/service/DossierTemplateImportService.java
View File

@ -604,8 +604,7 @@ public class DossierTemplateImportService {
private void updateTypes(ImportTemplateResult request, String dossierTemplateId) {
List<TypeEntity> currentTypes = dossierTemplatePersistenceService.getDossierTemplate(dossierTemplateId)
.getDossierTypes()
List<TypeEntity> currentTypes = dossierTemplatePersistenceService.getTypesForDossierTemplate(dossierTemplateId)
.stream()
.filter(t -> t.getDossierId() == null)
.collect(Collectors.toList());

29
persistence-service-v1/persistence-service-server-v1/src/test/java/com/iqser/red/service/peristence/v1/server/integration/tests/DossierTemplateImportTest.java
View File

@ -110,15 +110,34 @@ public class DossierTemplateImportTest extends AbstractPersistenceServerServiceT
}
private void testDossierTemplateImport(byte[] archive) {
private DossierTemplate testDossierTemplateImport(byte[] archive) {
var request = new ImportDossierTemplateRequest();
request.setArchive(archive);
request.setUpdateExistingDossierTemplate(false);
request.setUserId("system");
var request = ImportDossierTemplateRequest.builder().archive(archive).updateExistingDossierTemplate(false).userId("system").build();
DossierTemplate dossierTemplate = dossierTemplateManagementService.importDossierTemplate(request);
assertThat(dossierTemplate).isNotNull();
assertThat(dossierTemplate.getId()).isNotBlank();
return dossierTemplate;
}
@SneakyThrows
@Test
public void testDossierTemplateImportOverwrite() {
TenantContext.setTenantId("redaction");
var archive = loadFileFromClasspath("EFSA_sanitisation_GFL_v1.zip");
var dossierTemplate = testDossierTemplateImport(archive);
var request = ImportDossierTemplateRequest.builder()
.archive(archive)
.updateExistingDossierTemplate(true)
.userId("system")
.dossierTemplateId(dossierTemplate.getId())
.build();
DossierTemplate reimportedTemplate = dossierTemplateManagementService.importDossierTemplate(request);
assertThat(reimportedTemplate).isNotNull();
assertThat(reimportedTemplate.getId()).isNotBlank();
}