bug: 웨이팅 취소 시 DB 먼저 조회하고 Redis 데이터 삭제하도록 로직 변경

[Jjiggu]

작업 요약

웨이팅 취소 시 DB 먼저 조회하고 Redis 데이터 삭제하도록 로직 변경

Issue Link

#342

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

• 버그 수정

  • 예약 취소 시 오류 검증 개선 - 더 정확한 예외 처리로 문제 상황 파악 용이

• 개선

  • 예약 상태 관리 강화 - 취소된 예약 정보를 명시적으로 기록하여 데이터 신뢰성 향상

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

예약 취소 로직을 개선하여 명확한 예외 처리를 추가하고, 대기 상태를 직접 삭제하는 대신 CANCELLED 상태의 예약을 데이터베이스에 저장하도록 변경했습니다. 또한 예약 번호 존재 여부를 확인하는 저장소 조회 메서드를 추가했습니다.

Changes

Cohort / File(s)	요약
예약 서비스 취소 로직 nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java	cancelWaiting 메서드에서 정적 오류 메시지 import 추가, storeId 및 userId 검증 강화(StoreNotFoundException, UserNotFoundException, ReservationNotFoundException 사용), Redis 직접 삭제 대신 CANCELLED 상태 예약을 데이터베이스에 저장하는 로직 추가, 작업 순서 재조정(예약 저장 후 Redis 제거)
예약 저장소 쿼리 메서드 nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/reservation/repository/ReservationRepository.java	예약 번호로 존재 여부를 확인하는 새로운 쿼리 메서드 existsReservationByReservationNumber(String reservationNumber) 추가

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• refactor(Store): 입장 시간 처리 updatedAt으로 통일 #183: 동일한 cancelWaiting 흐름에서 CANCELLED 예약을 데이터베이스에 저장하는 로직 추가
• Refactor: 예외처리 보강 및 예약 번호를 통한 대기 상태 변경 로직 추가 #320: 예약 번호 기반 조회 및 예약 상태 변경 로직을 수정하여 예약 번호 존재 여부 확인 메서드 추가
• feat(Reservation): 대기열 조회 및 취소 #123: ReservationService.cancelWaiting 메서드 서명을 수정

Suggested reviewers

• HyemIin

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0a5e5cb and a8ce493.

📒 Files selected for processing (2)

• nowait-app-user-api/src/main/java/com/nowait/applicationuser/reservation/service/ReservationService.java
• nowait-domain/domain-core-rdb/src/main/java/com/nowait/domaincorerdb/reservation/repository/ReservationRepository.java

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}