[feat] 할인 쿠폰 등록 #71

kimhyun5u · 2024-08-15T10:14:39Z

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

Issue Link - #52

할인 쿠폰 등록
고정 할인 금액, 개수, 만료일을 입력해 할인 쿠폰을 등록할 수 있다.
할인 쿠폰의 제목을 입력할 수 있다.

개발을 진행하면서 다음과 같은 제약 조건을 추가했습니다.

쿠폰의 할인 금액은 0원 이상이어야 한다.
쿠폰의 만료일은 현재 시간 이후여야 한다.
쿠폰의 개수는 0개 이상이어야 한다.
쿠폰 제목은 빈값일 수 없다.
쿠폰 제목은 유일하다.
쿠폰의 제목은 100자 이하여야 한다.

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

Coupon: 쿠폰의 Entity
CreateCouponService: 쿠폰을 등록하는 서비스
CouponRepository: 쿠폰의 Repository
CouponApiController: 쿠폰의 ApiController

💡 이런 고민을 했어요.

처음 쿠폰 등록의 명칭을 issue 라고 생각했는데 issue 는 쿠폰을 구매자에게 발급할 때가 더 적합한 것 같아서 create로 수정했습니다!
쿠폰을 일단 등록한 뒤 사용자에게 부여하는 기록 테이블을 만들어 사용하는 게 좋아보입니다!

✅ 셀프 체크리스트

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

- '쿠폰은 고정 할인 금액, 개수, 만료일을 입력해 할인 쿠폰을 등록할 수 있다.' 요구사항에 맞도록 생성 테스트

- discountAmount: 고정 할인 금액 - quantity: 개수 - expired: 만료일

- coupons -> Coupons

- IllegalArgumentException -> InvalidCreationCouponException

- 잘못된 쿠폰 생성 시 발생하는 예외

- 테스트 환경을 수량이 0 미만, 할인 금액이 0 미만인 경우로 변경

- 쿠폰의 수량, 할인 금액, 만료일을 검증한다.

- CouponValidator 를 통한 검증 로직 추가

- IssueCouponServiceTest.testIssueCoupon: '쿠폰은 고정 할인 금액, 개수, 만료일을 입력해 할인 쿠폰을 등록할 수 있다.' 요구 사항에 맞춘 쿠폰 발급 테스트

- CouponFixture.createCoupon: test 에서 사용할 Coupon 객체를 생성

- Coupon은 제목을 가질 수 있다. - Coupon의 제목은 1자 이상, 100자 이하여야 한다.

- 쿠폰에 제목 추가

- 제목 검증 로직 추가

- Coupon field 추가에 따른 변경 사항

- test 내에서 사용할 TestCoupon 구현

- IssueCouponService.issueCoupon: 쿠폰 제목, 할인 금액, 수량, 만료일을 받아 쿠폰을 발급한다.

- 하위 예외 전파 테스트

