Connecta 163

pom.xml

@@ -33,7 +33,6 @@
     <commons-lang3.version>3.11</commons-lang3.version>
     <liquibase-maven-plugin.version>4.23.2</liquibase-maven-plugin.version>
     <h2.version>1.4.200</h2.version>
-    <powermock-module-junit4.version>2.0.9</powermock-module-junit4.version>
     <ehcache.version>2.10.6</ehcache.version>
     <easy-random-core.version>5.0.0</easy-random-core.version>
     <spring-boot-autoconfigure.version>3.0.6</spring-boot-autoconfigure.version>

src/main/java/de/caritas/cob/agencyservice/api/admin/service/agency/AgencyTopicEnrichmentService.java

@@ -5,6 +5,7 @@
 import de.caritas.cob.agencyservice.api.repository.agency.Agency;
 import de.caritas.cob.agencyservice.api.repository.agencytopic.AgencyTopic;
 import de.caritas.cob.agencyservice.api.service.TopicService;
+import de.caritas.cob.agencyservice.api.tenant.TenantContext;
 import java.util.List;
 import java.util.Map;
 import java.util.stream.Collectors;

src/main/java/de/caritas/cob/agencyservice/api/service/TopicCachingService.java

@@ -0,0 +1,37 @@
+package de.caritas.cob.agencyservice.api.service;
+
+import de.caritas.cob.agencyservice.api.service.securityheader.SecurityHeaderSupplier;
+import de.caritas.cob.agencyservice.config.CacheManagerConfig;
+import de.caritas.cob.agencyservice.config.apiclient.TopicServiceApiControllerFactory;
+import de.caritas.cob.agencyservice.topicservice.generated.ApiClient;
+import de.caritas.cob.agencyservice.topicservice.generated.web.TopicControllerApi;
+import de.caritas.cob.agencyservice.topicservice.generated.web.model.TopicDTO;
+import java.util.List;
+import lombok.NonNull;
+import lombok.RequiredArgsConstructor;
+import org.springframework.cache.annotation.Cacheable;
+import org.springframework.stereotype.Service;
+
+@Service
+@RequiredArgsConstructor
+public class TopicCachingService {
+
+  private final @NonNull TopicServiceApiControllerFactory topicServiceApiControllerFactory;
+  private final @NonNull SecurityHeaderSupplier securityHeaderSupplier;
+  private final @NonNull TenantHeaderSupplier tenantHeaderSupplier;
+
+  @Cacheable(cacheNames = CacheManagerConfig.TOPICS_CACHE)
+  public List<TopicDTO> getAllTopics(Long tenantId) {
+    // tenantId parameter is just used for caching per tenant, it is not required for the actual implementation, because it will be added by the tenantHeaderSupplier
+    TopicControllerApi controllerApi = topicServiceApiControllerFactory.createControllerApi();
+    addDefaultHeaders(controllerApi.getApiClient());
+    return controllerApi.getAllTopics();
+  }
+
+  private void addDefaultHeaders(ApiClient apiClient) {
+    var headers = this.securityHeaderSupplier.getKeycloakAndCsrfHttpHeaders();
+    tenantHeaderSupplier.addTenantHeader(headers);
+    headers.forEach((key, value) -> apiClient.addDefaultHeader(key, value.iterator().next()));
+  }
+
+}

src/main/java/de/caritas/cob/agencyservice/api/service/TopicService.java

@@ -1,35 +1,22 @@
 package de.caritas.cob.agencyservice.api.service;
 
-import de.caritas.cob.agencyservice.api.service.securityheader.SecurityHeaderSupplier;
-import de.caritas.cob.agencyservice.config.CacheManagerConfig;
-import de.caritas.cob.agencyservice.config.apiclient.TopicServiceApiControllerFactory;
-import de.caritas.cob.agencyservice.topicservice.generated.web.TopicControllerApi;
+import de.caritas.cob.agencyservice.api.tenant.TenantContext;
+
 import java.util.List;
 import lombok.NonNull;
 import lombok.RequiredArgsConstructor;
-import org.springframework.cache.annotation.Cacheable;
 import org.springframework.stereotype.Service;
 import de.caritas.cob.agencyservice.topicservice.generated.web.model.TopicDTO;
