[BE] 멤버 도메인 미사용 컬럼 삭제 및 필드명 통일 #956
Conversation
|
|
||
| import site.coduo.member.domain.Member; | ||
|
|
||
| public abstract class MemberDummy { |
There was a problem hiding this comment.
좋네요
그럼 다른 fixture 들도 이런 방식으로 바꾸는게 좋을까요?
There was a problem hiding this comment.
일단 테스트 코드 제가 담당하기로 해서 만약 member같이 중복되는 fixture들이 많으면 제가 최대한 수정해보겠습니다!
There was a problem hiding this comment.
캬 좋습니다 테스트 코드가 한층 더 깔꼼해졌네요
한가지 고민은 몇가지 fixture가 있는데 요 놈들은 test 패키지 하단에 fixture 패키지에 모여있습니다. MemberDummy 클래스는 이 fixture들과는 살짝 성격이 다른 거 같긴(fixture들은 상수 모아둠, memberdummy는 뭔가 메이커 느낌) 하지만 이렇게 패키지 구조를 가져가신 이유가 있으실까요?!
There was a problem hiding this comment.
멤버 관련 리팩터링은 브랜치 member-refactor 에서 한번에 모아서 dev로 가시려는거죠?
네 맞습니다.
한가지 고민은 몇가지 fixture가 있는데 요 놈들은 test 패키지 하단에 fixture 패키지에 모여있습니다. MemberDummy 클래스는 이 fixture들과는 살짝 성격이 다른 거 같긴(fixture들은 상수 모아둠, memberdummy는 뭔가 메이커 느낌) 하지만 이렇게 패키지 구조를 가져가신 이유가 있으실까요?!
우선 fixture package를 보지 못했어요. fixture 패키지로 이동 시켜도 큰 문제 없을 거 같아요.
| final Member me = MemberDummy.createDummy("pairNameA",jwtProvider.sign("pairA"), "pairA","loginIdA"); | ||
| memberRepository.save(me); | ||
|
|
||
| final Member pair = memberRepository.save( | ||
| Member.builder() | ||
| .userId("pairB") | ||
| .accessToken(jwtProvider.sign("pairB")) | ||
| .loginId("pairB") | ||
| .username("pairNameB") | ||
| .profileImage("some image") | ||
| .build() | ||
| ); | ||
| final Member pair =MemberDummy.createDummy("pairNameB",jwtProvider.sign("pairB"), "pairB", "loginIdB"); | ||
| memberRepository.save(pair); | ||
|
|
| final String providerLoginId, | ||
| final String providerUserId, | ||
| final String username, | ||
| final LocalDateTime deletedAt) { |
There was a problem hiding this comment.
요기도 마지막 파라미터 다음 엔터 한번 쳐주세욥
|
|
||
| import site.coduo.member.domain.Member; | ||
|
|
||
| public abstract class MemberDummy { |
연관된 이슈
구현한 기능
상세 설명
테스트 총 실행 시간: 13초 388
에서
7초 536으로 개선
2.SSE (이젠 안쓰이겠지만.. ) Fake 객체 Timeout Duration 500ms -> 1ms으로 변경. 2초가량 개선