Refactor: 예약 목록 조회 시 상태 필터링 가능하게 변경

Author: sso0om

back/src/main/java/com/back/domain/mentoring/reservation/controller/ReservationController.java

@@ -4,6 +4,7 @@
 import com.back.domain.member.member.service.MemberStorage;
 import com.back.domain.member.mentee.entity.Mentee;
 import com.back.domain.member.mentor.entity.Mentor;
+import com.back.domain.mentoring.reservation.constant.ReservationStatus;
 import com.back.domain.mentoring.reservation.dto.ReservationDto;
 import com.back.domain.mentoring.reservation.dto.request.ReservationRequest;
 import com.back.domain.mentoring.reservation.dto.response.ReservationPagingResponse;
@@ -33,10 +34,11 @@ public class ReservationController {
     @Operation(summary = "나의 예약 목록 조회", description = "본인의 예약 목록을 조회합니다. 로그인 후 조회할 수 있습니다.")
     public RsData<ReservationPagingResponse> getReservations(
         @RequestParam(defaultValue = "0") int page,
-        @RequestParam(defaultValue = "10") int size
+        @RequestParam(defaultValue = "10") int size,
+        @RequestParam(required = false)ReservationStatus status
     ) {
         Member member = rq.getActor();
-        Page<ReservationDto> reservationPage = reservationService.getReservations(member, page, size);
+        Page<ReservationDto> reservationPage = reservationService.getReservations(member, page, size, status);
         ReservationPagingResponse resDto = ReservationPagingResponse.from(reservationPage);
 
         return new RsData<>(

back/src/main/java/com/back/domain/mentoring/reservation/repository/ReservationRepository.java

@@ -33,19 +33,23 @@ Optional<Reservation> findByIdAndMember(
         SELECT r
         FROM Reservation r
         WHERE r.mentor.member.id = :memberId
+        AND (:status IS NULL OR r.status = :status)
         """)
     Page<Reservation> findAllByMentorMember(
         @Param("memberId") Long memberId,
+        @Param("status") ReservationStatus status,
         Pageable pageable
     );
 
     @Query("""
         SELECT r
         FROM Reservation r
         WHERE r.mentee.member = :member
+        AND (:status IS NULL OR r.status = :status)
         """)
     Page<Reservation> findAllByMenteeMember(
         @Param("member") Member member,
+        @Param("status") ReservationStatus status,
         Pageable pageable
     );
 

back/src/main/java/com/back/domain/mentoring/reservation/service/ReservationService.java

@@ -39,7 +39,7 @@ public class ReservationService {
     private final MentoringSessionService mentoringSessionService;
 
     @Transactional(readOnly = true)
-    public Page<ReservationDto> getReservations(Member member, int page, int size) {
+    public Page<ReservationDto> getReservations(Member member, int page, int size, ReservationStatus status) {
         Pageable pageable = PageRequest.of(
             page,
             size,
@@ -49,9 +49,9 @@ public Page<ReservationDto> getReservations(Member member, int page, int size) {
         Page<Reservation> reservations;
 
         if (member.getRole() == Member.Role.MENTOR) {
-            reservations = reservationRepository.findAllByMentorMember(member.getId(), pageable);
+            reservations = reservationRepository.findAllByMentorMember(member.getId(), status, pageable);
         } else {
-            reservations = reservationRepository.findAllByMenteeMember(member, pageable);
+            reservations = reservationRepository.findAllByMenteeMember(member, status, pageable);
         }
         return reservations.map(ReservationDto::from);
     }

back/src/test/java/com/back/domain/mentoring/reservation/service/ReservationServiceTest.java

@@ -101,14 +101,16 @@ void getReservations() {
 
             when(reservationRepository.findAllByMentorMember(
                 eq(mentor.getMember().getId()),
+                isNull(),
                 any(Pageable.class)
             )).thenReturn(reservationPage);
 
             // when
             Page<ReservationDto> result = reservationService.getReservations(
                 mentor.getMember(),
                 page,
-                size
+                size,
+                null
             );
 
             // then
@@ -118,6 +120,7 @@ void getReservations() {
             assertThat(result.getTotalPages()).isEqualTo(2);
             verify(reservationRepository).findAllByMentorMember(
                 eq(mentor.getMember().getId()),
+                isNull(),
                 any(Pageable.class)
             );
         }