- IssueCouponService.issueCoupon ``@Transactional` 추가

- 코드 스타일에 맞게 변경

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

- CouponApiControllerTest.testIssueCoupon 검증 부분 공통 응답 형식에 맞게 변경

- CouponApiController.issueCoupon: 응답에 맞게 쿠폰을 생성

- `@DisplayName` 등록

- Exception Handling 범위를 CustomerApiController 로 수정

- Request 제약 조건 추가

- RequestBody 제약조건 검증로직 추가

- assert 시 한 가지의 메소드만 수행하도록 수정

- IssueCouponServiceIntegrationTest.testFailWhenDuplicatedTitle: 쿠폰 발급 시 중복된 제목 입력 시 예외 처리 테스트

- title Unique 값으로 지정

- CouponErrorCode.DUPLICATE_COUPON_TITLE: 중복된 제목 입력 시 발생하는 에러 코드

- 중복된 제목 입력 시 발생하는 예외

- 중복된 제목 쿠폰 발행 시 예외 처리

- CouponErrorCode.DUPLICATE_COUPON_TITLE 상태코드 Conflict 로 변경

- CouponApiControllerTest.testIssueCouponFailWithDuplicateTitle: 중복된 제목 입력 시 실패 처리 테스트

- CouponExceptionHandler.handleDuplicateCouponTitleException: DuplicationCouponTitleException 을 처리한다.

- 쿠폰 등록의 명칭은 issue 보다 create 가 더 적합한 것 같아 변경했습니다. - issue 라는 행위는 추후 구매자에게 발급되는 경우에 사용하는 게 더 적합해보입니다.

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

src/main/java/camp/woowak/lab/web/api/coupon/CouponExceptionHandler.java

- title 검증 시 isBlank() 를 사용해 안정성 향상 Co-authored-by: 김현욱 <43038815+Hyeon-Uk@users.noreply.github.com>

…u_할인-쿠폰-등록

… into feature/52_kimhyun5u_할인-쿠폰-등록

Hyeon-Uk

LGTM :~)

june-777

확인 완료입니다! 고생하셨습니다 👍

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

kimhyun5u added 30 commits August 15, 2024 15:41

[test] CouponTest 추가

5afcd18

- '쿠폰은 고정 할인 금액, 개수, 만료일을 입력해 할인 쿠폰을 등록할 수 있다.' 요구사항에 맞도록 생성 테스트

[feat] Coupon 구현

6347dbe

- discountAmount: 고정 할인 금액 - quantity: 개수 - expired: 만료일

[feat] @table name 변경

25f9e88

- coupons -> Coupons

[feat] 예외 클래스 수정

293cbef

- IllegalArgumentException -> InvalidCreationCouponException

[feat] CouponErrorCode 구현

1661289

[feat] InvalidCreationCouponException 구현

d496710

- 잘못된 쿠폰 생성 시 발생하는 예외

[fix] 테스트 조건 수정

06d30e8

- 테스트 환경을 수량이 0 미만, 할인 금액이 0 미만인 경우로 변경

[feat] CouponValidator 추가

3b7efe1

- 쿠폰의 수량, 할인 금액, 만료일을 검증한다.

[feat] Coupon 생성자 수정

4c28b2b

- CouponValidator 를 통한 검증 로직 추가

[test] IssueCouponServiceTest 추가

6292a38

- IssueCouponServiceTest.testIssueCoupon: '쿠폰은 고정 할인 금액, 개수, 만료일을 입력해 할인 쿠폰을 등록할 수 있다.' 요구 사항에 맞춘 쿠폰 발급 테스트

[feat] CouponFixture 구현

2382e62

- CouponFixture.createCoupon: test 에서 사용할 Coupon 객체를 생성

[test] Coupon 요구 사항 추가에 의한 수정

a3d6710

- Coupon은 제목을 가질 수 있다. - Coupon의 제목은 1자 이상, 100자 이하여야 한다.

[feat] Coupon 수정

6635657

- 쿠폰에 제목 추가

[feat] CouponValidator 수정

d8f4d62

- 제목 검증 로직 추가

[fix] CouponFixture 수정

2f678f2

- Coupon field 추가에 따른 변경 사항

[fix] IssueCouponServiceTest 수정

08ac7d0

- Coupon field 추가에 따른 변경 사항

[feat] TestCoupon 구현

7147dd3

- test 내에서 사용할 TestCoupon 구현

[feat] CouponRepository 구현

88b052a

[feat] IssueCouponCommand 구현

4235ae8

[feat] IssueCouponService 구현

1d47464

- IssueCouponService.issueCoupon: 쿠폰 제목, 할인 금액, 수량, 만료일을 받아 쿠폰을 발급한다.

[style] 잘못된 공백 제거

fdceeb4

[test] IssueCouponServiceIntegrationTest 구현

9452b44

- 하위 예외 전파 테스트

[feat] IssueCouponService 수정

64db273

- IssueCouponService.issueCoupon ``@Transactional` 추가

[refactor] CouponErrorCode 수정

3000100

- 코드 스타일에 맞게 변경

[test] CouponApiControllerTest 구현

ec4bba1

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

[fix] CouponApiControllerTest 수정

0e8d4e9

- CouponApiControllerTest.testIssueCoupon 검증 부분 공통 응답 형식에 맞게 변경

[feat] CouponApiController 구현

247af9f

- CouponApiController.issueCoupon: 응답에 맞게 쿠폰을 생성

[feat] IssueCouponRequest 구현

a025747

[feat] IssueCouponResponse 구현

ae5dec5

[docs] CouponApiControllerTest.testIssueCoupon 수정

098eea8

- `@DisplayName` 등록

kimhyun5u added 14 commits August 15, 2024 18:31

[fix] CustomerExceptionHandler 범위 수정

1664829

