BlockService 리팩토링 및 테스트 작성 - Githubissues

Instagram-Clone-Coding / Spring_instagram-clone

Instagram Clone Coding - Backend using Spring Framework

http://ec2-52-79-71-191.ap-northeast-2.compute.amazonaws.com

MIT License

82 stars 23 forks source link

BlockService 리팩토링 및 테스트 작성 #218

Closed vectorch9 closed 2 years ago

vectorch9 commented 2 years ago

📌Linked Issues

Resolve: #214

✏Change Details

리팩토링

예외 및 에러코드 수정
축약할 수 있는 부분 수정

💬Comment

@seonpilKim 컨벤션 및 테스트 작성 스타일에 대해 의논하고자 Draft PR을 먼저 올립니다. 의논할 만한 부분에 PR 리뷰(커멘트)를 달테니 확인해주세요 !

예외에 빈줄 컨벤션 적용이 안되어있는데 이는 수정하겠습니다.

📑References

✅Check List

[ ] 추가한 기능에 대한 테스트는 모두 완료하셨나요?
[ ] 코드 정렬(Ctrl + Alt + L), 불필요한 코드나 오타는 없는지 확인하셨나요?

vectorch9 commented 2 years ago

BlockService 테스트 작성

두 메서드에 관련된 테스트 작성
가능한 모든 경우를 테스트 하였고, 테스트 명도 최대한 직관적으로 지었습니다.

의논하고 싶은 사항으로, unblock 테스트 메서드 중 아래와 같은 코드가 나옵니다.

final cloneproject.Instagram.domain.member.entity.Block block = cloneproject.Instagram.domain.member.entity.Block.builder()
            .member(member)
            .blockMember(target)
            .build();

BlockServiceTest내에 테스트 용 Block 클래스가 있어서 본래목적인 Block Entity를 쓰기 위해선 import문을 사용하지 못하고 위와 같이 명시적으로 호출을 하게 됩니다. Entity와 테스트할 메서드 명이 같은 경우가 block 밖에 없긴한데 그냥 두는게 나을까요..?