refactor(VolunteerApplyStatusChangeEvent): 인자, 파라미터 정리

- oldStatus 추가.
- 이벤트에 receiverId가 포함되는 것이 어색, volunteerId로 수정.

Author: m-a-king

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

@@ -16,7 +16,7 @@
 @SuperBuilder
 public class VolunteerApplyStatusChangeEvent extends ServerEvent<NotificationSubType> {
 
-    private final UUID receiverId;
+    private final UUID volunteerId;
     private final Long volunteerApplyId;
     private final UUID centerId;
     private final Long recruitBoardId;
@@ -25,15 +25,15 @@ public class VolunteerApplyStatusChangeEvent extends ServerEvent<NotificationSub
 
     @JsonCreator
     public VolunteerApplyStatusChangeEvent(
-            @JsonProperty("receiverId") UUID receiverId,
+            @JsonProperty("receiverId") UUID volunteerId,
             @JsonProperty("volunteerApplyId") Long volunteerApplyId,
             @JsonProperty("centerId") UUID centerId,
             @JsonProperty("recruitBoardId") Long recruitBoardId,
             @JsonProperty("oldStatus") ApplyStatus oldStatus,
             @JsonProperty("newStatus") ApplyStatus newStatus
     ) {
         super(ServerEventType.NOTIFICATION, NotificationSubType.VOLUNTEER_APPLY_STATUS_CHANGE, LocalDateTime.now());
-        this.receiverId = receiverId;
+        this.volunteerId = volunteerId;
         this.volunteerApplyId = volunteerApplyId;
         this.centerId = centerId;
         this.recruitBoardId = recruitBoardId;

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

@@ -6,6 +6,7 @@
 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;
 import com.somemore.volunteerapply.domain.VolunteerApply;
 import com.somemore.volunteerapply.event.VolunteerApplyStatusChangeEvent;
 import com.somemore.volunteerapply.repository.VolunteerApplyRepository;
@@ -38,10 +39,11 @@ public void approve(Long id, UUID centerId) {
         validateWriter(recruitBoard, centerId);
         validateBoardStatus(recruitBoard);
 
+        ApplyStatus oldStatus = apply.getStatus();
         apply.changeStatus(APPROVED);
         volunteerApplyRepository.save(apply);
 
-        publishVolunteerApplyStatusChangeEvent(apply.getVolunteerId(), id, recruitBoard, apply);
+        publishVolunteerApplyStatusChangeEvent(apply, recruitBoard, oldStatus);
     }
 
     private VolunteerApply getVolunteerApply(Long id) {
@@ -63,14 +65,15 @@ private void validateBoardStatus(RecruitBoard recruitBoard) {
         }
     }
 
-    private void publishVolunteerApplyStatusChangeEvent(UUID receiverId, Long id, RecruitBoard recruitBoard, VolunteerApply apply) {
+    private void publishVolunteerApplyStatusChangeEvent(VolunteerApply apply, RecruitBoard recruitBoard, ApplyStatus oldStatus) {
         VolunteerApplyStatusChangeEvent event = VolunteerApplyStatusChangeEvent.builder()
                 .type(ServerEventType.NOTIFICATION)
                 .subType(NotificationSubType.VOLUNTEER_APPLY_STATUS_CHANGE)
-                .receiverId(receiverId)
-                .volunteerApplyId(id)
+                .volunteerId(apply.getVolunteerId())
+                .volunteerApplyId(apply.getId())
                 .centerId(recruitBoard.getCenterId())
                 .recruitBoardId(recruitBoard.getId())
+                .oldStatus(oldStatus)
                 .newStatus(apply.getStatus())
                 .build();