Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 예매 생성 시 내가 선점한 좌석인지 검증하는 로직 추가 및 예매 토큰 인터셉터 버그 수정 #228

Merged
merged 13 commits into from
Jan 11, 2024

Conversation

annahxxl
Copy link
Collaborator

@annahxxl annahxxl commented Jan 11, 2024

구현 기능

feat

  • 예매 생성 시 내가 선점한 좌석인지 검증하는 로직 추가
  • 예매 토큰 인터셉터 적용 좌석 관련 path 잘못 매핑되어 있어 수정
  • 응답 바디 없는 컨트롤러 noContent 설정

refactor

  • 좌석 상태값 예매중 enum명 SELECTED로 수정
  • http 정리
  • 예매 에러 코드 순서 정리

test

  • 좌석 선점이 되어있지 않으면 예매 생성 실패하는 테스트 코드 추가
    • redis 의존적으로 코드가 변경되면서 redis 찌르고 오는 로직은 mocking하여 작성하였습니다. 그런데 이게 아무래도 실제 동작을 검증하는 것이 아니라서 맘에 쏙 들진 않네요..

todo

  • seatService 전반적인 코드 리팩터링 별도 이슈로 분리하여 작업 예정

resolve: #208

@annahxxl annahxxl added feature 기능 booking 예매 labels Jan 11, 2024
@annahxxl annahxxl self-assigned this Jan 11, 2024
@annahxxl annahxxl changed the title feat: 예매 생성 시 내가 선점한 좌석인지 검증하는 로직 추가 feat: 예매 생성 시 내가 선점한 좌석인지 검증하는 로직 추가 및 예매 토큰 인터셉터 버그 수정 Jan 11, 2024
@byulcode byulcode merged commit 2b79e13 into main Jan 11, 2024
1 check passed
@byulcode byulcode deleted the booking/208 branch January 11, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
booking 예매 feature 기능
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

feat: 예매 생성 시 내가 선점한 좌석인지 검증하는 로직 추가
2 participants