Merge pull request 'OZG-7131 set fachstelle host' (#517) from...

Merge pull request 'OZG-7131 set fachstelle host' (#517) from OZG-7131-Extend-Muk-Message into master

Reviewed-on: https://git.ozg-sh.de/ozgcloud-app/vorgang-manager/pulls/517


Reviewed-by: OZGCloud <ozgcloud@mgm-tp.com>

src/main/helm/templates/_helpers.tpl

@@ -164,6 +164,12 @@ app.kubernetes.io/namespace: {{ include "app.namespace" . }}
 {{- end -}}
 {{- end -}}
 
+{{- define "app.fachstelleHost" -}}
+{{- if (.Values.fachstelle).enabled -}}
+{{- required "fachstelle.host must be set" .Values.fachstelle.host | quote }}
+{{- end -}}
+{{- end -}}
+
 {{- define "app.getZufiManagerAddress" -}}
 {{- required "zufiManager.address must be set" (.Values.zufiManager).address -}}
 {{- end -}}
\ No newline at end of file

src/main/helm/templates/deployment.yaml

@@ -278,6 +278,8 @@ spec:
 
           - name: ozgcloud_alfa_host
             value: {{ include "app.alfaHost" . }}
+          - name: ozgcloud_fachstelle_host
+            value: {{ include "app.fachstelleHost" . }}
           - name: ozgcloud_vorgangManager_serviceAddress
             value: {{ include "app.ozgcloud_vorgangmanager_address" . }}
 

src/test/helm/deployment_fachstelle_host_test.yaml

+#
+# Copyright (C) 2024 Das Land Schleswig-Holstein vertreten durch den
+# Ministerpräsidenten des Landes Schleswig-Holstein
+# Staatskanzlei
+# Abteilung Digitalisierung und zentrales IT-Management der Landesregierung
+#
+# Lizenziert unter der EUPL, Version 1.2 oder - sobald
+# diese von der Europäischen Kommission genehmigt wurden -
+# Folgeversionen der EUPL ("Lizenz");
+# Sie dürfen dieses Werk ausschließlich gemäß
+# dieser Lizenz nutzen.
+# Eine Kopie der Lizenz finden Sie hier:
+#
+# https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
+#
+# Sofern nicht durch anwendbare Rechtsvorschriften
+# gefordert oder in schriftlicher Form vereinbart, wird
+# die unter der Lizenz verbreitete Software "so wie sie
+# ist", OHNE JEGLICHE GEWÄHRLEISTUNG ODER BEDINGUNGEN -
+# ausdrücklich oder stillschweigend - verbreitet.
+# Die sprachspezifischen Genehmigungen und Beschränkungen
+# unter der Lizenz sind dem Lizenztext zu entnehmen.
+#
+
+suite: test fachstelle host
+release:
+  name: vorgang-manager
+  namespace: sh-helm-test
+templates:
+  - templates/deployment.yaml
+set:
+  baseUrl: test.sh.ozg-cloud.de
+  ozgcloud:
+    environment: dev
+    bezeichner: helm
+  imagePullSecret: test-image-pull-secret
+tests:
+  - it: should set fachstelle host
+    set:
+      fachstelle:
+        enabled: true
+        host: https://hamburg.mein-kreis.de
+    asserts:
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: ozgcloud_fachstelle_host
+            value: https://hamburg.mein-kreis.de
+  - it: should fail if host url is not set
+    set:
+      fachstelle:
+        enabled: true
+    asserts:
+      - failedTemplate:
+          errorMessage: "fachstelle.host must be set"
+  - it: should not fail if fachstelle is not enabled and host is not set
+    asserts:
+      - notContains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: ozgcloud_fachstelle_host

vorgang-manager-server/src/test/java/de/ozgcloud/vorgang/collaboration/CollaborationITCase.java

@@ -33,6 +33,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.util.List;
 import java.util.Map;
+import java.util.Optional;
 import java.util.UUID;
 import java.util.concurrent.TimeUnit;
 
@@ -54,6 +55,7 @@ import org.springframework.data.mongodb.core.query.Criteria;
 import org.springframework.data.mongodb.core.query.Query;
 import org.springframework.data.mongodb.gridfs.GridFsTemplate;
 import org.springframework.test.annotation.DirtiesContext;
+import org.springframework.test.util.ReflectionTestUtils;
 
 import com.google.protobuf.ByteString;
 import com.thedeanda.lorem.LoremIpsum;
@@ -64,6 +66,7 @@ import de.ozgcloud.apilib.user.OzgCloudUserProfile;
 import de.ozgcloud.apilib.user.OzgCloudUserProfileService;
 import de.ozgcloud.collaboration.CollaborationManagerConfiguration;
 import de.ozgcloud.collaboration.CollaborationRequest;
+import de.ozgcloud.collaboration.CollaborationService;
 import de.ozgcloud.collaboration.CollaborationServiceGrpc.CollaborationServiceBlockingStub;
 import de.ozgcloud.collaboration.GrpcGetFileContentRequest;
 import de.ozgcloud.collaboration.request.CollaborationRequestId;
@@ -107,7 +110,7 @@ import net.devh.boot.grpc.client.inject.GrpcClient;
 })
 @DataITCase
 @WithMockCustomUser
-@DirtiesContext
+@DirtiesContext(classMode = DirtiesContext.ClassMode.AFTER_EACH_TEST_METHOD)
 class CollaborationITCase {
 
 	private static final String FIELD_COLLABORATION_VORGANG_ID = "collaborationVorgangId";
@@ -117,6 +120,8 @@ class CollaborationITCase {
 
 	@Autowired
 	private CommandService commandService;
+	@Autowired
+	private CollaborationService collaborationService;
 
 	@Autowired
 	private MongoOperations mongoOperations;
@@ -286,6 +291,8 @@ class CollaborationITCase {
 
 		private static final int COLLABORATION_LEVEL = 4;
 
+		@Nested
+		class TestSuccessfully {
 			@Captor
 			private ArgumentCaptor<PostfachNachricht> postfachNachrichtCaptor;
 
@@ -328,6 +335,18 @@ class CollaborationITCase {
 			}
 		}
 
+		@Test
+		void shouldFailOnMissingFachstelle() {
+			ReflectionTestUtils.setField(collaborationService, "urlProvider", Optional.empty());
+			var command = commandService.createCommand(buildCreateCollaborationVorgangCommand(vorgangId, COLLABORATION_LEVEL));
+
+			waitUntilCommandHasStatus(command.getId(), CommandStatus.ERROR);
+
+			command = mongoOperations.findById(command.getId(), Command.class);
+			assertThat(command.getErrorMessage()).contains("Fachstelle not configured.");
+		}
+	}
+
 	@DisplayName("Collaboration requests")
 	@Nested
 	class TestCollaborationRequests {

vorgang-manager-server/src/test/resources/application-itcase.yml

@@ -20,6 +20,10 @@ ozgcloud:
   notification:
     scheduling:
       enabled: false
+  fachstelle:
+    host: fachstelle.test.de
+  vorgang-manager:
+    serviceAddress: vorgang-manager.by-test-test
 
 mongock:
   enabled: false