- Exception Handling 범위를 CustomerApiController 로 수정

[feat] IssueCouponRequest 제약 조건 추가

15cae15

- Request 제약 조건 추가

[feat] CouponApiController.issueCoupon 수정

d79dbce

- RequestBody 제약조건 검증로직 추가

[style] CouponApiControllerTest 코드 스타일 수정

efbe8f5

[fix] IssueCouponServiceIntegrationTest 수정

30913e8

- assert 시 한 가지의 메소드만 수행하도록 수정

[test] IssueCouponServiceIntegrationTest 수정

49d75a0

- IssueCouponServiceIntegrationTest.testFailWhenDuplicatedTitle: 쿠폰 발급 시 중복된 제목 입력 시 예외 처리 테스트

[feat] Coupon 수정

8c44d68

- title Unique 값으로 지정

[feat] CouponErrorCode 수정

5e0d8c0

- CouponErrorCode.DUPLICATE_COUPON_TITLE: 중복된 제목 입력 시 발생하는 에러 코드

[feat] DuplicateCouponTitleException 구현

543891d

- 중복된 제목 입력 시 발생하는 예외

[feat] IssueCouponService.issueCoupon 수정

cf4cc90

- 중복된 제목 쿠폰 발행 시 예외 처리

[fix] CouponErrorCode 수정

bd2e52e

- CouponErrorCode.DUPLICATE_COUPON_TITLE 상태코드 Conflict 로 변경

[test] CouponApiControllerTest.testIssueCouponFailWithDuplicateTitle 추가

21c0a0f

- CouponApiControllerTest.testIssueCouponFailWithDuplicateTitle: 중복된 제목 입력 시 실패 처리 테스트

[feat] CouponExceptionHandler 구현

28340e2

- CouponExceptionHandler.handleDuplicateCouponTitleException: DuplicationCouponTitleException 을 처리한다.

[refactor] 등록 용어 변경

90b8236

- 쿠폰 등록의 명칭은 issue 보다 create 가 더 적합한 것 같아 변경했습니다. - issue 라는 행위는 추후 구매자에게 발급되는 경우에 사용하는 게 더 적합해보입니다.

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

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

kimhyun5u self-assigned this Aug 15, 2024

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

[기능] 할인 쿠폰 등록 #52

Closed

kimhyun5u requested a review from a team August 15, 2024 10:14

Hyeon-Uk requested changes Aug 15, 2024

View reviewed changes

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

src/main/java/camp/woowak/lab/web/api/coupon/CouponExceptionHandler.java Show resolved Hide resolved

kimhyun5u and others added 4 commits August 15, 2024 19:28

[fix] CouponValidator 수정

b6b1724

- title 검증 시 isBlank() 를 사용해 안정성 향상 Co-authored-by: 김현욱 <43038815+Hyeon-Uk@users.noreply.github.com>

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

0967102

…u_할인-쿠폰-등록

[merge] remote-tracking branch 'origin/feature/52_kimhyun5u_할인-쿠폰-등록'…

88c02aa

… into feature/52_kimhyun5u_할인-쿠폰-등록

[feat] CouponExceptionHandler 로깅 추가

ce67849

Hyeon-Uk approved these changes Aug 15, 2024

View reviewed changes

kimhyun5u requested review from june-777 and Dr-KoKo August 15, 2024 10:53

june-777 approved these changes Aug 15, 2024

View reviewed changes

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

kimhyun5u merged commit eab39b1 into main Aug 15, 2024
1 check passed

kimhyun5u deleted the feature/52_kimhyun5u_할인-쿠폰-등록 branch August 15, 2024 11:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 할인 쿠폰 등록 #71

[feat] 할인 쿠폰 등록 #71

kimhyun5u commented Aug 15, 2024 •

edited

Loading

Hyeon-Uk left a comment

june-777 left a comment •

edited

Loading

[feat] 할인 쿠폰 등록 #71

[feat] 할인 쿠폰 등록 #71

Conversation

kimhyun5u commented Aug 15, 2024 • edited Loading

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

Issue Link - #52

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

💡 이런 고민을 했어요.

✅ 셀프 체크리스트

Hyeon-Uk left a comment

Choose a reason for hiding this comment

june-777 left a comment • edited Loading

Choose a reason for hiding this comment

kimhyun5u commented Aug 15, 2024 •

edited

Loading

june-777 left a comment •

edited

Loading