From 2f6ef467785c248348c7d633d0bad656454aac74 Mon Sep 17 00:00:00 2001
From: Jan Zickermann <jan.zickermann@dataport.de>
Date: Wed, 2 Oct 2024 12:18:34 +0200
Subject: [PATCH] OZG-6748 Cleanup more tests

---
 .../enginebased/afm/AfmZustaendigeStelleMapperTest.java    | 7 ++++---
 .../DFoerdermittelEngineBasedSemantikAdapterTest.java      | 7 ++++---
 .../FormSolutionsEngineBasedAdapterITCase.java             | 7 +++++--
 3 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/afm/AfmZustaendigeStelleMapperTest.java b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/afm/AfmZustaendigeStelleMapperTest.java
index cb0a8fc25..e9abbeff9 100644
--- a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/afm/AfmZustaendigeStelleMapperTest.java
+++ b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/afm/AfmZustaendigeStelleMapperTest.java
@@ -37,6 +37,7 @@ import org.mockito.Spy;
 
 import de.ozgcloud.eingang.common.formdata.FormData;
 import de.ozgcloud.eingang.common.formdata.FormDataTestFactory;
+import de.ozgcloud.eingang.common.formdata.ZustaendigeStelle;
 
 class AfmZustaendigeStelleMapperTest {
 
@@ -79,7 +80,7 @@ class AfmZustaendigeStelleMapperTest {
 				var parsedFormData = parseZustaendigeStelleData(formData);
 
 				assertThat(parsedFormData.getZustaendigeStelles())
-						.extracting("organisationseinheitenId")
+						.extracting(ZustaendigeStelle::getOrganisationseinheitenId)
 						.containsExactly(AfmZustaendigeStelleTestFactory.ORGANISATIONSEINHEITEN_ID);
 			}
 
@@ -88,7 +89,7 @@ class AfmZustaendigeStelleMapperTest {
 				var parsedFormData = parseZustaendigeStelleData(formData);
 
 				assertThat(parsedFormData.getZustaendigeStelles())
-						.extracting("bezeichnung")
+						.extracting(ZustaendigeStelle::getBezeichnung)
 						.containsExactly(BEZEICHNUNG);
 			}
 
@@ -97,7 +98,7 @@ class AfmZustaendigeStelleMapperTest {
 				var parsedFormData = parseZustaendigeStelleData(formData);
 
 				assertThat(parsedFormData.getZustaendigeStelles())
-						.extracting("email")
+						.extracting(ZustaendigeStelle::getEmail)
 						.containsExactly(AfmZustaendigeStelleTestFactory.EMAIL);
 			}
 
diff --git a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
index 698b665b5..079902c9d 100644
--- a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
+++ b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
@@ -26,6 +26,7 @@ import de.ozgcloud.eingang.common.formdata.IncomingFile;
 import de.ozgcloud.eingang.common.formdata.PostfachAddressTestFactory;
 import de.ozgcloud.eingang.common.formdata.ServiceKontoTestFactory;
 import de.ozgcloud.eingang.common.formdata.StringBasedIdentifier;
+import de.ozgcloud.eingang.common.formdata.ZustaendigeStelle;
 import de.ozgcloud.eingang.common.formdata.ZustaendigeStelleTestFactory;
 import de.ozgcloud.eingang.semantik.common.ServiceKontoFactory;
 
@@ -230,7 +231,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
 				var formData = adapter.addOrganisationsEinheitId(inputFormData);
 
 				assertThat(formData.getZustaendigeStelles())
-						.extracting("organisationseinheitenId")
+						.extracting(ZustaendigeStelle::getOrganisationseinheitenId)
 						.containsExactly(ORGANISATIONS_EINHEIT_ID);
 			}
 		}
@@ -243,7 +244,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
 				var formData = adapter.addOrganisationsEinheitId(DFoerdermittelFormDataTestFactory.create());
 
 				assertThat(formData.getZustaendigeStelles())
-						.extracting("organisationseinheitenId")
+						.extracting(ZustaendigeStelle::getOrganisationseinheitenId)
 						.containsExactly(ORGANISATIONS_EINHEIT_ID);
 			}
 		}
@@ -266,7 +267,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
 				var formData = adapter.addOrganisationsEinheitId(inputFormData);
 
 				assertThat(formData.getZustaendigeStelles())
-						.extracting("organisationseinheitenId")
+						.extracting(ZustaendigeStelle::getOrganisationseinheitenId)
 						.containsExactly(ORGANISATIONS_EINHEIT_ID);
 			}
 
diff --git a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/formsolutions/FormSolutionsEngineBasedAdapterITCase.java b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/formsolutions/FormSolutionsEngineBasedAdapterITCase.java
index 854931d9a..b0535b604 100644
--- a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/formsolutions/FormSolutionsEngineBasedAdapterITCase.java
+++ b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/formsolutions/FormSolutionsEngineBasedAdapterITCase.java
@@ -50,6 +50,7 @@ import de.ozgcloud.eingang.common.formdata.FormData;
 import de.ozgcloud.eingang.common.formdata.FormDataTestFactory;
 import de.ozgcloud.eingang.common.formdata.IncomingFileGroupTestFactory;
 import de.ozgcloud.eingang.common.formdata.IncomingFileTestFactory;
+import de.ozgcloud.eingang.common.formdata.ZustaendigeStelle;
 import de.ozgcloud.eingang.semantik.SemantikAdapter;
 import lombok.SneakyThrows;
 
@@ -236,14 +237,16 @@ public class FormSolutionsEngineBasedAdapterITCase {
 			void shouldMap() {
 				var data = engineAdapter.parseFormData(formData);
 
-				assertThat(data.getZustaendigeStelles()).isNotNull();
+				assertThat(data.getZustaendigeStelles()).isNotNull().isNotEmpty();
 			}
 
 			@Test
 			void shouldMapOrganistaionseinheitenId() {
 				var data = engineAdapter.parseFormData(formData);
 
-				assertThat(data.getZustaendigeStelles().getFirst().getOrganisationseinheitenId()).isEqualTo("test");
+				assertThat(data.getZustaendigeStelles())
+						.extracting(ZustaendigeStelle::getOrganisationseinheitenId)
+						.containsExactly("test");
 			}
 		}
 	}
-- 
GitLab