OZG-6741 apply code review comments

src/main/helm/templates/deployment.yaml

@@ -124,7 +124,7 @@ spec:
             value: {{ .Values.ozgcloud.feature.organisationsEinheiten | quote }}
           {{- end }}
           {{- if eq ((.Values.ozgcloud).feature).organisationsEinheiten "true" }}
-          - name: ozgcloud_organisationseinheit_zufisucheuri
+          - name: ozgcloud_organisationseinheit_zufisearchuri
             value: {{ required "ozgcloud.organisationsEinheit.zufiSearchUri must be set if feature organisationsEinheiten is activated" ((.Values.ozgcloud).organisationsEinheit).zufiSearchUri}}
           {{- end }}
         envFrom:

src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitProperties.java

@@ -42,6 +42,10 @@ class OrganisationsEinheitProperties {
 
 	static final String ORGANISATIONS_EINHEIT_PROPERTIES_PREFIX = "ozgcloud.organisations-einheit";
 
+	/*
+	 * Uri pointing to the endpoint in Alfa that searches for Organisationseinheiten
+	 * in Zufi
+	 */
 	@NotBlank
-	private String zufiSucheUri;
+	private String zufiSearchUri;
 }

src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessor.java

@@ -61,6 +61,6 @@ class OrganisationsEinheitRootProcessor implements RepresentationModelProcessor<
 	}
 
 	private Link buildSearchOrganisationsEinheitLink() {
-		return Link.of(organisationsEinheitProperties.getZufiSucheUri(), REL_SEARCH_ORGANISATIONS_EINHEIT);
+		return Link.of(organisationsEinheitProperties.getZufiSearchUri(), REL_SEARCH_ORGANISATIONS_EINHEIT);
 	}
 }

src/test/helm/deployment_zufi_test.yaml

@@ -47,7 +47,7 @@ tests:
       - notContains:
           path: spec.template.spec.containers[0].env
           content:
-            name: ozgcloud_organisationseinheit_zufisucheuri
+            name: ozgcloud_organisationseinheit_zufisearchuri
           any: true
   - it: should not contain zufi search uri if organisationseinheit feature toggle is disabled
     set:
@@ -58,7 +58,7 @@ tests:
       - notContains:
           path: spec.template.spec.containers[0].env
           content:
-            name: ozgcloud_organisationseinheit_zufisucheuri
+            name: ozgcloud_organisationseinheit_zufisearchuri
           any: true
   - it: should set zufi search uri
     set:
@@ -71,7 +71,7 @@ tests:
       - contains:
           path: spec.template.spec.containers[0].env
           content:
-            name: ozgcloud_organisationseinheit_zufisucheuri
+            name: ozgcloud_organisationseinheit_zufisearchuri
             value: alfa/zufi/search/endpoint?searchBy={searchBy}
   - it: should fail if zufi search uri is not set
     set:

src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorITCase.java

 /*
- * Copyright (C) 2024 Das Land Schleswig-Holstein vertreten durch den
+ * Copyright (C) 2025 Das Land Schleswig-Holstein vertreten durch den
  * Ministerpräsidenten des Landes Schleswig-Holstein
  * Staatskanzlei
  * Abteilung Digitalisierung und zentrales IT-Management der Landesregierung
@@ -39,7 +39,7 @@ class OrganisationsEinheitRootProcessorITCase {
 	@Nested
 	@SpringBootTest(properties = {
 			"ozgcloud.feature.organisations-einheiten=true",
-			"ozgcloud.organisations-einheit.zufi-suche-url=foo"
+			"ozgcloud.organisations-einheit.zufi-search-url=foo"
 	})
 	@ITCase
 	class TestFeatureEnabled {
@@ -56,7 +56,7 @@ class OrganisationsEinheitRootProcessorITCase {
 	@Nested
 	@SpringBootTest(properties = {
 			"ozgcloud.feature.organisations-einheiten=false",
-			"ozgcloud.organisations-einheit.zufi-suche-url=foo"
+			"ozgcloud.organisations-einheit.zufi-search-url=foo"
 	})
 	@ITCase
 	class TestFeatureDisabled {

src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorTest.java

@@ -85,11 +85,11 @@ class OrganisationsEinheitRootProcessorTest {
 			@BeforeEach
 			void givenHasAdminRole() {
 				when(currentUserService.hasRole(anyString())).thenReturn(true);
-				when(organisationsEinheitProperties.getZufiSucheUri()).thenReturn(zufiSucheUri);
+				when(organisationsEinheitProperties.getZufiSearchUri()).thenReturn(zufiSucheUri);
 			}
 
 			@Nested
-			class OrganisationsEinheitenLinkRelation {
+			class TestOrganisationsEinheitenLinkRelation {
 
 				@Test
 				void shouldExist() {
@@ -110,7 +110,7 @@ class OrganisationsEinheitRootProcessorTest {
 			}
 
 			@Nested
-			class SearchOrganisationsEinheitenLink {
+			class TestSearchOrganisationsEinheitenLink {
 
 				@Test
 				void shouldExistsIfFeatureEnabled() {
@@ -123,7 +123,7 @@ class OrganisationsEinheitRootProcessorTest {
 				void shouldGetZufiSucheUri() {
 					processModel();
 
-					verify(organisationsEinheitProperties).getZufiSucheUri();
+					verify(organisationsEinheitProperties).getZufiSearchUri();
 
 				}