-import de.caritas.cob.agencyservice.topicservice.generated.ApiClient;
 
 @Service
 @RequiredArgsConstructor
 public class TopicService {
 
-  private final @NonNull TopicServiceApiControllerFactory topicServiceApiControllerFactory;
-  private final @NonNull SecurityHeaderSupplier securityHeaderSupplier;
-  private final @NonNull TenantHeaderSupplier tenantHeaderSupplier;
+  private final @NonNull TopicCachingService topicCachingService;
 
-  public List<TopicDTO> getAllTopics() {
-    TopicControllerApi controllerApi = topicServiceApiControllerFactory.createControllerApi();
-    addDefaultHeaders(controllerApi.getApiClient());
-    return controllerApi.getAllTopics();
-  }
 
-  private void addDefaultHeaders(ApiClient apiClient) {
-    var headers = this.securityHeaderSupplier.getKeycloakAndCsrfHttpHeaders();
-    tenantHeaderSupplier.addTenantHeader(headers);
-    headers.forEach((key, value) -> apiClient.addDefaultHeader(key, value.iterator().next()));
+  public List<TopicDTO> getAllTopics() {
+    return topicCachingService.getAllTopics(TenantContext.getCurrentTenant());
   }
 
 }

src/test/java/de/caritas/cob/agencyservice/api/admin/service/AgencyAdminServiceTenantAwareIT.java

@@ -21,6 +21,7 @@
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
+import org.mockito.Mockito;
 import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
 import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase.Replace;
 import org.springframework.boot.test.context.SpringBootTest;

src/test/java/de/caritas/cob/agencyservice/api/admin/service/agency/AgencyTopicEnrichmentServiceTest.java

@@ -13,6 +13,7 @@
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import de.caritas.cob.agencyservice.topicservice.generated.web.model.TopicDTO;
+import org.mockito.Mockito;
 import org.mockito.junit.jupiter.MockitoExtension;
 
 @ExtendWith(MockitoExtension.class)

src/test/java/de/caritas/cob/agencyservice/api/service/TopicCachingServiceTest.java

@@ -0,0 +1,59 @@
+package de.caritas.cob.agencyservice.api.service;
+
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Mockito.when;
+
+import com.google.common.collect.Lists;
+
+import de.caritas.cob.agencyservice.AgencyServiceApplication;
+import de.caritas.cob.agencyservice.api.service.securityheader.SecurityHeaderSupplier;
+import de.caritas.cob.agencyservice.config.apiclient.TopicServiceApiControllerFactory;
+
+import org.junit.jupiter.api.Test;
+import org.mockito.Mockito;
+import de.caritas.cob.agencyservice.topicservice.generated.web.model.TopicDTO;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
+import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase.Replace;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.http.HttpHeaders;
+import org.springframework.test.context.TestPropertySource;
+
+
+@SpringBootTest(classes = AgencyServiceApplication.class)
+@TestPropertySource(properties = "spring.profiles.active=testing")
+@AutoConfigureTestDatabase(replace = Replace.ANY)
+class TopicCachingServiceTest {
+
+  @MockBean
+  TopicServiceApiControllerFactory topicServiceApiControllerFactory;
+
+  @MockBean SecurityHeaderSupplier securityHeaderSupplier;
+
+  @MockBean TenantHeaderSupplier tenantHeaderSupplier;
+
+  @MockBean
+  de.caritas.cob.agencyservice.topicservice.generated.web.TopicControllerApi topicControllerApi;
+
+  @Autowired
+  TopicCachingService topicCachingService;
+
+  @Test
+  void getAllTopics_Should_CallTopicControllerApiToGetTopics_AndCacheResponsePerTenant() {
+    // given
+    when(securityHeaderSupplier.getKeycloakAndCsrfHttpHeaders()).thenReturn(new HttpHeaders());
+    when(topicServiceApiControllerFactory.createControllerApi()).thenReturn(topicControllerApi);
+    var expectedTopics = Lists.newArrayList(new TopicDTO(), new TopicDTO());
+    when(topicControllerApi.getAllTopics()).thenReturn(expectedTopics);
+
+    // when
+    var actualTopics = topicCachingService.getAllTopics(1L);
+    topicCachingService.getAllTopics(1L);
+
+    // then
+    assertThat(actualTopics).isEqualTo(expectedTopics);
+    Mockito.verify(topicControllerApi, Mockito.times(1)).getAllTopics();
+  }
+}

src/test/java/de/caritas/cob/agencyservice/api/service/TopicServiceTest.java

@@ -0,0 +1,28 @@
+package de.caritas.cob.agencyservice.api.service;
+
+import static org.mockito.Mockito.verify;
+
+import de.caritas.cob.agencyservice.api.tenant.TenantContext;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+@ExtendWith(MockitoExtension.class)
+class TopicServiceTest {
+
+  @Mock
+  private TopicCachingService topicCachingService;
+
+  @Test
+  void getTopics_Should_CallTopicCachingService() {
+    // given
+    TopicService topicService = new TopicService(topicCachingService);
+
+    // when
+    topicService.getAllTopics();
+
+    // then
+    verify(topicCachingService).getAllTopics(TenantContext.getCurrentTenant());
+  }
+}