fix:

* changed from email sending to logging the errors

src/main/java/de/caritas/cob/userservice/api/workflow/delete/service/DeleteInactiveSessionsAndUserService.java

@@ -30,7 +30,7 @@ public class DeleteInactiveSessionsAndUserService {
   private final @NonNull UserRepository userRepository;
   private final @NonNull SessionRepository sessionRepository;
   private final @NonNull DeleteUserAccountService deleteUserAccountService;
-  private final @NonNull WorkflowErrorMailService workflowErrorMailService;
+  private final @NonNull WorkflowErrorLogService workflowErrorLogService;
   private final @NonNull DeleteSessionService deleteSessionService;
   private final @NonNull InactivePrivateGroupsProvider inactivePrivateGroupsProvider;
 
@@ -49,12 +49,12 @@ public void deleteInactiveSessionsAndUsers() {
             .flatMap(Collection::stream)
             .collect(Collectors.toList());
 
-    sendWorkflowErrorsMail(workflowErrors);
+    logWorkflowErrors(workflowErrors);
   }
 
-  private void sendWorkflowErrorsMail(List<DeletionWorkflowError> workflowErrors) {
+  private void logWorkflowErrors(List<DeletionWorkflowError> workflowErrors) {
     if (isNotEmpty(workflowErrors)) {
-      this.workflowErrorMailService.buildAndSendErrorMail(workflowErrors);
+      this.workflowErrorLogService.logWorkflowErrors(workflowErrors);
     }
   }
 

src/main/java/de/caritas/cob/userservice/api/workflow/delete/service/WorkflowErrorLogService.java

@@ -0,0 +1,35 @@
+package de.caritas.cob.userservice.api.workflow.delete.service;
+
+import static org.apache.commons.collections4.CollectionUtils.isNotEmpty;
+
+import de.caritas.cob.userservice.api.workflow.delete.model.DeletionWorkflowError;
+import java.util.List;
+import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
+import org.springframework.stereotype.Service;
+
+/** Service class to log all deletion workflow errors. */
+@Slf4j
+@Service
+@RequiredArgsConstructor
+public class WorkflowErrorLogService {
+
+  public void logWorkflowErrors(List<DeletionWorkflowError> workflowErrors) {
+    if (isNotEmpty(workflowErrors)) {
+      workflowErrors.forEach(
+          workflowError ->
+              log.warn(
+                  "Errors during deletion workflow:"
+                      + " SourceType = {}; "
+                      + "TargetType = {}; "
+                      + "Identifier = {}; "
+                      + "Reason = {}; "
+                      + "Timestamp = {}.",
+                  workflowError.getDeletionSourceType(),
+                  workflowError.getDeletionTargetType(),
+                  workflowError.getIdentifier(),
+                  workflowError.getReason(),
+                  workflowError.getTimestamp()));
+    }
+  }
+}

src/test/java/de/caritas/cob/userservice/api/workflow/delete/service/DeleteInactiveSessionsAndUserServiceTest.java

@@ -30,7 +30,7 @@ public class DeleteInactiveSessionsAndUserServiceTest {
 
   @InjectMocks private DeleteInactiveSessionsAndUserService deleteInactiveSessionsAndUserService;
 
-  @Mock private WorkflowErrorMailService workflowErrorMailService;
+  @Mock private WorkflowErrorLogService workflowErrorLogService;
   @Mock private UserRepository userRepository;
   @Mock private SessionRepository sessionRepository;
   @Mock private DeleteUserAccountService deleteUserAccountService;
@@ -60,7 +60,7 @@ public void deleteInactiveSessionsAndUsers_Should_SendWorkflowErrorsMail() {
 
     deleteInactiveSessionsAndUserService.deleteInactiveSessionsAndUsers();
 
-    verify(workflowErrorMailService, Mockito.times(1)).buildAndSendErrorMail(any());
+    verify(workflowErrorLogService, Mockito.times(1)).logWorkflowErrors(any());
   }
 
   @Test
@@ -138,7 +138,7 @@ public void deleteInactiveSessionsAndUsers_Should_SendWorkflowErrorsMail() {
 
     deleteInactiveSessionsAndUserService.deleteInactiveSessionsAndUsers();
 
-    verify(workflowErrorMailService, Mockito.times(1)).buildAndSendErrorMail(any());
+    verify(workflowErrorLogService, Mockito.times(1)).logWorkflowErrors(any());
   }
 
   @Test
@@ -164,7 +164,7 @@ public void deleteInactiveSessionsAndUsers_Should_SendWorkflowErrorsMail() {
 
     deleteInactiveSessionsAndUserService.deleteInactiveSessionsAndUsers();
 
-    verify(workflowErrorMailService, Mockito.times(1)).buildAndSendErrorMail(any());
+    verify(workflowErrorLogService, Mockito.times(1)).logWorkflowErrors(any());
   }
 
   @Test
@@ -187,6 +187,6 @@ public void deleteInactiveSessionsAndUsers_Should_SendWorkflowErrorsMail() {
 
     deleteInactiveSessionsAndUserService.deleteInactiveSessionsAndUsers();
 
-    verify(workflowErrorMailService, Mockito.times(1)).buildAndSendErrorMail(any());
+    verify(workflowErrorLogService, Mockito.times(1)).logWorkflowErrors(any());
   }
 }

src/test/java/de/caritas/cob/userservice/api/workflow/delete/service/WorkflowErrorLogServiceTest.java

@@ -0,0 +1,76 @@
+package de.caritas.cob.userservice.api.workflow.delete.service;
+
+import static de.caritas.cob.userservice.api.helper.CustomLocalDateTime.nowInUtc;
+import static de.caritas.cob.userservice.api.workflow.delete.model.DeletionSourceType.ASKER;
+import static de.caritas.cob.userservice.api.workflow.delete.model.DeletionTargetType.ROCKET_CHAT;
+import static java.util.Collections.emptyList;
+
+import ch.qos.logback.classic.Logger;
+import ch.qos.logback.classic.spi.ILoggingEvent;
+import ch.qos.logback.core.read.ListAppender;
+import de.caritas.cob.userservice.api.workflow.delete.model.DeletionWorkflowError;
+import java.util.ArrayList;
+import java.util.List;
+import org.assertj.core.api.Assertions;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.InjectMocks;
+import org.mockito.junit.MockitoJUnitRunner;
+import org.slf4j.LoggerFactory;
+
+@RunWith(MockitoJUnitRunner.class)
+public class WorkflowErrorLogServiceTest {
+
+  @InjectMocks private WorkflowErrorLogService workflowErrorLogService;
+
+  private ListAppender<ILoggingEvent> listAppender;
+
+  @Before
+  public void setUp() {
+    Logger logger = (Logger) LoggerFactory.getLogger(WorkflowErrorLogService.class);
+    listAppender = new ListAppender<>();
+    listAppender.start();
+    logger.addAppender(listAppender);
+  }
+
+  @Test
+  public void logWorkflowErrors_Should_logNoErrors_When_workflowErrorsAreNull() {
+
+    // when
+    this.workflowErrorLogService.logWorkflowErrors(null);
+
+    // then
+    Assertions.assertThat(listAppender.list).isEmpty();
+  }
+
+  @Test
+  public void logWorkflowErrors_Should_logNoErrors_When_workflowErrorsAreEmpty() {
+
+    // when
+    this.workflowErrorLogService.logWorkflowErrors(emptyList());
+
+    // then
+    Assertions.assertThat(listAppender.list).isEmpty();
+  }
+
+  @Test
+  public void logWorkflowErrors_Should_logErrors_When_workflowErrorsExists() {
+    // given
+    List<DeletionWorkflowError> workflowErrors = new ArrayList<>();
+    workflowErrors.add(
+        DeletionWorkflowError.builder()
+            .deletionSourceType(ASKER)
+            .deletionTargetType(ROCKET_CHAT)
+            .timestamp(nowInUtc())
+            .reason("reason")
+            .identifier("id")
+            .build());
+
+    // when
+    this.workflowErrorLogService.logWorkflowErrors(workflowErrors);
+
+    // then
+    Assertions.assertThat(listAppender.list).hasSize(1);
+  }
+}