사용자 가입 시 신뢰도를 null로 설정 #536
Closed
Conversation
This file contains 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
apptie
approved these changes
Oct 5, 2023
| @@ -23,7 +23,7 @@ | |||
| @Entity | |||
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | |||
| @Getter | |||
| @EqualsAndHashCode(of = "id") | |||
| @EqualsAndHashCode(of = "id", callSuper = false) | |||
swonny
requested changes
Oct 5, 2023
| @@ -195,6 +196,15 @@ class AuctionServiceTest extends AuctionServiceFixture { | |||
| .hasMessage("지정한 아이디에 대한 경매를 찾을 수 없습니다."); | |||
| } | |||
|
|
|||
| @Test | |||
| void 지정한_아이디에_해당하는_경매의_판매자_신뢰도가_null이라면_서비스에서_반환하는_dto에서_판매자_신뢰도를_나타내는_부분도_null이다() { | |||
There was a problem hiding this comment.
선택
굉장히 자세한 메서드 네이밍이네요..!
그런데 해당 로직은 auction보다는 dto의 getter를 테스트하는 느낌이 더 강한 것 같습니다.
auctionServiceTest에서 진행해주신 이유가 있으신지 궁금합니다.
There was a problem hiding this comment.
혹시나 NPE가 발생하는지 궁금해서 개인적인 테스트를 해보았는데, 원래는 결과만 보고 롤백하려고 했는데 나중에 또 궁금할까 봐 있으면 좋을 것 같아서 커밋하였습니다. 처음 의도는 원래 커밋은 할 생각을 안 했고 그냥 개인적으로 확인차 테스트했던 거라 픽스처 클래스 만들고 하기 귀찮아서 있던 테스트에서 같이 했는데 커밋할 때는 또 생각 없이 해버렸네요ㅠ
그래서 결론은 dto 테스트로 옮기겠습니다~
예리하시네요 메리!
JJ503
approved these changes
Oct 5, 2023
| @@ -67,7 +67,7 @@ private User findOrPersistUser(final Oauth2Type oauth2Type, final UserInformatio | |||
| final User user = User.builder() | |||
| .name(oauth2Type.calculateNickname(calculateRandomNumber())) | |||
| .profileImage(findDefaultProfileImage()) | |||
| .reliability(0.0d) | |||
| .reliability(null) | |||
There was a problem hiding this comment.
질문
값 객체로 변경하게 된다면 null을 직접적으로 넣지 않고, 미리 null로 설정해 둔 값객체에 대한 static 필드를 사용하게 될까요?
There was a problem hiding this comment.
넵 값객체 적용한 pr에서 User 생성 시 reliability가 null이면 static 필드를 사용하도록 구현되어 있습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📄 작업 내용 요약
사용자 가입 시 신뢰도를 null로 설정
🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드
질문
커밋은 하지 않았지만 개인적으로 신뢰도 정렬 테스트코드 픽스처에 신뢰도가 null인 판매자의 경매를 추가하고 신뢰도 정렬을 해보았는데, 신뢰도가 null이면 제일 후순위이더라구요. 그래서 신뢰도 정렬을 했을 때 신뢰도가 0점인 사람보다 뒤에 보이게 되는데, 이 부분 관련해서 정책 회의가 필요할 것 같습니다.
📎 Issue 번호