[feat] 할인 쿠폰 발급 #82

kimhyun5u · 2024-08-15T16:22:46Z

💡 다음 이슈를 해결했어요.

Issue Link - #75

할인 쿠폰 발급

💡 이슈를 처리하면서 추가된 코드가 있어요.

CouponIssuance

쿠폰 발급 엔티티 입니다.

IssueCouponeService

쿠폰 발급 로직을 담당하는 서비스입니다.

💡 이런 고민을 했어요.

동시성 제어 성능 고민

낙관적 락 사용
비관적 락 사용

CouponIssuance 는 어디에 ?

할인 쿠폰 발급은 Coupon 도메인에서 담당한다.
+ 할인 쿠폰 발급은 Coupon 도메인에서 담당한다.
+ 할인 쿠폰 생성 시 Coupon 과 Customer 가 null 이 아니어야 한다.
+ 할인 쿠폰 만료일이 발급일보다 이전이여야한다.
+ 할인 쿠폰의 수량이 0일 때 발급 받을 수 없다.

✅ 셀프 체크리스트

내 코드를 스스로 검토했습니다.
필요한 테스트를 추가했습니다.
모든 테스트를 통과합니다.
브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
커밋 메세지를 컨벤션에 맞추었습니다.
wiki를 수정했습니다.

- CouponIssuanceTest.testConstruct: CouponIssuance 생성자 테스트

- CouponIssuance: Customer와 Coupon 을 갖는다. 또한, 발급일, 사용일을 기록한다.

- 잘못된 Exception 상속 수정

- CouponIssuanceTest.testConstructWithNullCoupon: Coupon 이 Null 인 경우 예외 처리 - CouponIssuanceTest.testConstructWithNullCustomer: Customer 가 Null 인 경우 예외 처리 - CouponIssuanceTest.testConstructWithExpiredCoupon: 만료된 쿠폰인 경우 예외 처리

- CouponErrorCode.EXPIRED_COUPON: 만료된 쿠폰 에러

- NullPointException 이 아닌 InvalidICreationIssuanceException 을 던지도록 변경

- CouponIssuanceErrorCode 를 사용하도록 변경

- CouponIssuance 생성 시 Customer 와 Coupon 의 NUll 여부 확인 및 Coupon 만료일이 발급일 전인지 확인

- errorCode 변경

- Coupon.isExpired: 쿠폰 만료 상태 확인 메소드

- CouponIssuance 생성 시 Null 값 과 쿠폰 만료 상태 검증

- Coupon 에서는 쿠폰이 만료되었더라도 에러를 발생할 이유가 없어 에러 소유를 CouponIssuance 로 넘김

- 만료일 테스트를 위해 수동으로 바꿀 수 있도록 변경

- CouponIssuanceTest.testConstructWithExpiredCoupon 에서 Coupon 이 정상적으로 생성된 후 만료되는 시나리오로 변경

- JpaRepository 기본 키 수정

- IssueCouponServiceTest.testIssueCoupon: 쿠폰 발급 검증 테스트

- test를 위해 사용할 Customer

- test를 위해 사용할 CouponIssuance

- CustomerFixture.createCustomer(UUID): 기본 설정에 id 값만 바뀌는 customer 생성 메소드

- CouponIssuanceFixture.createCouponIssuance: 테스트에서 사용할 CouponIssuance 생성

- IssueCouponService.issueCoupon: 구매자에게 쿠폰을 발행

- IssueCouponServiceTest.testIssueCouponFailWithNotExistCustomer: 사용자가 존재하지 않는 요청 검증 테스트 - IssueCouponServiceTest.testIssueCouponFailWithNotExistCoupon: 쿠폰이 존재하지 않는 요청 검증 테스트 - IssueCouponServiceTest.testIssueCouponFailWithExpiredCoupon: 만료된 쿠폰을 발행하는 요청 검증 테스트

- IssueCouponService.issueCoupon 에 ExpiredCouponException 발생 명시

- IssueCouponServiceTest.testIssueCouponFailWithInsufficientCouponQuantity: 수량 부족 시나리오 검증 테스트

- 테스트를 위해 커스텀 할 수 있도록 수정

- IssueCouponServiceIntegrationTest.testIssueCoupon: 쿠폰 발급 성공 검증 테스트 - IssueCouponServiceIntegrationTest.testIssueCouponFailWithInsufficientQuantity: 쿠폰 수량 부족 실패 검증 테스트 - IssueCouponServiceIntegrationTest.testIssueCouponWithConcurrency: 동시성 제어 검증 테스트

- CouponFixture.createCoupon(String, int, int, LocalDateTime): Coupon Entity 를 생성하는 메서드

- CouponRepository.findByIdWithPessimisticLock: 비관적 락 적용 조회 쿼리

- 쿠폰 조회 시 CouponRepository.findByIdWithPessimisticLock 사용하도록 변경

- `@DisplayName` 수정: 쿠폰 발급 -> 쿠폰 등록

- CouponApiControllerTest.testIssueCoupon: 조건 명시적으로 등록

