From 82e6b20ff4b4210e33a2cc0ddbb056e6086230d7 Mon Sep 17 00:00:00 2001
From: Felix Reichenbach <felix.reichenbach@mgm-tp.com>
Date: Fri, 10 Jan 2025 16:49:28 +0100
Subject: [PATCH] OZG-6741 apply code review comments
---
src/main/helm/templates/deployment.yaml | 2 +-
.../OrganisationsEinheitProperties.java | 6 +++++-
.../OrganisationsEinheitRootProcessor.java | 2 +-
src/test/helm/deployment_zufi_test.yaml | 6 +++---
.../OrganisationsEinheitRootProcessorITCase.java | 6 +++---
.../OrganisationsEinheitRootProcessorTest.java | 8 ++++----
6 files changed, 17 insertions(+), 13 deletions(-)
diff --git a/src/main/helm/templates/deployment.yaml b/src/main/helm/templates/deployment.yaml
index 79e17f06..6fd41075 100644
--- a/src/main/helm/templates/deployment.yaml
+++ b/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:
diff --git a/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitProperties.java b/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitProperties.java
index 95af01fb..cda6c085 100644
--- a/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitProperties.java
+++ b/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;
}
diff --git a/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessor.java b/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessor.java
index 34caabc6..6265c969 100644
--- a/src/main/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessor.java
+++ b/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);
}
}
diff --git a/src/test/helm/deployment_zufi_test.yaml b/src/test/helm/deployment_zufi_test.yaml
index eef7e755..58c8f7a4 100644
--- a/src/test/helm/deployment_zufi_test.yaml
+++ b/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:
diff --git a/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorITCase.java b/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorITCase.java
index 6e1d8b47..a36223d9 100644
--- a/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorITCase.java
+++ b/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorITCase.java
@@ -1,5 +1,5 @@
/*
- * 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 {
diff --git a/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorTest.java b/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorTest.java
index 3e1df175..93a4bbb2 100644
--- a/src/test/java/de/ozgcloud/admin/organisationseinheit/OrganisationsEinheitRootProcessorTest.java
+++ b/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();
}
--
GitLab