Skip to content

예약 취소 및 만료 구현 #11

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.

Sign up for GitHub

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

Jump to bottom
Open
JupitorCentral wants to merge 59 commits into
base: main
Choose a base branch
from
Open

예약 취소 및 만료 구현 #11

JupitorCentral wants to merge 59 commits into from

Conversation

@JupitorCentral
Copy link
Collaborator

@JupitorCentral JupitorCentral commented Dec 27, 2025

예약 취소 및 만료 구현

@JupitorCentral
UserManagementService - createUser 수정
fec3197
@JupitorCentral
사용자 생성 테스트 생성 - DB에 실제로 저장되는 persistentTest 와 @transactional 어노테이션으로 …
be10ca5 
…롤백하는 그냥 Test 파일 설정
@JupitorCentral
ReservationBookingService - book 테스트
c8d1147
@JupitorCentral
ReservationBookingService - book eventProxy 선언 위치 변경
59250bc
@JupitorCentral
Seat 테이블의 status 컬럼 타입 변경 - varchar(20) 씩이나 컬럼이 클 필요가 없음.
0de0449
@JupitorCentral
Reservation 테이블의 ReservationStatus 컬럼 타입 변경 - varchar(20) 씩이나 컬럼이 클 필…
d7fc0e1 
…요가 없음.
@JupitorCentral
IllegalReservationStatusDbValueException 추가
5b7c410
@JupitorCentral
Seat 단계를 진행하는 방식 변경
9f688e5
@JupitorCentral
ReservationBookingServiceIntegrationTest 테스트 변경 (setUp 단계추가), 이를 위한 S…
77dd612 
…eat eventId 컬럼 및 Event 엔티티 설저 변경

ReservationBookingRequest 생성자 패턴 builder 로 통일
@JupitorCentral
Foreign Constraint 에 해당하는 엔티티 필드를 읽기 전용으로 변경, F key 컬럼을 따로 설정
2d5e51a
@JupitorCentral
일관성이 없는 변수설정 제거
3231608
@JupitorCentral
Test method 가 하나인데 굳이 @beforeeach 쓸 필요가 있나 ?
34a287f
@JupitorCentral
노출되지 말아야 할 내부 클래스 숨김
fde3c7b
@JupitorCentral
ReservationBookingService - makeReservation 구현 변경
13485af
@JupitorCentral
f key 코드 삭제
18306ec
@JupitorCentral
불필요 테스트 코드 삭제
69ad066
@JupitorCentral
테스트 h2 + embedded redis 쓰게 설정
d8a57d5
@JupitorCentral
변수명 변경 ReservationBookingRequest.java -> ReservationRequest
b918283
@JupitorCentral
Seat 관련된 Service 생성
5b9ff84
@JupitorCentral
SeatService 생성 (ReservationQueryService 에서 Seat 관련된 메소드 추출)
05dda77 
SeatService - createValidSeat 테스트

Foreign Key 검증을 위한 ForeignKeyValidator 생성
@JupitorCentral
불필요한 ForeignKeyValidator 삭제
44bf751 
SeatService 에 seat 저장시 유효성 검증 추가
@JupitorCentral
SeatService - createValidSeat 테스트 추가
056be87
@JupitorCentral
SeatService - createValidSeat 테스트 주석
af1e3b7
@JupitorCentral
SeatService.java -> SeatManagementService.java
a63eda3
@JupitorCentral
Seat 생성시 일어날 수 있는 오류 추가
0828515
@JupitorCentral
포맷팅
9e26ddc
@JupitorCentral
주석 수정,
c36aed6 
SeatManagementService - createValidUser -> createUser (어차피 valid 해야 하니)
@JupitorCentral
Exception message 포맷 수정
3ca956b
@JupitorCentral
Reservation, Seat Enum 타입 값 수정
89db088
@JupitorCentral
Exception 의 파라미터 수정 (구현 숨기기), 테스트파일 위치 변경
a793100
@JupitorCentral
Event - 티켓 가격 추가
1be9459
@JupitorCentral
결제 service 생성 & 테스트
9bbf150
@JupitorCentral
불필요한 설정 코드 삭제
8fd2946 
db_creation_query.sql indent 조정

@ModelAttribute 어노테이션 삭제

Seat->Event f key 위반 Exception 위치 변경
@JupitorCentral
InvalidStatusException.java -> InvalidReservationStatusException
f4432f0 
모든 Exception 에 적당한 HttpStatus 설정
@JupitorCentral
f key 관계에 대해, key 를 따로 저장하는것이 아니라, 엔티티 그대로 보관하는 것으로 변경
3a5243c 
그리고, F key 관계를 가지는 엔티티에 대해서
F key constraint 가 생성되지 않도록 설정
(foreignKey = @foreignkey(ConstraintMode.NO_CONSTRAINT)
@JupitorCentral
@transactional 적용을 위해
28fc4b8 
 메서드에 public modifier 추가, @transactional annotation 을 메서드로 위치 변경
@JupitorCentral
객체지향화를 위한 메서드 추가
9938c32
@JupitorCentral
불필요한 변수 정의 삭제
4e37764
@JupitorCentral
proceed -> reserve, sell
ad0cb21
@JupitorCentral
proxy 테스트 추가.
92b04ab
@JupitorCentral
테스트 버그 수정, @DisplayName 설정
c13f45d
@JupitorCentral
Custom Exception 에 ProblemDetail 적용
186eef5
@JupitorCentral
no!
7f18b93
@JupitorCentral
Integration Test 위치 변경 (Unit Test가 아닌것들),
928af98 
PayManagementController 테스트 코드 추가
@JupitorCentral
Custom Exception 을 위한 GlobalExceptionHandler 의 잘못된 시그니쳐 수정
155b9fe 
PayManagementControllerTest 의 Exception 발생시 올바른 ProblemDetail 형식의 Json 값을 리턴하는지에 대한 테스트코드 추가
@JupitorCentral
중복 테스트 코드 generic 을 사용해서 줄이기
a423cb3
@JupitorCentral
ResponseEntityExceptionHandler 를 쓸 필요가 없었다.
11f462a
@JupitorCentral
내가 왜 썼는지 기억이 났다. Spring 이 제공하는 기본 핸들러를 사용하기 위해서이다.
720b37a
@JupitorCentral
github action 설정 (build, test)
dc1fc08
@JupitorCentral
ReservationBookingControllerTest 테스트코드 추가
fc7f838
@JupitorCentral
테스트 코드 수정
b93051e
@JupitorCentral JupitorCentral self-assigned this Dec 27, 2025
@JupitorCentral JupitorCentral linked an issue Dec 27, 2025 that may be closed by this pull request
예약 취소 + 예약 만료 구현 #10
Open
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@f-lab-bot f-lab-bot Awaiting requested review from f-lab-bot

@if-else-f if-else-f Awaiting requested review from if-else-f

Assignees

@JupitorCentral JupitorCentral

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

예약 취소 + 예약 만료 구현

2 participants

@JupitorCentral