1132 - spotbugsMain + pmd + spotlessApply

Author: ivicac

server/libs/automation/automation-configuration/automation-configuration-rest/automation-configuration-rest-impl/src/test/java/com/bytechef/automation/configuration/web/rest/ProjectApiControllerIntTest.java

@@ -36,6 +36,7 @@
 import com.bytechef.platform.category.domain.Category;
 import com.bytechef.platform.category.service.CategoryService;
 import com.bytechef.platform.category.web.rest.model.CategoryModel;
+import com.bytechef.platform.configuration.web.rest.mapper.NotificationMapperImpl;
 import com.bytechef.platform.tag.domain.Tag;
 import com.bytechef.platform.tag.web.rest.model.TagModel;
 import com.bytechef.platform.tag.web.rest.model.UpdateTagsRequestModel;
@@ -70,6 +71,9 @@ public class ProjectApiControllerIntTest {
     @Autowired
     private MockMvc mockMvc;
 
+    @MockitoBean
+    private NotificationMapperImpl notificationMapper;
+
     @MockitoBean
     private ProjectDeploymentFacade projectDeploymentFacade;
 

server/libs/automation/automation-configuration/automation-configuration-rest/automation-configuration-rest-impl/src/test/java/com/bytechef/automation/configuration/web/rest/WorkflowApiControllerIntTest.java

@@ -33,6 +33,7 @@
 import com.bytechef.platform.configuration.dto.WorkflowTaskDTO;
 import com.bytechef.platform.configuration.facade.ComponentConnectionFacade;
 import com.bytechef.platform.configuration.facade.WorkflowFacade;
+import com.bytechef.platform.configuration.web.rest.mapper.NotificationMapperImpl;
 import java.util.List;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
@@ -68,6 +69,9 @@ public class WorkflowApiControllerIntTest {
     @Autowired
     private MockMvc mockMvc;
 
+    @MockitoBean
+    private NotificationMapperImpl notificationMapper;
+
     @MockitoBean
     private ProjectDeploymentFacade projectDeploymentFacade;
 

server/libs/platform/platform-configuration/platform-configuration-api/src/main/java/com/bytechef/platform/configuration/notification/NotificationHandlerContext.java

@@ -72,7 +72,7 @@ public String getJobName() {
 
     private NotificationHandlerContext() {
 
-    };
+    }
 
     private Event.Type eventType;
     private Long jobId;

server/libs/platform/platform-configuration/platform-configuration-api/src/main/java/com/bytechef/platform/configuration/notification/NotificationSenderRegistry.java

@@ -27,16 +27,16 @@
  * @author Matija Petanjek
  */
 @Component
-public class NotificationSenderRegistry<T extends NotificationHandler> {
-    private final Map<Notification.Type, NotificationSender<T>> notificationSenderMap;
+public class NotificationSenderRegistry {
+    private final Map<Notification.Type, NotificationSender<?>> notificationSenderMap;
 
-    public NotificationSenderRegistry(List<NotificationSender<T>> notificationSenders) {
+    public NotificationSenderRegistry(List<NotificationSender<?>> notificationSenders) {
         this.notificationSenderMap = MapUtils.toMap(
             notificationSenders, NotificationSender::getType, notificationSender -> notificationSender);
     }
 
     @NonNull
-    public NotificationSender<T> getNotificationSender(Notification.Type type) {
+    public NotificationSender<?> getNotificationSender(Notification.Type type) {
         return notificationSenderMap.get(type);
     }
 }

server/libs/platform/platform-configuration/platform-configuration-service/src/test/java/com/bytechef/platform/configuration/service/NotificationServiceIntTest.java

@@ -23,19 +23,24 @@
 import com.bytechef.platform.configuration.domain.Notification;
 import com.bytechef.platform.configuration.repository.EventRepository;
 import com.bytechef.platform.configuration.repository.NotificationRepository;
+import com.bytechef.platform.mail.MailService;
 import java.util.List;
 import java.util.Map;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 /**
  * @author Matija Petanjek
  */
 @SpringBootTest(classes = NotificationIntTestConfiguration.class)
 public class NotificationServiceIntTest {
 
+    @MockitoBean
+    private MailService mailService;
+
     @Autowired
     private EventRepository eventRepository;
 

server/libs/platform/platform-coordinator/src/main/java/com/bytechef/platform/coordinator/event/listener/JobStatusApplicationEventListener.java

@@ -29,21 +29,22 @@
 import com.bytechef.platform.configuration.notification.NotificationSender;
 import com.bytechef.platform.configuration.notification.NotificationSenderRegistry;
 import com.bytechef.platform.configuration.service.NotificationService;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.util.List;
 import org.springframework.stereotype.Component;
 
 /**
  * @author Matija Petanjek
  */
 @Component
-@SuppressWarnings("unchecked")
 public class JobStatusApplicationEventListener implements ApplicationEventListener {
 
     private final JobService jobService;
     private final NotificationHandlerRegistry notificationHandlerRegistry;
     private final NotificationSenderRegistry notificationSenderRegistry;
     private final NotificationService notificationService;
 
+    @SuppressFBWarnings("EI")
     public JobStatusApplicationEventListener(
         JobService jobService, NotificationHandlerRegistry notificationHandlerRegistry,
         NotificationSenderRegistry notificationSenderRegistry, NotificationService notificationService) {
@@ -55,6 +56,9 @@ public JobStatusApplicationEventListener(
 
     }
 
+    @SuppressWarnings({
+        "rawtypes", "unchecked"
+    })
     public void onApplicationEvent(ApplicationEvent applicationEvent) {
         if (applicationEvent instanceof JobStatusApplicationEvent jobStatusApplicationEvent) {