OZG-5284 Export Attachments of PostfachMails

alfa-xdomea/src/main/java/de/ozgcloud/alfa/export/ExportService.java

@@ -54,6 +54,7 @@ class ExportService {
 	ExportData collectExportData(String vorgangId, String filenameId) {
 		var vorgang = exportVorgangService.getVorgang(vorgangId);
 		var kommentarsData = exportKommentarService.createExportData(vorgang);
+		var postfachMailData = exportNachrichtService.createExportData(vorgang);
 		var representations = exportFileService.getRepresentations(vorgang).toList();
 		var attachments = exportFileService.getAttachments(vorgang).toList();
 		var formEngineName = getFormEngineName(vorgang);
@@ -65,9 +66,13 @@ class ExportService {
 				.withAttachments(exportFileService.createDokumentTypes(attachments, formEngineName).toList())
 				.withHistorie(exportHistorieService.createHistorienProtokollInformationTypes(vorgang).toList())
 				.withKommentare(kommentarsData.getDokumentTypes())
-				.withPostfachMails(exportNachrichtService.createDokumentTypes(vorgang).toList())
+				.withPostfachMails(postfachMailData.getDokumentTypes())
 				.build();
-		var exportFiles = Stream.of(representations.stream(), attachments.stream(), kommentarsData.getAttachments().stream()).flatMap(s -> s)
+		var exportFiles = Stream
+				.of(representations.stream(), attachments.stream(),
+						kommentarsData.getAttachments().stream(),
+						postfachMailData.getAttachments().stream())
+				.flatMap(s -> s)
 				.toList();
 		return ExportData.builder().abgabe(abgabe).exportFilename(buildXmlFilename(filenameId)).exportFiles(exportFiles).build();
 	}

alfa-xdomea/src/main/java/de/ozgcloud/alfa/postfach/ExportNachrichtService.java

 package de.ozgcloud.alfa.postfach;
 
+import java.util.List;
 import java.util.Optional;
 import java.util.stream.Stream;
 
@@ -9,6 +10,7 @@ import de.ozgcloud.alfa.common.binaryfile.BinaryFileService;
 import de.ozgcloud.alfa.common.file.OzgFile;
 import de.ozgcloud.alfa.common.user.UserProfile;
 import de.ozgcloud.alfa.common.user.UserService;
+import de.ozgcloud.alfa.postfach.PostfachMailExportData.PostfachMailExportDataBuilder;
 import de.ozgcloud.alfa.vorgang.VorgangWithEingang;
 import de.xoev.xdomea.DokumentType;
 import lombok.RequiredArgsConstructor;
@@ -20,15 +22,22 @@ public class ExportNachrichtService {
 	private final BinaryFileService binaryFileService;
 	private final UserService userService;
 
-	public Stream<DokumentType> createDokumentTypes(VorgangWithEingang vorgang) {
-		var postfachMails = postfachMailService.getAll(vorgang.getId());
-		return postfachMails.map(mail -> buildDokumentType(mail, vorgang));
+	public PostfachMailExportData createExportData(VorgangWithEingang vorgang) {
+		var builder = PostfachMailExportData.builder();
+		postfachMailService.getAll(vorgang.getId()).forEach(mail -> addPostfachMailToBuilder(mail, builder, vorgang));
+		return builder.build();
 	}
 
-	DokumentType buildDokumentType(PostfachMail postfachMail, VorgangWithEingang vorgang) {
+	void addPostfachMailToBuilder(PostfachMail postfachMail, PostfachMailExportDataBuilder builder, VorgangWithEingang vorgang) {
+		var attachments = getAttachments(postfachMail).toList();
+		builder.attachments(attachments);
+		builder.dokumentType(buildDokumentType(postfachMail, attachments, vorgang));
+	}
+
+	DokumentType buildDokumentType(PostfachMail postfachMail, List<OzgFile> attachments, VorgangWithEingang vorgang) {
 		return DokumentTypeBuilder.builder()
 				.withPostfachMail(postfachMail)
-				.withOzgFiles(getAttachments(postfachMail).toList())
+				.withOzgFiles(attachments)
 				.withUserProfile(getUserProfile(postfachMail))
 				.withVorgang(vorgang)
 				.build();

alfa-xdomea/src/main/java/de/ozgcloud/alfa/postfach/PostfachMailExportData.java

+package de.ozgcloud.alfa.postfach;
+
+import java.util.List;
+
+import de.ozgcloud.alfa.common.file.OzgFile;
+import de.xoev.xdomea.DokumentType;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.Singular;
+
+@Builder
+@Getter
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
+public class PostfachMailExportData {
+
+	@Singular
+	private List<DokumentType> dokumentTypes;
+	@Singular
+	private List<OzgFile> attachments;
+
+}

alfa-xdomea/src/test/java/de/ozgcloud/alfa/export/ExportServiceTest.java

@@ -36,6 +36,8 @@ import de.ozgcloud.alfa.kommentar.ExportKommentarService;
 import de.ozgcloud.alfa.kommentar.KommentarsExportData;
 import de.ozgcloud.alfa.kommentar.KommentarsExportDataTestFactory;
 import de.ozgcloud.alfa.postfach.ExportNachrichtService;
+import de.ozgcloud.alfa.postfach.PostfachMailExportData;
+import de.ozgcloud.alfa.postfach.PostfachMailExportDataTestFactory;
 import de.ozgcloud.alfa.vorgang.EingangHeaderTestFactory;
 import de.ozgcloud.alfa.vorgang.EingangTestFactory;
 import de.ozgcloud.alfa.vorgang.ExportVorgangService;
@@ -132,10 +134,10 @@ class ExportServiceTest {
 		private final List<OzgFile> attachments = List.of(OzgFileTestFactory.create());
 		private final List<DokumentType> representationsDokumentTypes = List.of(DokumentTypeTestFactory.create());
 		private final List<DokumentType> attachmentsDokumentTypes = List.of(DokumentTypeTestFactory.create());
-		private final List<DokumentType> postfachMailsDokumentTypes = List.of(DokumentTypeTestFactory.create());
 		private final List<HistorienProtokollInformationType> historienProtokollInformationTypes = List.of(
 				HistorienProtokollInformationTypeTestFactory.create());
 		private final KommentarsExportData kommentarsExportData = KommentarsExportDataTestFactory.create();
+		private final PostfachMailExportData postfachMailExportData = PostfachMailExportDataTestFactory.create();
 
 		@Mock
 		private XdomeaNachrichtBuilder xdomeaNachrichtBuilder;
@@ -170,7 +172,7 @@ class ExportServiceTest {
 			when(xdomeaNachrichtBuilder.withAttachments(attachmentsDokumentTypes)).thenReturn(xdomeaNachrichtBuilder);
 			when(xdomeaNachrichtBuilder.withHistorie(historienProtokollInformationTypes)).thenReturn(xdomeaNachrichtBuilder);
 			when(xdomeaNachrichtBuilder.withKommentare(kommentarsExportData.getDokumentTypes())).thenReturn(xdomeaNachrichtBuilder);
-			when(xdomeaNachrichtBuilder.withPostfachMails(postfachMailsDokumentTypes)).thenReturn(xdomeaNachrichtBuilder);
+			when(xdomeaNachrichtBuilder.withPostfachMails(postfachMailExportData.getDokumentTypes())).thenReturn(xdomeaNachrichtBuilder);
 			xdomeaNachrichtBuilderMockedStatic.when(XdomeaNachrichtBuilder::builder).thenReturn(xdomeaNachrichtBuilder);
 			when(xdomeaNachrichtBuilder.build()).thenReturn(abgabe);
 		}
@@ -195,7 +197,7 @@ class ExportServiceTest {
 		}
 
 		private void setUpExportNachrichtService() {
-			when(exportNachrichtService.createDokumentTypes(vorgang)).thenReturn(postfachMailsDokumentTypes.stream());
+			when(exportNachrichtService.createExportData(vorgang)).thenReturn(postfachMailExportData);
 		}
 
 		@AfterEach
@@ -260,10 +262,10 @@ class ExportServiceTest {
 		}
 
 		@Test
-		void shouldCreatePostfachMailsDokumentTypes() {
+		void shouldCreatePostfachMailsExportData() {
 			callService();
 
-			verify(exportNachrichtService).createDokumentTypes(vorgang);
+			verify(exportNachrichtService).createExportData(vorgang);
 		}
 
 		@Test
@@ -305,7 +307,7 @@ class ExportServiceTest {
 		void shouldSetPostfachMails() {
 			callService();
 
-			verify(xdomeaNachrichtBuilder).withPostfachMails(postfachMailsDokumentTypes);
+			verify(xdomeaNachrichtBuilder).withPostfachMails(postfachMailExportData.getDokumentTypes());
 		}
 
 		@Test
@@ -348,10 +350,11 @@ class ExportServiceTest {
 			var exportData = callService();
 
 			assertThat(exportData.getExportFiles())
-					.hasSize(3)
+					.hasSize(4)
 					.containsAll(representations)
 					.containsAll(attachments)
-					.containsAll(kommentarsExportData.getAttachments());
+					.containsAll(kommentarsExportData.getAttachments())
+					.containsAll(postfachMailExportData.getAttachments());
 		}
 
 		@Test

alfa-xdomea/src/test/java/de/ozgcloud/alfa/postfach/ExportNachrichtServiceTest.java

@@ -25,7 +25,7 @@ import de.ozgcloud.alfa.common.user.UserProfile;
 import de.ozgcloud.alfa.common.user.UserProfileTestFactory;
 import de.ozgcloud.alfa.common.user.UserService;
 import de.ozgcloud.alfa.export.DokumentTypeTestFactory;
-import de.ozgcloud.alfa.vorgang.VorgangHeaderTestFactory;
+import de.ozgcloud.alfa.postfach.PostfachMailExportData.PostfachMailExportDataBuilder;
 import de.ozgcloud.alfa.vorgang.VorgangWithEingang;
 import de.ozgcloud.alfa.vorgang.VorgangWithEingangTestFactory;
 import de.xoev.xdomea.DokumentType;
@@ -46,47 +46,109 @@ class ExportNachrichtServiceTest {
 	private BinaryFileService binaryFileService;
 
 	@Nested
-	class TestCreateDokumentTypes {
+	class TestCreateExportData {
+		private final VorgangWithEingang vorgang = VorgangWithEingangTestFactory.create();
 		private final PostfachMail postfachMail = PostfachMailTestFactory.create();
+		private final PostfachMailExportData exportData = PostfachMailExportDataTestFactory.create();
 
-		private final VorgangWithEingang vorgang = VorgangWithEingangTestFactory.create();
+		private MockedStatic<PostfachMailExportData> postfachMailExportDataMockedStatic;
+		@Mock
+		private PostfachMailExportDataBuilder exportDataBuilder;
+
+		@BeforeEach
+		void setUpMock() {
+			postfachMailExportDataMockedStatic = mockStatic(PostfachMailExportData.class);
+			postfachMailExportDataMockedStatic.when(PostfachMailExportData::builder).thenReturn(exportDataBuilder);
+			when(postfachMailService.getAll(vorgang.getId())).thenReturn(Stream.of(postfachMail));
+			doNothing().when(service).addPostfachMailToBuilder(postfachMail, exportDataBuilder, vorgang);
+			when(exportDataBuilder.build()).thenReturn(exportData);
+		}
+
+		@AfterEach
+		void tearDown() {
+			postfachMailExportDataMockedStatic.close();
+		}
 
 		@Test
-		void shouldCallPostfachMailServiceGetAll() {
-			callCreateDokumentTypes();
+		void shouldLoadPostfachMails() {
+			callMethodToBeTested();
 
-			verify(postfachMailService).getAll(VorgangHeaderTestFactory.ID);
+			verify(postfachMailService).getAll(vorgang.getId());
 		}
 
 		@Test
-		void shouldCallBuildDokumentType() {
-			when(postfachMailService.getAll(VorgangHeaderTestFactory.ID)).thenReturn(Stream.of(postfachMail));
-			doReturn(DokumentTypeTestFactory.create()).when(service).buildDokumentType(postfachMail, vorgang);
+		void shouldAddNachrichtToBuilder() {
+			callMethodToBeTested();
 
-			callCreateDokumentTypes().toList();
+			verify(service).addPostfachMailToBuilder(postfachMail, exportDataBuilder, vorgang);
+		}
 
-			verify(service).buildDokumentType(postfachMail, vorgang);
+		@Test
+		void shouldBuildExportData() {
+			callMethodToBeTested();
+
+			verify(exportDataBuilder).build();
 		}
 
 		@Test
-		void shouldReturnBuiltDokumente() {
-			var expectedDokument1 = DokumentTypeTestFactory.create();
-			var expectedDokument2 = DokumentTypeTestFactory.create();
-			var postfachMail2 = PostfachMailTestFactory.create();
+		void shouldReturnBuiltExportData() {
+			var result = callMethodToBeTested();
 
-			when(postfachMailService.getAll(VorgangHeaderTestFactory.ID)).thenReturn(Stream.of(postfachMail, postfachMail2));
-			doReturn(expectedDokument1).when(service).buildDokumentType(postfachMail, vorgang);
-			doReturn(expectedDokument2).when(service).buildDokumentType(postfachMail2, vorgang);
+			assertThat(result).isEqualTo(exportData);
+		}
 
-			var resultDokumente = callCreateDokumentTypes();
+		private PostfachMailExportData callMethodToBeTested() {
+			return service.createExportData(vorgang);
+		}
+	}
 
-			assertThat(resultDokumente).containsExactly(expectedDokument1, expectedDokument2);
+	@Nested
+	class TestAddPostfachMailToBuilder {
+		private final VorgangWithEingang vorgang = VorgangWithEingangTestFactory.create();
+		private final PostfachMail postfachMail = PostfachMailTestFactory.create();
+		private final List<OzgFile> attachments = List.of(OzgFileTestFactory.create());
+		private final DokumentType dokumentType = DokumentTypeTestFactory.create();
+
+		@Mock
+		private PostfachMailExportDataBuilder exportDataBuilder;
+
+		@BeforeEach
+		void setUpMocks() {
+			doReturn(attachments.stream()).when(service).getAttachments(postfachMail);
+			doReturn(dokumentType).when(service).buildDokumentType(postfachMail, attachments, vorgang);
 		}
 
-		private Stream<DokumentType> callCreateDokumentTypes() {
-			return service.createDokumentTypes(vorgang);
+		@Test
+		void shouldCallGetAttachents() {
+			callMethodToBeTested();
+
+			verify(service).getAttachments(postfachMail);
+		}
+
+		@Test
+		void shouldAddAttachmentsToBuilder() {
+			callMethodToBeTested();
+
+			verify(exportDataBuilder).attachments(attachments);
+		}
+
+		@Test
+		void shouldBuildDokumentType() {
+			callMethodToBeTested();
+
+			verify(service).buildDokumentType(postfachMail, attachments, vorgang);
 		}
 
+		@Test
+		void shouldAddDokumentTypeToBuilder() {
+			callMethodToBeTested();
+
+			verify(exportDataBuilder).dokumentType(dokumentType);
+		}
+
+		private void callMethodToBeTested() {
+			service.addPostfachMailToBuilder(postfachMail, exportDataBuilder, vorgang);
+		}
 	}
 
 	@Nested
@@ -98,9 +160,7 @@ class ExportNachrichtServiceTest {
 		@Mock
 		private DokumentTypeBuilder dokumentTypeBuilder;
 
-		private final OzgFile ozgFile = OzgFileTestFactory.create();
-		private final Stream<OzgFile> ozgFileStream = Stream.of(ozgFile);
-		private final List<OzgFile> ozgFileList = List.of(ozgFile);
+		private final List<OzgFile> ozgFileList = List.of(OzgFileTestFactory.create());
 		private final Optional<UserProfile> userProfile = Optional.of(UserProfileTestFactory.create());
 
 		@BeforeEach
@@ -112,7 +172,6 @@ class ExportNachrichtServiceTest {
 			when(dokumentTypeBuilder.withOzgFiles(ozgFileList)).thenReturn(dokumentTypeBuilder);
 			when(dokumentTypeBuilder.withUserProfile(userProfile)).thenReturn(dokumentTypeBuilder);
 			when(dokumentTypeBuilder.withVorgang(vorgang)).thenReturn(dokumentTypeBuilder);
-			doReturn(ozgFileStream).when(service).getAttachments(postfachMail);
 			doReturn(userProfile).when(service).getUserProfile(postfachMail);
 		}
 
@@ -128,13 +187,6 @@ class ExportNachrichtServiceTest {
 			dokumentTypeBuilderMockedStatic.verify(() -> DokumentTypeBuilder.builder());
 		}
 
-		@Test
-		void shouldCallGetOzgFiles() {
-			callService();
-
-			verify(service).getAttachments(postfachMail);
-		}
-
 		@Test
 		void shouldCallGetUserProfile() {
 			callService();
@@ -188,7 +240,7 @@ class ExportNachrichtServiceTest {
 		}
 
 		private DokumentType callService() {
-			return service.buildDokumentType(postfachMail, vorgang);
+			return service.buildDokumentType(postfachMail, ozgFileList, vorgang);
 		}
 	}
 

alfa-xdomea/src/test/java/de/ozgcloud/alfa/postfach/PostfachMailExportDataTestFactory.java

+package de.ozgcloud.alfa.postfach;
+
+import de.ozgcloud.alfa.common.file.OzgFile;
+import de.ozgcloud.alfa.common.file.OzgFileTestFactory;
+import de.ozgcloud.alfa.export.DokumentTypeTestFactory;
+import de.ozgcloud.alfa.postfach.PostfachMailExportData.PostfachMailExportDataBuilder;
+import de.xoev.xdomea.DokumentType;
+
+public class PostfachMailExportDataTestFactory {
+
+	public static final DokumentType DOKUMENT_TYPE = DokumentTypeTestFactory.create();
+	public static final OzgFile OZG_FILE = OzgFileTestFactory.create();
+
+	public static PostfachMailExportData create() {
+		return createBuilder().build();
+	}
+
+	public static PostfachMailExportDataBuilder createBuilder() {
+		return PostfachMailExportData.builder()
+				.dokumentType(DOKUMENT_TYPE)
+				.attachment(OZG_FILE);
+	}
+}