CLAP-294 Refactor: 추상화 클래스 NotificationSender를 구현하도록 수정

<footer>
- 관련: #350

Author: joowojr

src/main/java/clap/server/application/port/outbound/webhook/SendAgitPort.java

@@ -4,5 +4,5 @@
 import clap.server.domain.model.task.Task;
 
 public interface SendAgitPort {
-    void sendAgit(PushNotificationTemplate request, Task task);
+    Long sendAgit(PushNotificationTemplate request, Task task);
 }

src/main/java/clap/server/application/service/history/PostCommentService.java

@@ -4,22 +4,22 @@
 import clap.server.adapter.outbound.persistense.entity.member.constant.MemberRole;
 import clap.server.adapter.outbound.persistense.entity.notification.constant.NotificationType;
 import clap.server.adapter.outbound.persistense.entity.task.constant.TaskHistoryType;
-import clap.server.application.port.inbound.history.SaveCommentAttachmentUsecase;
-import clap.server.application.port.inbound.history.SaveCommentUsecase;
 import clap.server.application.port.inbound.domain.MemberService;
 import clap.server.application.port.inbound.domain.TaskService;
+import clap.server.application.port.inbound.history.SaveCommentAttachmentUsecase;
+import clap.server.application.port.inbound.history.SaveCommentUsecase;
 import clap.server.application.port.outbound.s3.S3UploadPort;
 import clap.server.application.port.outbound.task.CommandAttachmentPort;
 import clap.server.application.port.outbound.task.CommandCommentPort;
 import clap.server.application.port.outbound.taskhistory.CommandTaskHistoryPort;
 import clap.server.application.service.webhook.SendNotificationService;
 import clap.server.common.annotation.architecture.ApplicationService;
-import clap.server.domain.policy.attachment.FilePathPolicy;
 import clap.server.domain.model.member.Member;
 import clap.server.domain.model.task.Attachment;
 import clap.server.domain.model.task.Comment;
 import clap.server.domain.model.task.Task;
 import clap.server.domain.model.task.TaskHistory;
+import clap.server.domain.policy.attachment.FilePathPolicy;
 import lombok.RequiredArgsConstructor;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.web.multipart.MultipartFile;

src/main/java/clap/server/application/service/webhook/NotificationSender.java

@@ -0,0 +1,7 @@
+package clap.server.application.service.webhook;
+
+import clap.server.adapter.outbound.api.dto.PushNotificationTemplate;
+
+public interface NotificationSender {
+    void send(PushNotificationTemplate template);
+}

src/main/java/clap/server/application/service/webhook/SendAgitService.java

@@ -1,18 +1,26 @@
 package clap.server.application.service.webhook;
 
 import clap.server.adapter.outbound.api.dto.PushNotificationTemplate;
+import clap.server.adapter.outbound.persistense.entity.notification.constant.NotificationType;
+import clap.server.application.port.inbound.domain.TaskService;
 import clap.server.application.port.outbound.webhook.SendAgitPort;
-import clap.server.common.annotation.architecture.ApplicationService;
 import clap.server.domain.model.task.Task;
 import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
 
-@ApplicationService
+@Service
 @RequiredArgsConstructor
 public class SendAgitService {
 
     private final SendAgitPort agitPort;
+    private final TaskService taskService;
 
     public void sendAgit(PushNotificationTemplate request, Task task) {
-        agitPort.sendAgit(request, task);
+        Long agitPostId = agitPort.sendAgit(request, task);
+
+        if (request.notificationType().equals(NotificationType.TASK_REQUESTED)) {
+            task.updateAgitPostId(agitPostId);
+            taskService.upsert(task);
+        }
     }
 }

src/main/java/clap/server/application/service/webhook/SendKaKaoWorkService.java

@@ -2,16 +2,16 @@
 
 import clap.server.adapter.outbound.api.dto.PushNotificationTemplate;
 import clap.server.application.port.outbound.webhook.SendKaKaoWorkPort;
-import clap.server.common.annotation.architecture.ApplicationService;
 import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
 
-@ApplicationService
+@Service
 @RequiredArgsConstructor
-public class SendKaKaoWorkService {
+public class SendKaKaoWorkService implements NotificationSender {
 
     private final SendKaKaoWorkPort sendKaKaoWorkPort;
 
-    public void sendKaKaoWork(PushNotificationTemplate request) {
+    public void send(PushNotificationTemplate request) {
         sendKaKaoWorkPort.sendKakaoWork(request);
     }
 }

src/main/java/clap/server/application/service/webhook/SendNotificationService.java

@@ -3,7 +3,6 @@
 import clap.server.adapter.inbound.web.dto.notification.request.SseRequest;
 import clap.server.adapter.outbound.api.dto.PushNotificationTemplate;
 import clap.server.adapter.outbound.persistense.entity.notification.constant.NotificationType;
-import clap.server.application.port.inbound.domain.TaskService;
 import clap.server.application.port.outbound.notification.CommandNotificationPort;
 import clap.server.application.port.outbound.webhook.SendSsePort;
 import clap.server.common.annotation.architecture.ApplicationService;
@@ -14,6 +13,7 @@
 import org.springframework.scheduling.annotation.Async;
 
 import java.util.concurrent.CompletableFuture;
+
 import static clap.server.domain.model.notification.Notification.createTaskNotification;
 
 @ApplicationService
@@ -25,7 +25,6 @@ public class SendNotificationService {
     private final SendWebhookEmailService sendWebhookEmailService;
     private final SendKaKaoWorkService sendKaKaoWorkService;
     private final CommandNotificationPort commandNotificationPort;
-    private final TaskService taskService;
 
     @Async("notificationExecutor")
     public void sendPushNotification(Member receiver, NotificationType notificationType,
@@ -83,12 +82,6 @@ public void sendAgitNotification(NotificationType notificationType,
                 message,
                 commenterName
         );
-
-        if(notificationType.equals(NotificationType.TASK_REQUESTED)){
-            Long agitPostId = sendAgitService.sendAgit(pushNotificationTemplate, task);
-
-            task.updateAgitPostId(agitPostId);
-            taskService.upsert(task);
-        }
+        sendAgitService.sendAgit(pushNotificationTemplate, task);
     }
 }

src/main/java/clap/server/application/service/webhook/SendWebhookEmailService.java

@@ -2,16 +2,16 @@
 
 import clap.server.adapter.outbound.api.dto.PushNotificationTemplate;
 import clap.server.application.port.outbound.webhook.SendWebhookEmailPort;
-import clap.server.common.annotation.architecture.ApplicationService;
 import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
 
-@ApplicationService
+@Service
 @RequiredArgsConstructor
-public class SendWebhookEmailService {
+public class SendWebhookEmailService implements NotificationSender {
 
     private final SendWebhookEmailPort sendWebhookEmailPort;
 
-    public void sendEmail(PushNotificationTemplate request) {
+    public void send(PushNotificationTemplate request) {
         sendWebhookEmailPort.sendWebhookEmail(request);
     }
 }