feat(VolunteerApplyStatusChangeEvent): 거절 이벤트 처리

- 이전 상태와 현 상태 비교 검증.
- 중복 코드 최소화.

Author: m-a-king

src/main/java/com/somemore/volunteerapply/event/VolunteerApplyStatusChangeEvent.java

@@ -5,7 +5,9 @@
 import com.somemore.global.common.event.ServerEvent;
 import com.somemore.global.common.event.ServerEventType;
 import com.somemore.notification.domain.NotificationSubType;
+import com.somemore.recruitboard.domain.RecruitBoard;
 import com.somemore.volunteerapply.domain.ApplyStatus;
+import com.somemore.volunteerapply.domain.VolunteerApply;
 import lombok.Getter;
 import lombok.experimental.SuperBuilder;
 
@@ -40,4 +42,18 @@ public VolunteerApplyStatusChangeEvent(
         this.oldStatus = oldStatus;
         this.newStatus = newStatus;
     }
+
+    public static VolunteerApplyStatusChangeEvent from(VolunteerApply apply, RecruitBoard recruitBoard, ApplyStatus oldStatus) {
+
+        return VolunteerApplyStatusChangeEvent.builder()
+                .type(ServerEventType.NOTIFICATION)
+                .subType(NotificationSubType.VOLUNTEER_APPLY_STATUS_CHANGE)
+                .volunteerId(apply.getVolunteerId())
+                .volunteerApplyId(apply.getId())
+                .centerId(recruitBoard.getCenterId())
+                .recruitBoardId(recruitBoard.getId())
+                .oldStatus(oldStatus)
+                .newStatus(apply.getStatus())
+                .build();
+    }
 }

src/main/java/com/somemore/volunteerapply/service/ApproveVolunteerApplyService.java

@@ -1,9 +1,7 @@
 package com.somemore.volunteerapply.service;
 
 import com.somemore.global.common.event.ServerEventPublisher;
-import com.somemore.global.common.event.ServerEventType;
 import com.somemore.global.exception.BadRequestException;
-import com.somemore.notification.domain.NotificationSubType;
 import com.somemore.recruitboard.domain.RecruitBoard;
 import com.somemore.recruitboard.usecase.query.RecruitBoardQueryUseCase;
 import com.somemore.volunteerapply.domain.ApplyStatus;
@@ -65,18 +63,14 @@ private void validateBoardStatus(RecruitBoard recruitBoard) {
         }
     }
 
-    private void publishVolunteerApplyStatusChangeEvent(VolunteerApply apply, RecruitBoard recruitBoard, ApplyStatus oldStatus) {
-        VolunteerApplyStatusChangeEvent event = VolunteerApplyStatusChangeEvent.builder()
-                .type(ServerEventType.NOTIFICATION)
-                .subType(NotificationSubType.VOLUNTEER_APPLY_STATUS_CHANGE)
-                .volunteerId(apply.getVolunteerId())
-                .volunteerApplyId(apply.getId())
-                .centerId(recruitBoard.getCenterId())
-                .recruitBoardId(recruitBoard.getId())
-                .oldStatus(oldStatus)
-                .newStatus(apply.getStatus())
-                .build();
+    private void publishVolunteerApplyStatusChangeEvent(VolunteerApply apply,
+                                                        RecruitBoard recruitBoard,
+                                                        ApplyStatus oldStatus) {
 
-        serverEventPublisher.publish(event);
+        if (apply.getStatus() == oldStatus) {
+            return;
+        }
+
+        serverEventPublisher.publish(VolunteerApplyStatusChangeEvent.from(apply, recruitBoard, oldStatus));
     }
 }

src/main/java/com/somemore/volunteerapply/service/RejectVolunteerApplyService.java

@@ -5,10 +5,13 @@
 import static com.somemore.global.exception.ExceptionMessage.UNAUTHORIZED_RECRUIT_BOARD;
 import static com.somemore.volunteerapply.domain.ApplyStatus.REJECTED;
 
+import com.somemore.global.common.event.ServerEventPublisher;
 import com.somemore.global.exception.BadRequestException;
 import com.somemore.recruitboard.domain.RecruitBoard;
 import com.somemore.recruitboard.usecase.query.RecruitBoardQueryUseCase;
+import com.somemore.volunteerapply.domain.ApplyStatus;
 import com.somemore.volunteerapply.domain.VolunteerApply;
+import com.somemore.volunteerapply.event.VolunteerApplyStatusChangeEvent;
 import com.somemore.volunteerapply.repository.VolunteerApplyRepository;
 import com.somemore.volunteerapply.usecase.RejectVolunteerApplyUseCase;
 import java.util.UUID;
@@ -23,20 +26,24 @@ public class RejectVolunteerApplyService implements RejectVolunteerApplyUseCase
 
     private final VolunteerApplyRepository volunteerApplyRepository;
     private final RecruitBoardQueryUseCase recruitBoardQueryUseCase;
+    private final ServerEventPublisher serverEventPublisher;
 
     @Override
     public void reject(Long id, UUID centerId) {
-        VolunteerApply apply = getApply(id);
+        VolunteerApply apply = getVolunteerApply(id);
         RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(apply.getRecruitBoardId());
 
         validateWriter(recruitBoard, centerId);
         validateBoardStatus(recruitBoard);
 
+        ApplyStatus oldStatus = apply.getStatus();
         apply.changeStatus(REJECTED);
         volunteerApplyRepository.save(apply);
+
+        publishVolunteerApplyStatusChangeEvent(apply, recruitBoard, oldStatus);
     }
 
-    private VolunteerApply getApply(Long id) {
+    private VolunteerApply getVolunteerApply(Long id) {
         return volunteerApplyRepository.findById(id).orElseThrow(
                 () -> new BadRequestException(NOT_EXISTS_VOLUNTEER_APPLY)
         );
@@ -55,4 +62,14 @@ private void validateBoardStatus(RecruitBoard recruitBoard) {
         }
     }
 
+    private void publishVolunteerApplyStatusChangeEvent(VolunteerApply apply,
+                                                        RecruitBoard recruitBoard,
+                                                        ApplyStatus oldStatus) {
+
+        if (apply.getStatus() == oldStatus) {
+            return;
+        }
+
+        serverEventPublisher.publish(VolunteerApplyStatusChangeEvent.from(apply, recruitBoard, oldStatus));
+    }
 }