Stabilize schedule lifecycle CI tests

Author: jjoonleo

ontime-back/src/main/java/devkor/ontime_back/service/ScheduleService.java

@@ -12,6 +12,7 @@
 import java.time.Instant;
 import java.time.Duration;
 import java.time.LocalDateTime;
+import java.time.temporal.ChronoUnit;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -184,7 +185,7 @@ public StartScheduleResponseDto startSchedule(Long userId, UUID scheduleId) {
         assertScheduleNotFinished(schedule);
 
         if (schedule.getStartedAt() == null) {
-            schedule.startSchedule(Instant.now());
+            schedule.startSchedule(nowForPersistence());
             freezePreparationSnapshotIfNeeded(schedule);
             scheduleRepository.save(schedule);
         }
@@ -277,7 +278,7 @@ public List<LatenessHistoryResponse> getLatenessHistory(Long userId) {
     // 지각 시간 업데이트
     @Transactional
     public void updateLatenessTime(Schedule schedule, Integer latenessTime) {
-        schedule.finish(latenessTime, Instant.now());
+        schedule.finish(latenessTime, nowForPersistence());
         scheduleRepository.save(schedule);
     }
 
@@ -300,7 +301,7 @@ public void finishSchedule(Long userId, UUID scheduleId, FinishPreparationDto fi
             throw new GeneralException(SCHEDULE_NOT_STARTED);
         }
 
-        schedule.finish(finishPreparationDto.getLatenessTime(), Instant.now());
+        schedule.finish(finishPreparationDto.getLatenessTime(), nowForPersistence());
         scheduleRepository.save(schedule);
         userService.updatePunctualityScore(userId, schedule.getDoneStatus());
     }
@@ -437,4 +438,8 @@ private Integer defaultNonNegative(Integer value) {
         return value == null ? 0 : Math.max(value, 0);
     }
 
+    private Instant nowForPersistence() {
+        return Instant.now().truncatedTo(ChronoUnit.SECONDS);
+    }
+
 }

ontime-back/src/test/java/devkor/ontime_back/service/UserServiceTest.java

@@ -19,6 +19,7 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
+import static org.assertj.core.data.Offset.offset;
 import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.when;
@@ -265,9 +266,10 @@ void updatePunctualityWithAbnormalDoesNotCount(){
 
         User updatedUser = userService.updatePunctualityScore(addedUser.getId(), DoneStatus.ABNORMAL);
 
+        assertThat(updatedUser.getPunctualityScore()).isCloseTo(calculatePunctualityScore(3, 1), offset(0.0001f));
         assertThat(updatedUser)
-                .extracting("punctualityScore", "scheduleCountAfterReset", "latenessCountAfterReset")
-                .contains(calculatePunctualityScore(3, 1), 3, 1);
+                .extracting("scheduleCountAfterReset", "latenessCountAfterReset")
+                .contains(3, 1);
     }
 
     @DisplayName("성실도 점수 업데이트할 때 존재하지 않는 유저id를 인자로 넘기는 경우 예외가 발생한다.")