- CouponApiControllerTest.testIssueCoupon: 쿠폰 발급 요청 검증 테스트

- session 등록 시 SessionConst.SESSION_CUSTOMER_KEY 사용하도록 수정 - 응답 코드 CREATED 로 변경

- CouponApiController.issueCoupon: 쿠폰 발급 API

- InsufficientCouponQuantityException message 수정

- IssueCouponService.issueCoupon throws 주석 오류 수정

- CouponApiControllerTest.testIssueCouponFailWithoutSession: 세션 없이 쿠폰 발급 요청 검증 테스트 - CouponApiControllerTest.testIssueCouponFailWithNotExistsId: 존재하지 않는 쿠폰 또는 구매자 쿠폰 발급 요청 검증 테스트 - CouponApiControllerTest.testIssueCouponFailWithExpiredCoupon: 만료된 쿠폰 발급 요청 검증 테스트 - CouponApiControllerTest.testIssueCouponFailWithInsufficientQuantity: 수량이 부족한 쿠폰 발급 요청 검증 테스트

- InvalidICreationIssuanceException, ExpiredCouponException 에 대한 예외 핸들링 추가

- 비관적 락 적용에 따른 테스트 코드 수정

…u_할인-쿠폰-발급

src/main/java/camp/woowak/lab/coupon/domain/Coupon.java

src/main/java/camp/woowak/lab/coupon/domain/CouponIssuance.java

Hyeon-Uk

다른 부분은 동훤님이 잘 찝어주셔서 사소한 리뷰 남기겠습니다!

src/main/java/camp/woowak/lab/coupon/domain/CouponIssuance.java

- 발급 시간 변경 제한 Co-authored-by: 김현욱 <43038815+Hyeon-Uk@users.noreply.github.com>

Hyeon-Uk

LGTM ! :~)

Dr-KoKo

LGTM!

kimhyun5u added 30 commits August 15, 2024 21:04

[test] CouponIssuanceTest 구현

58b40e9

- CouponIssuanceTest.testConstruct: CouponIssuance 생성자 테스트

[feat] CouponIssuance 구현

2a9ea1d

- CouponIssuance: Customer와 Coupon 을 갖는다. 또한, 발급일, 사용일을 기록한다.

[fix] DuplicateCouponTitleException 수정

b18a1c4

- 잘못된 Exception 상속 수정

[feat] CouponErrorCode 수정

d2f5b69

- CouponErrorCode.EXPIRED_COUPON: 만료된 쿠폰 에러

[feat] ExpiredCouponException 구현

7b968c5

[fix] CouponIssuanceTest 테스트 조건 수정

3d94a01

- NullPointException 이 아닌 InvalidICreationIssuanceException 을 던지도록 변경

[feat] CouponIssuanceErrorCode 구현

a78ffb6

[fix] ExpiredCouponException 수정

0edbd05

- CouponIssuanceErrorCode 를 사용하도록 변경

[feat] InvalidICreationIssuanceException 구현

dcc1a5b

[feat] CouponIssuanceValidator 구현

ab07e62

- CouponIssuance 생성 시 Customer 와 Coupon 의 NUll 여부 확인 및 Coupon 만료일이 발급일 전인지 확인

[fix] CouponIssuanceErrorCode 수정

c7300a5

- errorCode 변경

[feat] Coupon 수정

f912959

- Coupon.isExpired: 쿠폰 만료 상태 확인 메소드

[feat] CouponIssuance 수정

4f5ed5b

- CouponIssuance 생성 시 Null 값 과 쿠폰 만료 상태 검증

[fix] CouponErrorCode 수정

eeaff6c

- Coupon 에서는 쿠폰이 만료되었더라도 에러를 발생할 이유가 없어 에러 소유를 CouponIssuance 로 넘김

[fix] TestCoupon 수정

2d0c017

- 만료일 테스트를 위해 수동으로 바꿀 수 있도록 변경

[fix] CouponIssuanceTest 수정

fa6f35a

- CouponIssuanceTest.testConstructWithExpiredCoupon 에서 Coupon 이 정상적으로 생성된 후 만료되는 시나리오로 변경

[feat] CouponIssuanceRepository 구현

55999ee

[fix] CustomerRepository 수정

776eedd

- JpaRepository 기본 키 수정

[test] IssueCouponServiceTest 구현

e2bcbf5

- IssueCouponServiceTest.testIssueCoupon: 쿠폰 발급 검증 테스트

[feat] TestCustomer 구현

3c542d3

- test를 위해 사용할 Customer

[feat] TestCouponIssuance 구현

60e55cb

- test를 위해 사용할 CouponIssuance

[feat] CustomerFixture 수정

908f356

- CustomerFixture.createCustomer(UUID): 기본 설정에 id 값만 바뀌는 customer 생성 메소드

[feat] CouponIssuanceFixture 구현

203b319

- CouponIssuanceFixture.createCouponIssuance: 테스트에서 사용할 CouponIssuance 생성

[feat] IssueCouponCommand 구현

e4ba264

[feat] IssueCouponService 구현

d44df70

- IssueCouponService.issueCoupon: 구매자에게 쿠폰을 발행

