diff --git a/src/main/java/de/ozgcloud/admin/reporting/AggregationMappingRepository.java b/src/main/java/de/ozgcloud/admin/reporting/AggregationMappingRepository.java
index 437b5f0ee6040f6547bf6d0dc58960c48c170818..f762a367dfafb7023fbf89e87670d35470c91058 100644
--- a/src/main/java/de/ozgcloud/admin/reporting/AggregationMappingRepository.java
+++ b/src/main/java/de/ozgcloud/admin/reporting/AggregationMappingRepository.java
@@ -38,5 +38,5 @@ interface AggregationMappingRepository extends MongoRepository<AggregationMappin
List<AggregationMapping> findAll();
@Query("{}")
- List<AggregationMapping> findAllUnsecured();
+ List<AggregationMapping> findAllBypassAuth();
}
diff --git a/src/main/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoService.java b/src/main/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoService.java
index 72197f814280aa2948b06fe8c7c33bbf8398737a..a1fcc5f3225611a64ddc9020eee9334e1eee2115 100644
--- a/src/main/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoService.java
+++ b/src/main/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoService.java
@@ -40,7 +40,7 @@ class ReportingAggregationManagerSettingDtoService implements SettingDtoService
@Override
public ReportingAggregationManagerSettingDto getDataTransferObject() {
- return mapToDto(repository.findAllUnsecured());
+ return mapToDto(repository.findAllBypassAuth());
}
ReportingAggregationManagerSettingDto mapToDto(List<AggregationMapping> aggregationMappings) {
diff --git a/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingEnvironmentSettingITCase.java b/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingEnvironmentSettingITCase.java
index 197adf567c5b2dd23b8cc004dafb230a31cbfc1d..8f7ff06904aa7944a209d49a4dc649ee1a885429 100644
--- a/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingEnvironmentSettingITCase.java
+++ b/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingEnvironmentSettingITCase.java
@@ -63,7 +63,7 @@ public class AggregationMappingEnvironmentSettingITCase {
var result = getAggregationSettings();
assertThat(result.andReturn().getResponse().getContentAsString())
- .isEqualTo(PropertiesTestFactory.createAggregationProperties());
+ .isEqualToIgnoringWhitespace(PropertiesTestFactory.createAggregationProperties());
}
private ResultActions getAggregationSettings() throws Exception {
diff --git a/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingRepositoryITCase.java b/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingRepositoryITCase.java
index 9a0a9ca6e5259b3d6dc3004cedddd43a4a4e665f..1935627949511059d414374bc63efa4a9b85e605 100644
--- a/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingRepositoryITCase.java
+++ b/src/test/java/de/ozgcloud/admin/reporting/AggregationMappingRepositoryITCase.java
@@ -26,6 +26,7 @@ package de.ozgcloud.admin.reporting;
import static org.assertj.core.api.Assertions.*;
import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.data.mongodb.core.MongoOperations;
@@ -50,32 +51,37 @@ class AggregationMappingRepositoryITCase {
operations.dropCollection(AggregationMapping.COLLECTION_NAME);
}
- @Test
- @WithMockUser(roles = UserRole.DATENBEAUFTRAGUNG)
- void shouldLoadOnlyFieldMapping() {
- operations.save(AggregationMappingTestFactory.create());
- operations.save(SettingTestFactory.create());
+ @Nested
+ class TestFindAll {
+ @Test
+ @WithMockUser(roles = UserRole.DATENBEAUFTRAGUNG)
+ void shouldLoadOnlyFieldMapping() {
+ operations.save(AggregationMappingTestFactory.create());
+ operations.save(SettingTestFactory.create());
- var result = repository.findAll();
+ var result = repository.findAll();
- assertThat(result).hasSize(1);
- }
-
- @Test
- void shouldLoadWithoutRole() {
- operations.save(AggregationMappingTestFactory.create());
+ assertThat(result).hasSize(1);
+ }
- var result = repository.findAllUnsecured();
+ @Test
+ @WithMockUser
+ void shouldThrowExceptionOnMissingRole() {
+ operations.save(AggregationMappingTestFactory.create());
- assertThat(result).hasSize(1);
+ assertThatThrownBy(() -> repository.findAll()).isInstanceOf(AuthorizationDeniedException.class);
+ }
}
- @Test
- @WithMockUser
- void shouldThrowExceptionOnMissingRole() {
- operations.save(AggregationMappingTestFactory.create());
+ @Nested
+ class TestFindAllBypassAuth {
+ @Test
+ void shouldLoadWithoutRole() {
+ operations.save(AggregationMappingTestFactory.create());
- assertThatThrownBy(() -> repository.findAll()).isInstanceOf(AuthorizationDeniedException.class);
- }
+ var result = repository.findAllBypassAuth();
+ assertThat(result).hasSize(1);
+ }
+ }
}
diff --git a/src/test/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoServiceTest.java b/src/test/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoServiceTest.java
index db27eeb32d0776b378f333749c0238efd90c6515..aae1ec4b72eb4a4304a26a7d4ab0b318001379d9 100644
--- a/src/test/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoServiceTest.java
+++ b/src/test/java/de/ozgcloud/admin/reporting/ReportingAggregationManagerSettingDtoServiceTest.java
@@ -88,13 +88,13 @@ class ReportingAggregationManagerSettingDtoServiceTest {
void shouldCallRepository() {
service.getDataTransferObject();
- verify(repository).findAllUnsecured();
+ verify(repository).findAllBypassAuth();
}
@Test
void shouldCallMapToDto() {
AggregationMapping mapping = AggregationMappingTestFactory.create();
- when(repository.findAllUnsecured()).thenReturn(List.of(mapping));
+ when(repository.findAllBypassAuth()).thenReturn(List.of(mapping));
service.getDataTransferObject();
diff --git a/src/test/java/de/ozgcloud/admin/setting/PropertiesTestFactory.java b/src/test/java/de/ozgcloud/admin/setting/PropertiesTestFactory.java
index 3c149bd9bcf9a8ff46c0cc434b2f5a48ee629bb7..62df9cc7d05c095068b1bd12a0585d5ba08b8ccd 100644
--- a/src/test/java/de/ozgcloud/admin/setting/PropertiesTestFactory.java
+++ b/src/test/java/de/ozgcloud/admin/setting/PropertiesTestFactory.java
@@ -73,7 +73,7 @@ public class PropertiesTestFactory {
}
public static String createAggregationProperties() {
- return TestUtils.loadTextFile("yamlTemplates/settings/aggregation.json.tmpl",
+ return TestUtils.loadTextFile("jsonTemplates/settings/aggregation.json.tmpl",
FORM_ENGINE_NAME_1,
FORM_ID_1,
SOURCE_PATH_1_1,
diff --git a/src/test/resources/jsonTemplates/settings/aggregation.json.tmpl b/src/test/resources/jsonTemplates/settings/aggregation.json.tmpl
new file mode 100644
index 0000000000000000000000000000000000000000..ce56cc5aebc81c35d9678f6d152118217a5241a3
--- /dev/null
+++ b/src/test/resources/jsonTemplates/settings/aggregation.json.tmpl
@@ -0,0 +1,30 @@
+{
+ "name": "OzgCloud_AggregationManager",
+ "profiles": [
+ "default"
+ ],
+ "label": null,
+ "version": null,
+ "state": null,
+ "propertySources": [
+ {
+ "name": "",
+ "source": {
+ "ozgcloud.aggregation.aggregationMappings[0].formIdentifier.formEngineName": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].formIdentifier.formId": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[0].sourcePath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[0].targetPath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[1].sourcePath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[1].targetPath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[2].sourcePath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[0].fieldMappings[2].targetPath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].formIdentifier.formEngineName": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].formIdentifier.formId": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].fieldMappings[0].sourcePath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].fieldMappings[0].targetPath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].fieldMappings[1].sourcePath": "%s",
+ "ozgcloud.aggregation.aggregationMappings[1].fieldMappings[1].targetPath": "%s"
+ }
+ }
+ ]
+}
\ No newline at end of file
diff --git a/src/test/resources/yamlTemplates/settings/aggregation.json.tmpl b/src/test/resources/yamlTemplates/settings/aggregation.json.tmpl
deleted file mode 100644
index b4db7c454072b20b8eb93e8e8fc7dbcaa32d9234..0000000000000000000000000000000000000000
--- a/src/test/resources/yamlTemplates/settings/aggregation.json.tmpl
+++ /dev/null
@@ -1 +0,0 @@
-{"name":"OzgCloud_AggregationManager","profiles":["default"],"label":null,"version":null,"state":null,"propertySources":[{"name":"","source":{"ozgcloud.aggregation.aggregationMappings[0].formIdentifier.formEngineName":"%s","ozgcloud.aggregation.aggregationMappings[0].formIdentifier.formId":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[0].sourcePath":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[0].targetPath":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[1].sourcePath":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[1].targetPath":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[2].sourcePath":"%s","ozgcloud.aggregation.aggregationMappings[0].fieldMappings[2].targetPath":"%s","ozgcloud.aggregation.aggregationMappings[1].formIdentifier.formEngineName":"%s","ozgcloud.aggregation.aggregationMappings[1].formIdentifier.formId":"%s","ozgcloud.aggregation.aggregationMappings[1].fieldMappings[0].sourcePath":"%s","ozgcloud.aggregation.aggregationMappings[1].fieldMappings[0].targetPath":"%s","ozgcloud.aggregation.aggregationMappings[1].fieldMappings[1].sourcePath":"%s","ozgcloud.aggregation.aggregationMappings[1].fieldMappings[1].targetPath":"%s"}}]}
\ No newline at end of file