GETP-113 feat: 피플 좋아요 취소 기능 구현

[wlgns12370]

✨ 구현한 기능

• 피플 좋아요 취소 기능

📢 논의하고 싶은 내용

• PeopleLikeService.get(Long Long) 메서드를 테스트를 하지 않는다면 private 처리를 할 수 있는데 테스트를 위해서 public으로 변경하였습니다. 이런 경우 최종 배포전 테스트까지는 public으로 두다가 private으로 변경하는 게 어떨까요? 이 부분은 잘 몰라서 논의해 보고 싶네요!
• 객체지향 다형성을 높이기 위해서 PeopleLikeService.get메서드에 오버로딩을 적용해 보았습니다! 해당 부분에 대해서 의견 부탁드립니다!
  PeopleService 에 getByMemberId와 getByPeopleId는 매개변수가 Long타입 하나로 중복됩니다. 하지만 매개변수 타입이 중복되지 않는 ServiceClass의 get메서드는 앞으로 오버로딩 하는 방법으로 사용하면 어떨까요?

private PeopleLike get(Optional<PeopleLike> peopleLike) {
    return peopleLike.orElseThrow(() -> new BusinessLogicException(PeopleLikeErrorCode.PEOPLE_LIKE_NOT_FOUND));
}

public PeopleLike get(Long memberId, Long peopleId) {
    return get(peopleLikeRepository.findByPeople_PeopleIdAndClient_ClientId(memberId, peopleId));
}

🎸 기타

• TestFixture 파일도 Domain 기준으로 분리하는 게 어떨까요??

[scv1702]

테스트 Fixture에 관한 것은 domain/{domainName}/support 또는 domain/{domainName}/fixture 이런 식으로 분리하는 것이 저도 좋아보여요.

[scv1702]

수고하셨습니다! 리뷰 참고해주세요.

[github-actions]

Test Results

102 tests  +4   102 ✅ +4   19s ⏱️ +2s
 69 suites  - 1     0 💤 ±0 
 69 files    - 1     0 ❌ ±0 

Results for commit f7691ee. ± Comparison against base commit b90f2b7.