[🚀 사이클2 - 미션 (예약 변경/취소와 에러 처리)] 워넬 미션 제출합니다.

[jeonwonjun]

체크 리스트

• 미션의 필수 요구사항을 모두 구현했나요?
• Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
• 애플리케이션이 정상적으로 실행되나요?

베이스 코드 선택 체크

• 이전 미션의 내 코드에서 시작
• 이전 미션의 페어의 코드에서 시작

어떤 부분에 집중하여 리뷰해야 할까요?

안녕하세요 아마찌!

이번 미션은 Reservation 위주로 예외처리를 진행했습니다. 추후 Theme, ReservationTime에 대해서도 예외처리 진행할 예정입니다!

질문은 언제든 환영합니다
이번 리뷰도 잘부탁드려요 🙇🏻‍♂️

궁금한 점

1. ErrorResponse의 책임 범위

현재 ErrorResponse에서 사용자에게 다음 행동(action)까지 전달할지 고민했습니다.
다음 행동까지 전달하게 된다면 유연성이 떨어지고, 작은 수정이 발생해도 서버코드를 수정해야하는 일이 발생할 것 같아서 현재는 code, message만 전달하게 했습니다.
백엔드는 에러 원인까지만 책임지고, 실제 사용자 행동 유도는 프론트에서 처리하는 것이 더 자연스러운 구조일까요?

예시

{
  "code": "RESERVATION_ALREADY_EXISTS",
  "message": "이미 예약된 시간입니다."
}

vs

{
  "code": "RESERVATION_ALREADY_EXISTS",
  "message": "이미 예약이 존재합니다.",
  "action": "다른 시간을 선택해주세요."
}

2. Service 테스트의 방향성

ReservationServiceTest에서 본인_예약_변경_성공과 같은 성공 테스트는 DAO의 작동 여부가 필요한 테스트라고 생각됩니다.

Service 테스트에서 실제 변경 결과까지 검증해야 하는지,
아니면 정책 판단 후 DAO 호출 여부만 검증하는 것이 더 적절한지 고민입니다.

예시

then(reservationDao).should()
    .updateDateTimeById(...)

3. selectById에서 Optional을 사용하는 이유

기존에는 queryForObject()를 사용했는데, Optional을 반환하기 위해 query() + findAny() 방식으로 변경했습니다.
이런 방식이 자연스러운지 궁금합니다.

예시

List<Reservation> reservations = jdbcTemplate.query(sql, rowMapper, id);
return reservations.stream().findAny();