[#66] 결제 & 예약 확정 로직 수정 #67

pricelees · 2025-10-17T04:59:05Z

pricelees commented

2025-10-17 04:59:05 +00:00

📝 관련 이슈 및 PR

PR과 관련된 이슈 번호

#66

✨ 작업 내용

이전 #64 의 작업 이후, 결제 & 예약 확정 API 로직은 총 3개의 독립된 트랜잭션을 사용함.
검증 & 배치 충돌 방지를 위한 첫 번째 트랜잭션 이외의 다른 트랜잭션은 불필요하다고 판단함. -> PG사 성공 응답이 오면 나머지 작업은 @Async 처리 후 바로 성공 응답

🧪 테스트

변경된 로직에 대한 통합 테스트 완료
성능 테스트 결과 P95 응답 시간 327.01ms -> 235.52ms / 평균 응답 시간 77.85 -> 68.16ms / 최대 응답 시간 5.26 -> 4.08초 단축

📚 참고 자료 및 기타

## 📝 관련 이슈 및 PR **PR과 관련된 이슈 번호** - #66 ## ✨ 작업 내용  - 이전 #64 의 작업 이후, 결제 & 예약 확정 API 로직은 총 3개의 독립된 트랜잭션을 사용함. - 검증 & 배치 충돌 방지를 위한 첫 번째 트랜잭션 이외의 다른 트랜잭션은 불필요하다고 판단함. -> PG사 성공 응답이 오면 나머지 작업은 \@Async 처리 후 바로 성공 응답 ## 🧪 테스트  - 변경된 로직에 대한 통합 테스트 완료 - 성능 테스트 결과 P95 응답 시간 327.01ms -> 235.52ms / 평균 응답 시간 77.85 -> 68.16ms / 최대 응답 시간 5.26 -> 4.08초 단축 ## 📚 참고 자료 및 기타

pricelees added the

Kind/Enhancement

label 2025-10-17 04:59:05 +00:00

pricelees self-assigned this 2025-10-17 04:59:06 +00:00

pricelees added 17 commits 2025-10-17 04:59:06 +00:00

refactor: Dockerfile 단순화 및 별도 스크립트 추가 f6ef6e21ec

refactor: 로컬 테스트를 위한 payment API 기본 경로 수정 e3cfb6c78b

fix: 빌드 스크립트 오타 수정 e6cfd7b68b

refactor: k6 스크립트 수정 747245d9ac

- 초기 셋업 데이터 로드는 로컬에서 받아오도록 수정
- tag 추가로 ID별 구분이 아닌 API별 구분 집계

feat: PaymentDetail 도메인 객체 및 매핑 확장함수 추가 257fcb517d

feat: PaymentEvent 객체 정의 및 매핑 확장함수 추가 f4b9d1207e

feat: ReservationConfirmEvent 객체 정의 d0ee55be95

feat: 결제 정보를 저장하는 PaymentEventListener 및 테스트 추가 e0e7902654

feat: PaymentClient에서 받은 외부 API 응답을 이벤트로 매핑하는 확장함수 추가 및 이전 커밋의 PaymentEventListenerTest 패키지 이동 dbd2b9fb0c

refactor: PaymentService 내 결제 확정 로직에 이벤트 발행 추가 및 테스트 c1eb1aa2b4

refactor: 결제 확정 로직 변화에 따른 테스트 및 API 수정 c3330e5652

- 기존의 결제 확정 API 및 테스트 제거(취소는 유지)
- PaymentWriter 제거
- 테스트 코드 반영

refactor: 성능 테스트 환경 설정을 위해 사용되는 Repository 별도 분리 cce59e522e

feat: 예약 확정 EventListener 및 테스트 66bf68826b

refactor: OrderService의 결제 & 예약 확정 로직에 이벤트 발행 추가 a83352c733

remove: OrderService 로직 변화로 사용하지 않게 된 클래스 제거 9fe576af11

test: PaymentServiceTest에 결제 수단별 테스트 추가 385f98fb21

test: 변경된 OrderService의 확정 로직 테스트 반영 b636ac926e

pricelees merged commit 06f7faf7f9 into main

2025-10-17 04:59:12 +00:00

pricelees referenced this issue from a commit

2025-10-17 04:59:13 +00:00

[#66] 결제 & 예약 확정 로직 수정 (#67)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: pricelees/roomescape-refactored#67