feat: 무료 주문 생성하기 API 구현 #520
Conversation
Walkthrough이번 변경사항은 무료 주문 생성 기능을 추가하고, 결제 금액이 0원인 경우의 주문 처리 로직을 개선합니다. 새로운 상수 Changes
Sequence Diagram(s)sequenceDiagram
participant Customer
participant OrderService
participant Order
participant Membership
participant Coupon
Customer->>OrderService: createFreeOrder()
OrderService->>Membership: checkMembership()
OrderService->>Coupon: validateCoupon()
OrderService->>Order: createFree()
Order->>OrderService: return order details
OrderService->>Customer: return order confirmation
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (11)
- src/main/java/com/gdschongik/gdsc/domain/common/vo/Money.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/domain/coupon/domain/Coupon.java (2 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/api/AdminOrderController.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/api/OnboardingOrderController.java (2 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java (3 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java (4 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/domain/OrderValidator.java (3 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/dto/request/OrderFreeCreateRequest.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/global/exception/ErrorCode.java (1 hunks)
- src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java (2 hunks)
- src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java (4 hunks)
Files skipped from review due to trivial changes (1)
- src/main/java/com/gdschongik/gdsc/domain/coupon/domain/Coupon.java
Additional comments not posted (24)
src/main/java/com/gdschongik/gdsc/domain/order/dto/request/OrderFreeCreateRequest.java (3)
1-1: 패키지 선언 확인패키지 선언이 올바르게 되어 있습니다.
3-6: 필요한 유효성 검사 주석 추가유효성 검사 주석이 적절하게 추가되었습니다.
8-11: DTO 필드와 유효성 검사 확인필드와 유효성 검사 주석이 올바르게 정의되었습니다.
orderNanoId는 길이 제한이 있고,membershipId와issuedCouponId는 양수여야 합니다.src/main/java/com/gdschongik/gdsc/domain/order/api/OnboardingOrderController.java (3)
6-6: 새로운 DTO 요청 추가새로운
OrderFreeCreateRequestDTO 요청이 올바르게 추가되었습니다.
32-32: 주문 완료 메서드 요약 수정
completeOrder메서드의 요약이 적절하게 수정되었습니다.
39-44: 무료 주문 생성 엔드포인트 추가무료 주문을 생성하는 새로운 엔드포인트가 올바르게 추가되었습니다. 요청을 처리하고 적절한 응답을 반환합니다.
src/main/java/com/gdschongik/gdsc/domain/order/api/AdminOrderController.java (3)
40-41: 메서드 요약 및 설명 수정
getCompletedOrderPayment메서드의 요약 및 설명이 적절하게 수정되었습니다.
44-44: 메서드 이름 변경메서드 이름이
getCompletedPaidOrderPayment로 변경되었습니다. 이 변경은 메서드의 목적을 더 명확하게 합니다.
48-48: 주문 취소 메서드 설명 확장
cancelOrder메서드의 설명이 확장되어 주문 취소 시의 추가 작업을 명확하게 설명합니다.src/main/java/com/gdschongik/gdsc/domain/common/vo/Money.java (1)
37-38: 잘했습니다! 새로운 상수ZERO는 올바르게 초기화되었으며, 코드의 가독성과 재사용성을 높입니다.src/main/java/com/gdschongik/gdsc/domain/order/domain/OrderValidator.java (2)
70-72: 잘했습니다!Money.ZERO를 사용하여 코드의 명확성과 일관성이 향상되었습니다.
102-131: 잘했습니다!validateFreeOrderCreate메서드는 무료 주문 생성에 필요한 모든 검증을 포괄적으로 수행합니다.src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java (3)
102-112: 잘했습니다!createFree메서드는 무료 주문을 올바르게 생성하며, 적절한 상태와 금액 값을 초기화합니다.
157-159: 잘했습니다!isFree메서드는 주문이 무료인지 올바르게 판단합니다.
Line range hint
135-148:
잘했습니다!validateCancelable메서드는 무료 주문에 대한 검증을 추가하여 주문 취소 로직을 강화합니다.src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java (2)
111-116: 메소드 이름 변경 및 로직 추가 확인
getCompletedPaidOrderPayment메소드 이름 변경과 무료 주문이 아닌지 확인하는 로직 추가가 적절합니다. 이로 인해 메소드의 목적이 명확해졌습니다.
147-165: 무료 주문 생성 메소드 추가 확인
createFreeOrder메소드가 무료 주문 생성을 처리하기 위해 추가되었습니다. 필요한 검증을 포함하고 있으며, 주문을 저장소에 저장하는 로직도 적절합니다.src/main/java/com/gdschongik/gdsc/global/exception/ErrorCode.java (2)
120-120: 에러 코드 이름 변경 확인
ORDER_COMPLETED_NOT_FOUND에러 코드가ORDER_COMPLETED_PAID_NOT_FOUND로 변경되었습니다. 이로 인해 유료 주문과 관련된 에러임이 명확해졌습니다.
122-122: 새로운 에러 코드 추가 확인무료 주문을 취소할 수 없는 경우를 처리하기 위한
ORDER_CANCEL_FREE_ORDER에러 코드가 추가되었습니다. 이는 에러 처리 범위를 확장하여 적절합니다.src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java (2)
44-83: 무료 주문 생성 테스트 추가 확인
무료주문_생성할때클래스에 무료 주문 생성과 관련된 테스트 메소드가 추가되었습니다. 이 테스트들은 무료 주문이 올바르게 생성되고 상태와 최종 결제 금액이 검증되는지 확인합니다.
86-157: 주문 취소 테스트 추가 확인
주문_취소할때클래스에 무료 주문 취소 시 실패하는지 확인하는 테스트 메소드가 추가되었습니다. 이 테스트는 무료 주문을 취소하려고 할 때 예상된 예외가 발생하는지 확인합니다.src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java (3)
Line range hint
14-14: 상수MONEY_0_WON제거 확인
MONEY_0_WON상수가 제거되었습니다. 이제Money.ZERO를 사용하여 0 값을 나타냅니다. 코드의 일관성을 유지하기 위한 좋은 변경입니다.
382-382:Money.ZERO사용 확인
Order.createPending메서드에서MONEY_0_WON대신Money.ZERO를 사용합니다. 이는 0 값의 일관된 표현을 보장합니다.다른 코드베이스에서도
Money.ZERO가 일관되게 사용되는지 확인하는 것이 좋습니다.
302-302:Money.ZERO사용 확인
Order.createPending메서드에서MONEY_0_WON대신Money.ZERO를 사용합니다. 이는 0 값의 일관된 표현을 보장합니다.다른 코드베이스에서도
Money.ZERO가 일관되게 사용되는지 확인하는 것이 좋습니다.Verification successful
Money.ZERO사용 확인
Order.createPending메서드에서MONEY_0_WON대신Money.ZERO를 사용합니다. 이는 0 값의 일관된 표현을 보장합니다.다른 코드베이스에서도
Money.ZERO가 일관되게 사용되는지 확인한 결과, 여러 테스트 파일 및 주요 애플리케이션 파일에서 일관되게 사용되고 있음을 확인했습니다.
src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.javasrc/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.javasrc/main/java/com/gdschongik/gdsc/domain/order/domain/Order.javasrc/main/java/com/gdschongik/gdsc/domain/order/domain/OrderValidator.javasrc/main/java/com/gdschongik/gdsc/domain/coupon/domain/Coupon.javasrc/main/java/com/gdschongik/gdsc/domain/common/vo/Money.java이로써
Money.ZERO의 일관된 사용이 확인되었습니다.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify all usages of `Money.ZERO` in the codebase. # Test: Search for the usage of `Money.ZERO`. Expect: Consistent usage across the codebase. rg --type java -A 5 $'Money.ZERO'Length of output: 5403
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (6)
- src/main/java/com/gdschongik/gdsc/domain/order/api/OnboardingOrderController.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java (2 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/domain/MoneyInfo.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java (3 hunks)
- src/main/java/com/gdschongik/gdsc/global/exception/ErrorCode.java (1 hunks)
- src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java (2 hunks)
Files skipped from review as they are similar to previous changes (4)
- src/main/java/com/gdschongik/gdsc/domain/order/api/OnboardingOrderController.java
- src/main/java/com/gdschongik/gdsc/domain/order/application/OrderService.java
- src/main/java/com/gdschongik/gdsc/domain/order/domain/Order.java
- src/main/java/com/gdschongik/gdsc/global/exception/ErrorCode.java
Additional comments not posted (7)
src/main/java/com/gdschongik/gdsc/domain/order/domain/MoneyInfo.java (1)
62-64: 새로운 메서드 추가 확인됨:isFree이 메서드는
finalPaymentAmount가Money.ZERO인지 확인합니다. 로직이 명확하고 올바르게 구현되었습니다.src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderTest.java (6)
47-65: 테스트 메서드 추가 확인됨:주문상태는_완료이다이 테스트는 무료 주문이 생성될 때 주문 상태가
COMPLETED인지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
67-84: 테스트 메서드 추가 확인됨:최종결제금액이_0원이_아니면_실패한다이 테스트는 최종 결제 금액이 0원이 아닌 경우 예외가 발생하는지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
90-111: 테스트 메서드 추가 확인됨:대기상태이면_실패한다이 테스트는 주문이 대기 상태일 때 취소를 시도하면 예외가 발생하는지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
113-136: 테스트 메서드 추가 확인됨:취소상태이면_실패한다이 테스트는 주문이 이미 취소된 상태일 때 취소를 시도하면 예외가 발생하는지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
138-159: 테스트 메서드 추가 확인됨:무료주문이면_실패한다이 테스트는 무료 주문을 취소하려고 할 때 예외가 발생하는지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
161-185: 테스트 메서드 추가 확인됨:완료상태이면_성공한다이 테스트는 주문이 완료 상태일 때 성공적으로 취소될 수 있는지 확인합니다. 테스트 설정과 검증이 올바르게 구현되었습니다.
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java (6 hunks)
Files skipped from review as they are similar to previous changes (1)
- src/test/java/com/gdschongik/gdsc/domain/order/domain/OrderValidatorTest.java
| @@ -37,15 +37,15 @@ public ResponseEntity<Page<OrderAdminResponse>> getOrders( | |||
| } | |||
|
|
|||
| @Operation( | |||
| summary = "완료된 주문 결제정보 조회하기", | |||
| description = "주문 결제정보를 조회합니다. 토스페이먼츠 API의 결제 정보인 Payment 객체를 반환합니다. 완료된 주문에 대해서만 조회 가능합니다.") | |||
| summary = " 완료된 유료 주문 결제정보 조회하기", | |||
There was a problem hiding this comment.
| summary = " 완료된 유료 주문 결제정보 조회하기", | |
| summary = "완료된 유료 주문 결제정보 조회하기", |
| @@ -98,6 +98,26 @@ public static Order createPending( | |||
| .build(); | |||
| } | |||
|
|
|||
| public static Order createFree( | |||
| String nanoId, Membership membership, @Nullable IssuedCoupon issuedCoupon, MoneyInfo moneyInfo) { | |||
There was a problem hiding this comment.
무료 주문이면 쿠폰이 반드시 필요한거아닌가요??
There was a problem hiding this comment.
회비가 0원인 경우에는 쿠폰 없이 무료 주문이 가능하다고 봤어요
물론 회비를 무조건 양수로 지정하면 항상 쿠폰이 필요해지겠지만
쿠폰을 꼭 써야 무료 주문이다 보다는 최종결제금액이 0원이면 무료 주문이다 가 더 의미적으로 적절하다고 봤습니다
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/main/java/com/gdschongik/gdsc/domain/order/api/AdminOrderController.java (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- src/main/java/com/gdschongik/gdsc/domain/order/api/AdminOrderController.java
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
Money.ZERO와 같이 상수로 생성할 수 있도록 했습니다📚 기타
Summary by CodeRabbit
Summary by CodeRabbit