기본 이름이 중복되어 생성되는 문제 해결 #30
Conversation
| public static class NullOrEmptyNameException extends MemberException { | ||
|
|
||
| public NullOrEmptyNameException() { | ||
| super(ExceptionMessage.NULL_OREMPTY_NAME); |
There was a problem hiding this comment.
OR와 EMPTY 사이에 언더바 하나 넣어주는 것이 어떨까요?
| @@ -45,6 +45,22 @@ class AuthenticationServiceTest extends AuthenticationServiceTestFixture { | |||
| }); | |||
| } | |||
|
|
|||
| @Test | |||
| void 로그인시_존재하지_않는_사용자이며_oauthid가_50자를_초과하는_경우_이름을_50자까지만_저장한_후_토큰_정보를_반환한다() { | |||
There was a problem hiding this comment.
실제로 이름이 50자까지 저장되었는지 비교검증은 필요없을까요?
There was a problem hiding this comment.
해당 부분에 대한 검증에 대해서 고민 중에 있었습니다.
일단 현재 방식으로 한 이유는 50자를 초과하면 오류가 발생해 반환 자체가 안 될 것이기 때문에 정상처리 되었을 것이라 이해했습니다.
그리고 반환된 토큰에서 id를 도출하고 userRepository를 통해 사용자 정보를 가져오는 게 조금 귀찮아서 일단, when 절에서 호출한 메서드를 통해 도출된 값으로 검증을 진행했습니다.
그런데, 효선님께서 말씀하신 방식이 좀 더 이해가 쉬울 것 같다는 생각을 하기도 했습니다.
그래서 그 편이 더 이해가 쉽다면 해당 방식으로 진행하도록 하겠습니다!
There was a problem hiding this comment.
해당 테스트에 비교 검증하는 부분이 있다면 이해가 더 쉬울 것 같아 이 부분만 추가해주시면 좋을 것 같습니다!
There was a problem hiding this comment.
헉 테스트하길 잘했어요..!
제가 substring을 아무 생각 없이 했더니 잘못 걸어뒀더라고요..!🥲
역시 귀찮다고 테스트를 허투루 하면 안 되겠군요...
좋은 의견 감사합니다!
| @@ -119,6 +110,10 @@ public String getEmail() { | |||
| return email.getValue(); | |||
| } | |||
|
|
|||
| public String getName() { | |||
There was a problem hiding this comment.
vo 값을 가져오는 경우 해당 vo에서 get 해오도록 했는데 정수님 코드 보고 도메인 엔티티에 직접 get 메서드 정의해 사용하면 좋을 것 같아서 이 부분도 수정하도록 하겠습니다!
| @@ -45,6 +45,22 @@ class AuthenticationServiceTest extends AuthenticationServiceTestFixture { | |||
| }); | |||
| } | |||
|
|
|||
| @Test | |||
| void 로그인시_존재하지_않는_사용자이며_oauthid가_50자를_초과하는_경우_이름을_50자까지만_저장한_후_토큰_정보를_반환한다() { | |||
There was a problem hiding this comment.
해당 테스트에 비교 검증하는 부분이 있다면 이해가 더 쉬울 것 같아 이 부분만 추가해주시면 좋을 것 같습니다!
|
도메인의 객체 생성 시 값이 null이 들어오는 필드에 대해 기본 값으로 설정해 주기로 한 적이 있는데,
name의 경우unique인 것을 까먹고 수정을 진행했습니다.그래서 두 명 이상의 사용자 생성 시 문제가 발생하게 되는 경우가 존재해 이에 대해 해결하기 위한 pr입니다.
수정 사항은 간단하기에 확인해 주시면 감사하겠습니다!
질문