구매자 회원가입 #16

kimhyun5u · 2024-08-11T05:17:54Z

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

Issue Link - #4

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

Customer

Vendor 생성과 동일하게 생성 시 검증로직을 통과해야합니다.

SignUpCustomerService

Customer 를 생성하는 서비스 입니다.
생성 중 RuntimeException 이 발생하면 롤백 되도록 메소드 단위에서 @Transactional 걸어 주었습니다.

CustomerController

Cutomer 컨트롤러입니다.
서비스를 활용해 비즈니스 로직을 수행한 후 유저에게 응답을 전달합니다.

💡 이런 고민을 했어요.

Exception vs RuntimeException

Service 에서 @Transactional 로 묶으면 예기치 못한 RuntimeException 도 롤백할 수 있도록 CustomException을 RuntimeException 을 상속하도록함

✅ 셀프 체크리스트

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

- NoOpPasswordEncoder: 아무런 기능을 제공하지 않는 PasswordEncoder

- DuplicateException: unique 제약조건에 부합하지 않을 때 발생하는 예외 - InvalidCreationException: 기타 제약조건에 부합하지 않는 경우 발생하는 예외

- 구매자 회원가입에 필요해 미리 구현

- 구매자 회원가입 테스트 구현 - 정상 회원가입 테스트 - 중복 이메일 회원가입 불가 테스트

- 이름, 이메일, 비밀번호, 전화번호 필드 추가 - 필드 검증 메서드 추가

- 구매자 영속화를 위한 Repository 인터페이스 구현

- SignUpCustomerService.signUp 파라미터 구현

- 구매자 회원가입 구현

`+` AuthController: "/auth" 컨텍스트 처리 `+` ApiResponse: 공통 응답처리를 위한 모델 `+` ErrorCode: 공통 예외처리를 위한 코드 `+` SignUpVendorRequest: POST /auth/vendor 요청을 받는 dto

- 구매자 회원가입 요청 DTO 생성

- 구매자 회원가입 API 구현

- 구매자 회원가입 API 클래스명 변경

- 구매자 제약조건 테스트(점주 제약조건 참조함)

전화번호를 검증하기 위한 어노테이션 및 커스텀 Validator 구현

`~` show-sql: 실행되는 sql 조회 `~` generate-ddl: 테스트시 테이블 자동 생성 `~` ddl-auto: 어플리케이션 생성시 생성, 종료시 삭제 `~` open-in-view: 의도하지 않은 조회 쿼리 발생 방지

- 구매자 API RequestBody 제약조건 테스트

- 이름 길이 1~50 - 이메일 - 비밀번호 8~20 - 전화번호

- Repository 의존성 제거

- 반복되는 생성 추상화

- given, when, then 추가

- 파라미터 검증 시 항상 잘못된 전화번호 형식이 들어가는 오류 수정

- 중복 이메일 회원가입 시 롤백 테스트 작성

@transactional

- @transactional 추가

@transactional

- RuntimeException 을 사용해 @transactional 내의 예상하지 못한 RuntimeException이 발생해도 트랜잭션이 롤백되도록 수정

@notblank

- email 필드에 @notblank 추가 - 이메일이 "" 이어도 통과되는 문제 수정

- 중복 이메일 에러 처리 정책 변경으로 인한 테스트 코드 수정

…4_kimhyun5u_구매자_회원가입 # Conflicts: # src/main/java/camp/woowak/lab/common/exception/ErrorCode.java # src/main/java/camp/woowak/lab/common/exception/HttpStatusException.java

- ErrorCode 삭제 후 CustomerErrorCode 적용

- CustomerController -> CustomerApiController

- camp.woowak.lab.web.dto.request.customer 밑으로 이동

- BadRequestException(ErrorCode, String) 생성자 추가

- 서버에 남기고 싶은 로그는 직접 입력하도록 수정 - 에러코드에 있는 메시지는 클라이언트를 위한 것

- InvalidCreationException(ErrorCode, String) 생성자 추가

- SignUp API 응답의 Response DTO 추가

- 유틸리티 클래스에 private 생성자 추가

…매자_회원가입

…매자_회원가입 # Conflicts: # src/main/java/camp/woowak/lab/customer/domain/Customer.java # src/main/java/camp/woowak/lab/payaccount/repository/PayAccountRepository.java # src/main/java/camp/woowak/lab/web/authentication/NoOpPasswordEncoder.java # src/main/java/camp/woowak/lab/web/authentication/PasswordEncoder.java # src/main/java/camp/woowak/lab/web/authentication/config/AuthenticationConfig.java # src/test/java/camp/woowak/lab/payaccount/domain/TestPayAccount.java

