test: HitTrackerService.tomorrow() 테스트 코드

build.gradle

@@ -35,7 +35,7 @@ dependencies {
 	implementation 'com.fasterxml.jackson.core:jackson-databind'
 	implementation 'org.springframework.boot:spring-boot-starter-data-redis'
 	implementation 'org.springdoc:springdoc-openapi-starter-webmvc-ui:2.0.2'
-	implementation ('it.ozimov:embedded-redis:0.7.3') { exclude group: "org.slf4j", module: "slf4j-simple" }
+	testImplementation 'com.github.codemonstur:embedded-redis:1.0.0'
 	testImplementation 'org.junit.jupiter:junit-jupiter-api:5.10.0'
 	testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine'
 }

src/main/java/it/numble/hittracker/common/config/AppConfig.java

@@ -5,6 +5,8 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 
+import java.time.Clock;
+
 @Configuration
 public class AppConfig {
 
@@ -14,4 +16,9 @@ public ObjectMapper objectMapper() {
         objectMapper.setSerializationInclusion(JsonInclude.Include.NON_NULL);
         return objectMapper;
     }
+
+    @Bean
+    public Clock clock() {
+        return Clock.systemDefaultZone();
+    }
 }

src/main/java/it/numble/hittracker/scheduler/DailyHitLogScheduler.java

@@ -4,17 +4,21 @@
 import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Component;
 
+import java.time.Clock;
+
 @Component
 public class DailyHitLogScheduler {
 
     private final HitTrackerService hitTrackerService;
+    private final Clock clock;
 
-    public DailyHitLogScheduler(HitTrackerService hitTrackerService) {
+    public DailyHitLogScheduler(HitTrackerService hitTrackerService, Clock clock) {
         this.hitTrackerService = hitTrackerService;
+        this.clock = clock;
     }
 
     @Scheduled(cron = "0 0 0 * * ?")
     public void logDailyHit() {
-        hitTrackerService.tomorrow();
+        hitTrackerService.tomorrow(clock);
     }
 }

src/main/java/it/numble/hittracker/service/HitTrackerService.java

@@ -6,6 +6,8 @@
 import it.numble.hittracker.repository.DailyHitLogRepository;
 import it.numble.hittracker.repository.HitRepository;
 import it.numble.hittracker.repository.UrlRepository;
+
+import java.time.Clock;
 import java.time.LocalDate;
 import java.util.Comparator;
 import java.util.List;
@@ -43,15 +45,15 @@ public List<DailyHitLog> getStatistics(String url) {
         return dailyHitLogRepository.findAllByUrl(savedUrl);
     }
 
-    public void tomorrow() {
+    public void tomorrow(Clock clock) {
         List<DailyHitLog> logs = dailyHitLogRepository.findAll();
-        LocalDate today = LocalDate.now().minusDays(1);
+        LocalDate today = LocalDate.now(clock).minusDays(1);
 
         // 일간조회수 모두 0으로 초기화 및 저장
         List<Url> urls = urlRepository.findAll();
         for (Url url : urls) {
             DailyHitLog dailyHitLog = new DailyHitLog(today, hitRepository.getTodayHit(url.getUrl()), url);
-            DailyHitLog savedDailyHitLog = dailyHitLogRepository.save(dailyHitLog);
+            dailyHitLogRepository.save(dailyHitLog);
 
             hitRepository.deleteTodayHit(url.getUrl());
         }

src/test/java/it/numble/hittracker/service/HitTrackerServiceTests.java

@@ -17,10 +17,11 @@
 import org.springframework.boot.test.context.TestConfiguration;
 import redis.embedded.RedisServer;
 
-import java.time.LocalDate;
+import java.io.IOException;
+import java.time.*;
 import java.util.List;
 
-import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
+import static org.assertj.core.api.Assertions.assertThat;
 
 @SpringBootTest(classes = TestRedisConfiguration.class)
 @Transactional
@@ -84,24 +85,56 @@ void getStatistics() {
         assertThat(dailyHitLogs.size()).isEqualTo(1);
         assertThat(dailyHitLogs.get(0)).isEqualTo(savedLog);
     }
+
+    @Test
+    void tomorrow() {
+        // given
+        String url = "http://test.com";
+        Url savedUrl = urlRepository.save(new Url(url));
+        hitRepository.hit(url);
+
+        LocalDate NOW = LocalDate.of(2020, 1, 9);
+        ZoneId zone = ZoneOffset.systemDefault();
+        Instant fixedInstant = NOW.atStartOfDay(zone).toInstant();
+        Clock mockClock = Clock.fixed(fixedInstant, zone);
+
+        DailyHitLog givenToBeDeleted = dailyHitLogRepository.save(new DailyHitLog(LocalDate.of(2020, 1, 1), 0, savedUrl));
+        DailyHitLog givenToBeRemaining = dailyHitLogRepository.save(new DailyHitLog(LocalDate.of(2020, 1, 2), 0, savedUrl));
+
+        // when
+        hitTrackerService.tomorrow(mockClock);
+
+        // then
+        assertThat(hitRepository.getTodayHit(url)).isEqualTo(0);
+        assertThat(hitRepository.getTotalHit(url)).isEqualTo(1);
+
+        List<DailyHitLog> dailyHitLogs = dailyHitLogRepository.findAllByUrl(urlRepository.findByUrl(url).orElseThrow());
+        assertThat(dailyHitLogs.size()).isEqualTo(2);
+        assertThat(dailyHitLogs).contains(givenToBeRemaining);
+        assertThat(dailyHitLogs).doesNotContain(givenToBeDeleted);
+
+        DailyHitLog newLog = dailyHitLogs.stream().filter(log -> log != givenToBeRemaining).findFirst().orElseThrow();
+        assertThat(newLog.getDailyHit()).isEqualTo(1);
+        assertThat(newLog.getDate()).isEqualTo(NOW.minusDays(1));   
+    }
 }
 
 @TestConfiguration
 class TestRedisConfiguration {
 
     private final RedisServer redisServer;
 
-    public TestRedisConfiguration(RedisProperties redisProperties) {
+    public TestRedisConfiguration(RedisProperties redisProperties) throws IOException {
         this.redisServer = new RedisServer(redisProperties.getRedisPort());
     }
 
     @PostConstruct
-    public void postConstruct() {
+    public void postConstruct() throws IOException {
         redisServer.start();
     }
 
     @PreDestroy
-    public void preDestroy() {
+    public void preDestroy() throws IOException {
         redisServer.stop();
     }
 }