OZG-6706 OZG-7497 Remove logic from BescheidRemoteService

alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidManagerFeatures.java

+/*
+ * 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.
+ */
+package de.ozgcloud.alfa.bescheid;
+
+import lombok.Builder;
+import lombok.Getter;
+
+@Builder
+@Getter
+class BescheidManagerFeatures {
+
+	private boolean canCreateBescheidDocument;
+}

alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidManagerFeaturesMapper.java

+/*
+ * 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.
+ */
+package de.ozgcloud.alfa.bescheid;
+
+import org.mapstruct.Mapper;
+import org.mapstruct.NullValueCheckStrategy;
+import org.mapstruct.ReportingPolicy;
+
+import de.ozgcloud.document.bescheid.GrpcBescheidManagerFeatures;
+
+@Mapper(unmappedTargetPolicy = ReportingPolicy.ERROR, nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS)
+interface BescheidManagerFeaturesMapper {
+
+	BescheidManagerFeatures fromGrpc(GrpcBescheidManagerFeatures grpcBescheidManagerFeatures);
+}

alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidRemoteService.java

@@ -46,6 +46,8 @@ class BescheidRemoteService {
 	private BescheidServiceBlockingStub bescheidServiceStub;
 	@Autowired
 	private BescheidMapper bescheidMapper;
+	@Autowired
+	private BescheidManagerFeaturesMapper bescheidManagerFeaturesMapper;
 
 	public Optional<Bescheid> getBescheidDraft(String vorgangId) {
 		var request = buildGetBescheidDraftRequest(vorgangId);
@@ -83,9 +85,8 @@ class BescheidRemoteService {
 		return GrpcGetAllBescheidRequest.newBuilder().setVorgangId(vorgangId).build();
 	}
 
-	// TODO Logik in den Service verschieben
-	public boolean canCreateBescheidDocument() {
+	public Optional<BescheidManagerFeatures> getBescheidManagerFeatures() {
 		var response = bescheidServiceStub.getConfig(GrpcBescheidManagerConfigRequest.newBuilder().build());
-		return response.hasFeatures() && response.getFeatures().getCanCreateBescheidDocument();
+		return response.hasFeatures() ? Optional.of(bescheidManagerFeaturesMapper.fromGrpc(response.getFeatures())) : Optional.empty();
 	}
 }
\ No newline at end of file

alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidService.java

@@ -59,7 +59,7 @@ public class BescheidService {
 	}
 
 	public boolean canCreateBescheidDocumentAutomatically() {
-		return remoteService.canCreateBescheidDocument();
+		return remoteService.getBescheidManagerFeatures().map(BescheidManagerFeatures::isCanCreateBescheidDocument).orElse(false);
 	}
 
 	public boolean existsBescheid(String vorgangId) {

alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidManagerFeaturesMapperTest.java

+/*
+ * 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.
+ */
+package de.ozgcloud.alfa.bescheid;
+
+import static org.assertj.core.api.Assertions.*;
+
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.ValueSource;
+import org.mapstruct.factory.Mappers;
+
+import de.ozgcloud.document.bescheid.GrpcBescheidManagerFeatures;
+
+class BescheidManagerFeaturesMapperTest {
+
+	private final BescheidManagerFeaturesMapper mapper = Mappers.getMapper(BescheidManagerFeaturesMapper.class);
+
+	@ParameterizedTest
+	@ValueSource(booleans = {true, false})
+	void shouldMapCanCreateBescheidDocument(boolean canCreate) {
+		var grpcFeatures = GrpcBescheidManagerFeatures.newBuilder().setCanCreateBescheidDocument(canCreate).build();
+
+		var features = mapper.fromGrpc(grpcFeatures);
+
+		assertThat(features.isCanCreateBescheidDocument()).isEqualTo(canCreate);
+	}
+}

alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidRemoteServiceTest.java

@@ -57,6 +57,8 @@ class BescheidRemoteServiceTest {
 	private BescheidServiceBlockingStub bescheidServiceStub;
 	@Mock
 	private BescheidMapper bescheidMapper;
+	@Mock
+	private BescheidManagerFeaturesMapper featuresMapper;
 
 	@Nested
 	class TestGetBescheidDraft {
@@ -206,51 +208,68 @@ class BescheidRemoteServiceTest {
 	}
 
 	@Nested
-	class TestCanCreateBescheidDocument {
+	class TestGetBescheidManagerFeatures {
 
 		private final GrpcBescheidManagerConfigRequest request = GrpcBescheidManagerConfigRequestTestFactory.create();
 		private final GrpcBescheidManagerConfigResponse respone = GrpcBescheidManagerConfigResponseTestFactory.create();
 
+		@Test
+		void shouldCallGrpcService() {
+			when(bescheidServiceStub.getConfig(request)).thenReturn(respone);
+			when(featuresMapper.fromGrpc(any())).thenReturn(BescheidManagerFeatures.builder().build());
+
+			service.getBescheidManagerFeatures();
+
+			verify(bescheidServiceStub).getConfig(request);
+		}
+
+		@Nested
+		class OnFeaturesArePresent {
+
+			private final BescheidManagerFeatures mappedFeatures = BescheidManagerFeatures.builder().build();
+
 			@BeforeEach
 			void setUp() {
 				when(bescheidServiceStub.getConfig(request)).thenReturn(respone);
+				when(featuresMapper.fromGrpc(any())).thenReturn(mappedFeatures);
 			}
 
 			@Test
-		void shouldCallRemoteService() {
-			service.canCreateBescheidDocument();
+			void shouldCallMapper() {
+				service.getBescheidManagerFeatures();
 
-			verify(bescheidServiceStub).getConfig(request);
+				verify(featuresMapper).fromGrpc(GrpcBescheidManagerFeaturesTestFactory.create());
 			}
 
 			@Test
-		void shouldReturnTrue() {
-			var canCreate = service.canCreateBescheidDocument();
+			void shouldReturnMappedFeatures() {
+				var features = service.getBescheidManagerFeatures();
 
-			assertThat(canCreate).isTrue();
+				assertThat(features).isNotEmpty().get().isSameAs(mappedFeatures);
+			}
 		}
 
-		@Test
-		void shouldReturnFalseIfNoFeaturesAvailable() {
+		@Nested
+		class OnFeaturesAreAbsent {
+
+			@BeforeEach
+			void init() {
 				when(bescheidServiceStub.getConfig(request)).thenReturn(GrpcBescheidManagerConfigResponse.newBuilder().build());
+			}
 
-			var canCreate = service.canCreateBescheidDocument();
+			@Test
+			void shouldNotCallMapper() {
+				service.getBescheidManagerFeatures();
 
-			assertThat(canCreate).isFalse();
+				verify(featuresMapper, never()).fromGrpc(any());
 			}
 
 			@Test
-		void shouldReturnFalseIfFeatureDisabled() {
-			var respones = GrpcBescheidManagerConfigResponseTestFactory.createBuilder()
-					.setFeatures(GrpcBescheidManagerFeaturesTestFactory.createBuilder()
-							.setCanCreateBescheidDocument(false)
-							.build())
-					.build();
-			when(bescheidServiceStub.getConfig(request)).thenReturn(respones);
-
-			var canCreate = service.canCreateBescheidDocument();
+			void shouldReturnEmpty() {
+				var features = service.getBescheidManagerFeatures();
 
-			assertThat(canCreate).isFalse();
+				assertThat(features).isEmpty();
+			}
 		}
 	}
 

alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidServiceTest.java

@@ -168,19 +168,31 @@ class BescheidServiceTest {
 
 		@Test
 		void shouldCallRemoteService() {
+			when(remoteService.getBescheidManagerFeatures()).thenReturn(Optional.of(BescheidManagerFeatures.builder().build()));
+
 			service.canCreateBescheidDocumentAutomatically();
 
-			verify(remoteService).canCreateBescheidDocument();
+			verify(remoteService).getBescheidManagerFeatures();
+		}
+
+		@Test
+		void shouldReturnFalseIfFeaturesAreEmpty() {
+			when(remoteService.getBescheidManagerFeatures()).thenReturn(Optional.empty());
+
+			var canCreate = service.canCreateBescheidDocumentAutomatically();
+
+			assertThat(canCreate).isFalse();
 		}
 
 		@ParameterizedTest
 		@ValueSource(booleans = { true, false })
-		void shouldRetrun(boolean canCreateBescheidDocument) {
-			when(remoteService.canCreateBescheidDocument()).thenReturn(canCreateBescheidDocument);
+		void shouldReturnFeatureValue(boolean featureValue) {
+			when(remoteService.getBescheidManagerFeatures()).thenReturn(
+					Optional.of(BescheidManagerFeatures.builder().canCreateBescheidDocument(featureValue).build()));
 
-			var canCreateAutomatically = service.canCreateBescheidDocumentAutomatically();
+			var canCreate = service.canCreateBescheidDocumentAutomatically();
 
-			assertThat(canCreateAutomatically).isEqualTo(canCreateBescheidDocument);
+			assertThat(canCreate).isEqualTo(featureValue);
 		}
 	}