Dr-KoKo

잘 봤습니다. LGTM!

src/main/java/camp/woowak/lab/customer/domain/CustomerValidator.java

src/main/java/camp/woowak/lab/customer/service/SignUpCustomerService.java

src/main/java/camp/woowak/lab/web/dto/request/SignUpVendorRequest.java

june-777

고생많으셨습니다 👍

src/main/java/camp/woowak/lab/customer/domain/CustomerValidator.java

src/main/java/camp/woowak/lab/customer/service/SignUpCustomerService.java

- Validation 실패 시 메시지 명시

- Validate 시 에서 필요한 값을 Validator 의 field 로 추출

- Validation 실패 시 메시지 명시

kimhyun5u and others added 27 commits August 10, 2024 19:59

[chore] .gitkeep 삭제

17a4cc8

[feat] PasswordEncoder 추가

38c81a2

- NoOpPasswordEncoder: 아무런 기능을 제공하지 않는 PasswordEncoder

[feat] Vendor CustomException 추가

deb1f8b

- DuplicateException: unique 제약조건에 부합하지 않을 때 발생하는 예외 - InvalidCreationException: 기타 제약조건에 부합하지 않는 경우 발생하는 예외

[feat] PayAccountRepository 구현

030a8c0

- 구매자 회원가입에 필요해 미리 구현

[test] SignUpCustomerServiceTest

bffe230

- 구매자 회원가입 테스트 구현 - 정상 회원가입 테스트 - 중복 이메일 회원가입 불가 테스트

[feat] Customer 회원가입 조건에 맞도록 수정

0313777

- 이름, 이메일, 비밀번호, 전화번호 필드 추가 - 필드 검증 메서드 추가

[feat] CustomerRepository 구현

d7482dc

- 구매자 영속화를 위한 Repository 인터페이스 구현

[feat] SignUpCustomerCommand 구현

8a94701

- SignUpCustomerService.signUp 파라미터 구현

[feat] SignUpCustomerService 구현

04f61a0

- 구매자 회원가입 구현

[feat] Vendor 회원가입 전송 계층 구현

2900681

`+` AuthController: "/auth" 컨텍스트 처리 `+` ApiResponse: 공통 응답처리를 위한 모델 `+` ErrorCode: 공통 예외처리를 위한 코드 `+` SignUpVendorRequest: POST /auth/vendor 요청을 받는 dto

[feat] SignUpCustomerRequest 구현

4d6a20c

- 구매자 회원가입 요청 DTO 생성

[feat] AuthController.signUpCustomer 구현

f93f83d

- 구매자 회원가입 API 구현

[refactor] AuthController -> CustomerController

c1bd5dc

- 구매자 회원가입 API 클래스명 변경

[test] Customer 테스트

07cd051

- 구매자 제약조건 테스트(점주 제약조건 참조함)

[feat] Phone 검증 기능

17fb832

전화번호를 검증하기 위한 어노테이션 및 커스텀 Validator 구현

[feat] jpa 설정

bbafb20

`~` show-sql: 실행되는 sql 조회 `~` generate-ddl: 테스트시 테이블 자동 생성 `~` ddl-auto: 어플리케이션 생성시 생성, 종료시 삭제 `~` open-in-view: 의도하지 않은 조회 쿼리 발생 방지

[test] CustomerController 테스트

a9201d7

- 구매자 API RequestBody 제약조건 테스트

[feat] SignUpCustomerRequest 제약 조건 추가

2d54d49

- 이름 길이 1~50 - 이메일 - 비밀번호 8~20 - 전화번호

[feat] CustomerController.signUp 파라미터 Valid 추가

98f450b

[refactor] SignUpCustomerServiceTest 수정

6d718e5

- Repository 의존성 제거

[feat] CustomerFixture 추가

5197be3

- 반복되는 생성 추상화

[chore] CustomerControllerTest 주석 추가

476489c

- given, when, then 추가

[fix] CustomerControllerTest 로직 오류 수정

4adf5de

- 파라미터 검증 시 항상 잘못된 전화번호 형식이 들어가는 오류 수정

[test] SignUpCustomerServiceIntegrationTest 추가

5f51c1e

- 중복 이메일 회원가입 시 롤백 테스트 작성

[feat] SignUpCustomerService.signUp 수정

1ceeb0d

- @transactional 추가

[feat] DuplicateException, InvalidCreationException 수정

8dcc908

