Conversation
Feature/week1 user register
사용자 등록 spy 테스트 추가
사용자 정보 조회 기능 구현
포인트 도메인 기능 구현
# Conflicts: # apps/commerce-api/src/test/java/com/loopers/domain/point/PointServiceIntegrationTest.java # apps/commerce-api/src/test/java/com/loopers/domain/user/UserUnitTest.java # apps/commerce-api/src/test/java/com/loopers/interfaces/api/PointV1E2ETest.java
포인트 사용 기능 추가
# Conflicts: # .codeguide/loopers-1-week.md # apps/commerce-api/src/main/java/com/loopers/CommerceApiApplication.java # apps/commerce-api/src/main/java/com/loopers/application/example/ExampleFacade.java # apps/commerce-api/src/main/java/com/loopers/application/example/ExampleInfo.java # apps/commerce-api/src/main/java/com/loopers/domain/example/ExampleModel.java # apps/commerce-api/src/main/java/com/loopers/domain/example/ExampleService.java # apps/commerce-api/src/main/java/com/loopers/infrastructure/example/ExampleJpaRepository.java # apps/commerce-api/src/main/java/com/loopers/infrastructure/example/ExampleRepositoryImpl.java # apps/commerce-api/src/main/java/com/loopers/interfaces/api/ApiControllerAdvice.java # apps/commerce-api/src/main/java/com/loopers/interfaces/api/ApiResponse.java # apps/commerce-api/src/main/java/com/loopers/interfaces/api/example/ExampleV1ApiSpec.java # apps/commerce-api/src/main/java/com/loopers/interfaces/api/example/ExampleV1Controller.java # apps/commerce-api/src/main/java/com/loopers/interfaces/api/example/ExampleV1Dto.java # apps/commerce-api/src/main/java/com/loopers/support/error/ErrorType.java # apps/commerce-api/src/test/java/com/loopers/domain/example/ExampleModelTest.java # apps/commerce-api/src/test/java/com/loopers/domain/example/ExampleServiceIntegrationTest.java # apps/commerce-api/src/test/java/com/loopers/interfaces/api/ExampleV1ApiE2ETest.java # apps/commerce-api/src/test/java/com/loopers/support/error/CoreExceptionTest.java
1주차 작업물 병합
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Summary
💬 Review Points
ApiControllerAdvice.java로 공통으로 예외 사항을 적용하는 방식으로 일관성있는 에러 메시지 응답을 구성하였습니다. 이 부분에서 이후로 다양한 예외를 고려할때 마다 추가를 하면 너무 많은 코드량이 나올것 같은데, 멘토님들은 어떤식으로 예외 처리를 유지보수를 원활하게 하시는지 궁금합니다.PointRepository.findByUsername에서 사용자 정보 기반으로 조회하는 로직을 구성하였는데, 없을 경우 null 을 던지는 방식으로 구성하였습니다. 이럴경우 어느 레이어에서 명확히 null 예외를 처리 하는것이 좋을지 리뷰를 받고 싶습니다.✅ Checklist
회원 가입
🧱 단위 테스트
영문 및 숫자 10자 이내형식에 맞지 않으면, User 객체 생성에 실패한다.xx@yy.zz형식에 맞지 않으면, User 객체 생성에 실패한다.yyyy-MM-dd형식에 맞지 않으면, User 객체 생성에 실패한다.🔗 통합 테스트
🌐 E2E 테스트
400 Bad Request응답을 반환한다.내 정보 조회
🔗 통합 테스트
🌐 E2E 테스트
404 Not Found응답을 반환한다.포인트 조회
🔗 통합 테스트
🌐 E2E 테스트
X-USER-ID헤더가 없을 경우,400 Bad Request응답을 반환한다.포인트 충전
🧱 단위 테스트
🔗 통합 테스트
🧱 통합 테스트
📎 References