[docs] IssueCouponService 주석 수정

a322afb

- IssueCouponService.issueCoupon 에 ExpiredCouponException 발생 명시

[test] IssueCouponServiceTest 수정

6203301

- IssueCouponServiceTest.testIssueCouponFailWithInsufficientCouponQuantity: 수량 부족 시나리오 검증 테스트

[feat] TestCoupon 수정

601bd0d

- 테스트를 위해 커스텀 할 수 있도록 수정

kimhyun5u added 15 commits August 16, 2024 00:36

[feat] CouponFixture 수정

8e1ad32

- CouponFixture.createCoupon(String, int, int, LocalDateTime): Coupon Entity 를 생성하는 메서드

[feat] CouponRepository 수정

035af13

- CouponRepository.findByIdWithPessimisticLock: 비관적 락 적용 조회 쿼리

[feat] IssueCouponService 수정

dba84a2

- 쿠폰 조회 시 CouponRepository.findByIdWithPessimisticLock 사용하도록 변경

[docs] CouponApiControllerTest 수정

b42a7f2

- `@DisplayName` 수정: 쿠폰 발급 -> 쿠폰 등록

[fix] CouponApiControllerTest 수정

c148289

- CouponApiControllerTest.testIssueCoupon: 조건 명시적으로 등록

[test] CouponApiControllerTest 수정

10143d9

- CouponApiControllerTest.testIssueCoupon: 쿠폰 발급 요청 검증 테스트

[fix] CouponApiControllerTest 수정

435e59d

- session 등록 시 SessionConst.SESSION_CUSTOMER_KEY 사용하도록 수정 - 응답 코드 CREATED 로 변경

[feat] IssueCouponResponse 구현

b72fbc0

[feat] CouponApiController 수정

ace0702

- CouponApiController.issueCoupon: 쿠폰 발급 API

[feat] IssueCouponService 수정

8aff497

- InsufficientCouponQuantityException message 수정

[docs] IssueCouponService 수정

588a223

- IssueCouponService.issueCoupon throws 주석 오류 수정

[feat] CouponExceptionHandler 수정

bb1650e

- InvalidICreationIssuanceException, ExpiredCouponException 에 대한 예외 핸들링 추가

[fix] IssueCouponServiceTest 수정

f770f0d

- 비관적 락 적용에 따른 테스트 코드 수정

kimhyun5u added the ✨ Feature 기능 개발 label Aug 15, 2024

kimhyun5u added this to the 프로토타입 만들기 milestone Aug 15, 2024

kimhyun5u requested review from Hyeon-Uk, june-777 and Dr-KoKo August 15, 2024 16:22

kimhyun5u self-assigned this Aug 15, 2024

kimhyun5u linked an issue Aug 15, 2024 that may be closed by this pull request

[기능] 할인 쿠폰 발급 #75

Closed

[merge] remote-tracking branch 'origin/main' into feature/75_kimhyun5…

49b9fef

…u_할인-쿠폰-발급

Dr-KoKo reviewed Aug 16, 2024

View reviewed changes

src/main/java/camp/woowak/lab/coupon/domain/Coupon.java Show resolved Hide resolved

src/main/java/camp/woowak/lab/coupon/domain/CouponIssuance.java Show resolved Hide resolved

Hyeon-Uk requested changes Aug 16, 2024

View reviewed changes

src/main/java/camp/woowak/lab/coupon/domain/CouponIssuance.java Outdated Show resolved Hide resolved

[feat] CouponIssuance 수정

94cfa48

- 발급 시간 변경 제한 Co-authored-by: 김현욱 <43038815+Hyeon-Uk@users.noreply.github.com>

Hyeon-Uk approved these changes Aug 16, 2024

View reviewed changes

Dr-KoKo approved these changes Aug 16, 2024

View reviewed changes

kimhyun5u merged commit c838237 into main Aug 16, 2024
1 check passed

kimhyun5u deleted the feature/75_kimhyun5u_할인-쿠폰-발급 branch August 16, 2024 03:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 할인 쿠폰 발급 #82

[feat] 할인 쿠폰 발급 #82

kimhyun5u commented Aug 15, 2024 •

edited

Loading

Hyeon-Uk left a comment

Hyeon-Uk left a comment

Dr-KoKo left a comment

[feat] 할인 쿠폰 발급 #82

[feat] 할인 쿠폰 발급 #82

Conversation

kimhyun5u commented Aug 15, 2024 • edited Loading

💡 다음 이슈를 해결했어요.

Issue Link - #75

💡 이슈를 처리하면서 추가된 코드가 있어요.

CouponIssuance

IssueCouponeService

💡 이런 고민을 했어요.

동시성 제어 성능 고민

CouponIssuance 는 어디에 ?

✅ 셀프 체크리스트

Hyeon-Uk left a comment

Choose a reason for hiding this comment

Hyeon-Uk left a comment

Choose a reason for hiding this comment

Dr-KoKo left a comment

Choose a reason for hiding this comment

kimhyun5u commented Aug 15, 2024 •

edited

Loading