- RuntimeException 을 사용해 @transactional 내의 예상하지 못한 RuntimeException이 발생해도 트랜잭션이 롤백되도록 수정

[fix] SignUpCustomerRequest

f0aa992

- email 필드에 @notblank 추가 - 이메일이 "" 이어도 통과되는 문제 수정

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

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

kimhyun5u requested a review from Hyeon-Uk August 11, 2024 05:17

kimhyun5u added 16 commits August 13, 2024 09:20

[test] CustomerControllerTest 수정

140dbcc

- 중복 이메일 에러 처리 정책 변경으로 인한 테스트 코드 수정

Merge remote-tracking branch 'refs/remotes/origin/main' into feature/…

e2bd830

…4_kimhyun5u_구매자_회원가입 # Conflicts: # src/main/java/camp/woowak/lab/common/exception/ErrorCode.java # src/main/java/camp/woowak/lab/common/exception/HttpStatusException.java

[chore] .gitkeep 제거

9f90cf1

[chore] 사용하지 않는 코드 제거

ca72425

[chore] 사용하지 않는 코드 제거

c38cbdb

[refactor] 패키지 구조 변경

ba7ef95

[test] 에러 코드 정책 변경으로 인한 수정

3b07854

- ErrorCode 삭제 후 CustomerErrorCode 적용

[chore] @repository 제거

d3d1cfc

[chore] CustomerController 이름 변경

6be87cd

- CustomerController -> CustomerApiController

[refactor] SignUpCustomerRequest 패키지 구조 변경

e6b0b0c

- camp.woowak.lab.web.dto.request.customer 밑으로 이동

[feat] BadRequestException 생성자 추가

f6d3346

- BadRequestException(ErrorCode, String) 생성자 추가

[feat] 에러코드 정책 변경에 따른 수정

16ff26c

- 서버에 남기고 싶은 로그는 직접 입력하도록 수정 - 에러코드에 있는 메시지는 클라이언트를 위한 것

[feat] 에러코드 정책 변경에 따른 수정

f932cae

[feat] InvalidCreationException 수정

54840c1

- InvalidCreationException(ErrorCode, String) 생성자 추가

[feat] SignUpCustomerResponse 추가

c8784cd

- SignUp API 응답의 Response DTO 추가

[feat] CustomerValidator private 생성자 추가

6f125c9

- 유틸리티 클래스에 private 생성자 추가

kimhyun5u requested a review from Hyeon-Uk August 13, 2024 05:38

kimhyun5u marked this pull request as ready for review August 13, 2024 05:39

kimhyun5u added 3 commits August 13, 2024 15:13

Merge remote-tracking branch 'origin/main' into feature/4_kimhyun5u_구…

0b1eb38

…매자_회원가입

[style] 코드 스타일 수정 적용

ae0385d

Dr-KoKo approved these changes Aug 13, 2024

View reviewed changes

Hyeon-Uk reviewed Aug 13, 2024

View reviewed changes

june-777 approved these changes Aug 13, 2024

View reviewed changes

src/main/java/camp/woowak/lab/customer/domain/CustomerValidator.java Show resolved Hide resolved

src/main/java/camp/woowak/lab/customer/service/SignUpCustomerService.java Show resolved Hide resolved

kimhyun5u added 4 commits August 13, 2024 16:21

[feat] SignUpCustomerRequest annotation param 수정

7a44fac

- Validation 실패 시 메시지 명시

[refactor] Validation 상수 값 추출

b28cc5d

- Validate 시 에서 필요한 값을 Validator 의 field 로 추출

[feat] SignUpCustomerRequest annotation param 수정

7a1e3fb

- Validation 실패 시 메시지 명시

[chore] 불필요한 throws 제거

b68368a

kimhyun5u merged commit 8c3c80d into main Aug 13, 2024
1 check passed

kimhyun5u deleted the feature/4_kimhyun5u_구매자_회원가입 branch August 13, 2024 08:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

구매자 회원가입 #16

구매자 회원가입 #16

kimhyun5u commented Aug 11, 2024 •

edited

Loading

Dr-KoKo left a comment

june-777 left a comment

구매자 회원가입 #16

구매자 회원가입 #16

Conversation

kimhyun5u commented Aug 11, 2024 • edited Loading

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

Issue Link - #4

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

Customer

SignUpCustomerService

CustomerController

💡 이런 고민을 했어요.

Exception vs RuntimeException

✅ 셀프 체크리스트

Dr-KoKo left a comment

Choose a reason for hiding this comment

june-777 left a comment

Choose a reason for hiding this comment

kimhyun5u commented Aug 11, 2024 •

edited

Loading