test(recruit-board): 시간 검증 로직 변경 테스트

 - Service 계층에서 Clock 사용

Author: leebs0521

src/test/java/com/somemore/domains/recruitboard/controller/RecruitBoardCommandApiControllerTest.java

@@ -150,7 +150,7 @@ void updateRecruitBoard() throws Exception {
 
         given(imageUploadUseCase.uploadImage(any())).willReturn(mockImageUrl);
         willDoNothing().given(updateRecruitBoardUseCase)
-                .updateRecruitBoard(any(), any(), any(UUID.class), anyString(), any());
+                .updateRecruitBoard(any(), any(), any(UUID.class), anyString());
 
         MockMultipartHttpServletRequestBuilder builder = multipart("/api/recruit-board/{id}", 1);
         builder.with(new RequestPostProcessor() {
@@ -187,7 +187,7 @@ void updateRecruitBoardLocation() throws Exception {
                 .build();
 
         willDoNothing().given(updateRecruitBoardUseCase)
-                .updateRecruitBoardLocation(any(), any(), any(UUID.class), any());
+                .updateRecruitBoardLocation(any(), any(), any(UUID.class));
 
         String requestBody = objectMapper.writeValueAsString(requestDto);
 
@@ -213,7 +213,7 @@ void updateRecruitBoardStatus() throws Exception {
                 status);
         String requestBody = objectMapper.writeValueAsString(dto);
         willDoNothing().given(updateRecruitBoardUseCase)
-                .updateRecruitBoardStatus(any(), any(), any(UUID.class), any(LocalDateTime.class));
+                .updateRecruitBoardStatus(any(), any(), any(UUID.class));
 
         // when
         mockMvc.perform(patch("/api/recruit-board/{id}", 1L)

src/test/java/com/somemore/domains/recruitboard/service/UpdateRecruitBoardServiceTest.java

@@ -1,5 +1,14 @@
 package com.somemore.domains.recruitboard.service;
 
+import static com.somemore.domains.recruitboard.domain.RecruitStatus.RECRUITING;
+import static com.somemore.domains.recruitboard.domain.VolunteerCategory.ADMINISTRATIVE_SUPPORT;
+import static com.somemore.domains.recruitboard.domain.VolunteerCategory.OTHER;
+import static com.somemore.support.fixture.LocalDateTimeFixture.createCurrentDateTime;
+import static com.somemore.support.fixture.LocalDateTimeFixture.createStartDateTime;
+import static com.somemore.support.fixture.LocalDateTimeFixture.createUpdateStartDateTime;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Mockito.doReturn;
+
 import com.somemore.domains.location.domain.Location;
 import com.somemore.domains.location.repository.LocationRepository;
 import com.somemore.domains.recruitboard.domain.RecruitBoard;
@@ -10,22 +19,18 @@
 import com.somemore.domains.recruitboard.repository.RecruitBoardJpaRepository;
 import com.somemore.domains.recruitboard.repository.RecruitBoardRepository;
 import com.somemore.support.IntegrationTestSupport;
+import java.math.BigDecimal;
+import java.time.Clock;
+import java.time.LocalDateTime;
+import java.time.ZoneId;
+import java.util.UUID;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.mock.mockito.SpyBean;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.math.BigDecimal;
-import java.time.LocalDateTime;
-import java.util.UUID;
-
-import static com.somemore.domains.recruitboard.domain.RecruitStatus.RECRUITING;
-import static com.somemore.domains.recruitboard.domain.VolunteerCategory.ADMINISTRATIVE_SUPPORT;
-import static com.somemore.domains.recruitboard.domain.VolunteerCategory.OTHER;
-import static com.somemore.support.fixture.LocalDateTimeFixture.*;
-import static org.assertj.core.api.Assertions.assertThat;
-
 @Transactional
 class UpdateRecruitBoardServiceTest extends IntegrationTestSupport {
 
@@ -41,7 +46,11 @@ class UpdateRecruitBoardServiceTest extends IntegrationTestSupport {
     @Autowired
     private LocationRepository locationRepository;
 
+    @SpyBean
+    private Clock clock;
+
     private RecruitBoard recruitBoard;
+
     private UUID centerId;
 
     @BeforeEach
@@ -73,8 +82,11 @@ void updateRecruitBoard() {
                 .admitted(false)
                 .build();
 
+        setMockClock(current);
+
         // when
-        updateRecruitBoardService.updateRecruitBoard(dto, recruitBoard.getId(), centerId, newImgUrl, current);
+        updateRecruitBoardService.updateRecruitBoard(dto, recruitBoard.getId(), centerId,
+                newImgUrl);
 
         // then
         RecruitBoard updatedRecruitBoard = recruitBoardRepository.findById(recruitBoard.getId())
@@ -107,8 +119,10 @@ void updateRecruitBoardLocation() {
                 .longitude(BigDecimal.valueOf(127.2222222))
                 .build();
 
+        setMockClock(current);
+
         // when
-        updateRecruitBoardService.updateRecruitBoardLocation(dto, recruitBoard.getId(), centerId, current);
+        updateRecruitBoardService.updateRecruitBoardLocation(dto, recruitBoard.getId(), centerId);
 
         // then
         RecruitBoard updateRecruitBoard = recruitBoardRepository.findById(recruitBoard.getId())
@@ -130,10 +144,12 @@ void updateRecruitBoardStatus() {
         // given
         Long recruitBoardId = recruitBoard.getId();
         RecruitStatus newStatus = RecruitStatus.CLOSED;
-        LocalDateTime currentDateTime = createCurrentDateTime();
+        LocalDateTime current = createCurrentDateTime();
+
+        setMockClock(current);
 
         // when
-        updateRecruitBoardService.updateRecruitBoardStatus(newStatus, recruitBoardId, centerId, currentDateTime);
+        updateRecruitBoardService.updateRecruitBoardStatus(newStatus, recruitBoardId, centerId);
 
         // then
         RecruitBoard findBoard = recruitBoardRepository.findById(recruitBoardId).orElseThrow();
@@ -147,7 +163,8 @@ private static RecruitBoard createRecruitBoard(UUID centerId, Long locationId) {
         return createRecruitBoard(centerId, locationId, startDateTime, endDateTime);
     }
 
-    private static RecruitBoard createRecruitBoard(UUID centerId, Long locationId, LocalDateTime startDateTime, LocalDateTime endDateTime) {
+    private static RecruitBoard createRecruitBoard(UUID centerId, Long locationId,
+            LocalDateTime startDateTime, LocalDateTime endDateTime) {
 
         RecruitmentInfo recruitmentInfo = RecruitmentInfo.builder()
                 .region("경기")
@@ -177,4 +194,10 @@ private static Location createLocation() {
                 .latitude(BigDecimal.valueOf(127.11111))
                 .build();
     }
+
+    private void setMockClock(LocalDateTime current) {
+        doReturn(current.atZone(ZoneId.systemDefault()).toInstant())
+                .when(clock)
+                .instant();
+    }
 }