refact: record 조회 기준 시간 오전 0시로 + validation 추가

Author: KSH0326

src/main/java/com/back/domain/study/record/controller/StudyRecordController.java

@@ -5,6 +5,8 @@
 import com.back.domain.study.record.service.StudyRecordService;
 import com.back.global.common.dto.RsData;
 import com.back.global.security.user.CustomUserDetails;
+import io.swagger.v3.oas.annotations.Operation;
+import io.swagger.v3.oas.annotations.tags.Tag;
 import jakarta.validation.Valid;
 import lombok.RequiredArgsConstructor;
 import org.springframework.format.annotation.DateTimeFormat;
@@ -18,12 +20,15 @@
 @RestController
 @RequiredArgsConstructor
 @RequestMapping("/api/plans/records")
+@Tag(name = "StudyPlan", description = "학습 계획 관련 API")
 public class StudyRecordController {
     private final StudyRecordService studyRecordService;
 
     // ======================= 생성 ======================
     // 학습 기록 생성
     @PostMapping
+    @Operation( summary = "학습 기록 생성",
+            description = "기록을 생성합니다.")
     public ResponseEntity<RsData<StudyRecordResponseDto>> createStudyRecord(
             @AuthenticationPrincipal CustomUserDetails user,
             @Valid @RequestBody StudyRecordRequestDto request

src/main/java/com/back/domain/study/record/dto/StudyRecordRequestDto.java

@@ -1,5 +1,6 @@
 package com.back.domain.study.record.dto;
 
+import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
@@ -10,10 +11,15 @@
 @Getter
 @NoArgsConstructor
 public class StudyRecordRequestDto {
+    @NotNull(message = "계획 ID는 필수입니다.")
     private Long planId;
+    @NotNull(message = "방 ID는 필수입니다.")
     private Long roomId;
+    @NotNull(message = "시작 시간은 필수입니다.")
     private LocalDateTime startTime;
+    @NotNull(message = "종료 시간은 필수입니다.")
     private LocalDateTime endTime;
+
     private Long duration;
     private List<PauseInfoRequestDto> pauseInfos = new ArrayList<>();
 

src/main/java/com/back/domain/study/record/service/StudyRecordService.java

@@ -54,12 +54,9 @@ public StudyRecordResponseDto createStudyRecord(Long userId, StudyRecordRequestD
             throw new CustomException(ErrorCode.PLAN_FORBIDDEN);
         }
 
-        // 방 조회 (우선은 옵셔널로 설정)
-        Room room = null;
-        if (request.getRoomId() != null) {
-            room = roomRepository.findById(request.getRoomId())
-                    .orElseThrow(() -> new CustomException(ErrorCode.ROOM_NOT_FOUND));
-        }
+        // 방 조회 (필수)
+        Room room = roomRepository.findById(request.getRoomId())
+                .orElseThrow(() -> new CustomException(ErrorCode.ROOM_NOT_FOUND));
 
         // 일시정지 정보를 엔티티로 생성
         List<PauseInfo> pauseInfos = request.getPauseInfos().stream()
@@ -125,9 +122,9 @@ public List<StudyRecordResponseDto> getStudyRecordsByDate(Long userId, LocalDate
         User user = userRepository.findById(userId)
                 .orElseThrow(() -> new CustomException(ErrorCode.USER_NOT_FOUND));
 
-        // 오전 4시 기준 하루의 시작과 끝 설정
-        LocalDateTime startOfDay = date.atTime(4, 0, 0);
-        LocalDateTime endOfDay = date.plusDays(1).atTime(4, 0, 0);
+        // 오전 0시 기준 하루의 시작과 끝 설정
+        LocalDateTime startOfDay = date.atStartOfDay();
+        LocalDateTime endOfDay = date.plusDays(1).atStartOfDay();
 
         // 시작~종료 시간을 포함하는 일자의 학습 기록 조회
         List<StudyRecord> records = studyRecordRepository
@@ -151,8 +148,8 @@ private void checkAndNotifyDailyGoalAchievement(Long userId, LocalDate date) {
 
             // 오늘 완료한 계획 개수
             int completedCount = 0;
-            LocalDateTime startOfDay = date.atTime(4, 0);
-            LocalDateTime endOfDay = date.plusDays(1).atTime(4, 0);
+            LocalDateTime startOfDay = date.atStartOfDay();
+            LocalDateTime endOfDay = date.plusDays(1).atStartOfDay();
 
             for (StudyPlan plan : todayPlans) {
                 boolean hasRecord = studyRecordRepository.existsByStudyPlanIdAndDate(

src/test/java/com/back/domain/study/record/controller/StudyRecordControllerTest.java

@@ -250,36 +250,7 @@ void t3() throws Exception {
     }
 
     @Test
-    @DisplayName("학습 기록 조회 - 전날 밤~당일 새벽 기록의 경우")
-    void t4() throws Exception {
-        mvc.perform(post("/api/plans/records")
-                        .header("Authorization", "Bearer faketoken")
-                        .contentType(MediaType.APPLICATION_JSON)
-                        .content("""
-                    {
-                        "planId": %d,
-                        "startTime": "2025-10-02T23:00:00",
-                        "endTime": "2025-10-03T02:00:00",
-                        "duration": 10800,
-                        "pauseInfos": []
-                    }
-                    """.formatted(singlePlan.getId())))
-                .andExpect(status().isOk());
-
-        // 10월 2일로 조회 (04:00 기준이므로 이 기록이 포함되어야 함)
-        ResultActions resultActions = mvc.perform(get("/api/plans/records?date=2025-10-02")
-                        .header("Authorization", "Bearer faketoken"))
-                .andDo(print());
-
-        resultActions
-                .andExpect(status().isOk())
-                .andExpect(jsonPath("$.data", hasSize(1)))
-                .andExpect(jsonPath("$.data[0].startTime").value("2025-10-02T23:00:00"))
-                .andExpect(jsonPath("$.data[0].endTime").value("2025-10-03T02:00:00"));
-        }
-
-    @Test
-    @DisplayName("학습 기록 조회 - 전날 밤~당일 오전 4시 이후 끝난 기록의 경우")
+    @DisplayName("학습 기록 조회 - 전날 밤 시작 ~ 당일 끝난 기록의 경우")
     void t5() throws Exception {
         mvc.perform(post("/api/plans/records")
                         .header("Authorization", "Bearer faketoken")