인프런 커뮤니티 질문&답변

//CommunityCommandService.java
public void updateCommunity(Long communityId, String description, List<String> newTags) {  
    Community community = communityRepository.findCommunityById(communityId);    
  
    community.updateCommunity(description, newTags);  
}

만약, 이런 코드가 있다고 하면

Service는 데이터를 받고 Community 클래스에게 실제 데이터 변경을 위임합니다.

//Community.java
public void updateCommunity(String description, List<String> tags) {
        this.description = new Description(description);
        this.hashtags.updateTags(tags, this);
}

Service 클래스를 테스트할 때

상태검증으로 테스트한다면,

//CommunityCommandService.java
@Test
void 상태검증_테스트() {
        community = new Community("dummy Intro", List.of("dummy tag"));
        given(communityRepository.findById(any)).willReturn(community);

        communityCommandService.updateCommunity("new intro", List.of("new tag"));

        assertThat(community.getIntroduce).isEqualTo("new intro");
        assertThat(community.getTags).containsExactly("new tag");
    }

이렇게 테스트하게 됩니다.

사실 데이터의 변경 자체는 CommunityTest에서 테스트 하였습니다.

//CommunityTest.java
   @Test
void updateCommunity() {
        final Community community = TestCommunity.builder().build();

        community.updateCommunity("커뮤니티 소개란 입니다.", List.of("태그1", "태그2"));

        assertThat(community.getHashtags()).extracting("tag").containsExactly("태그1", "태그2");
        assertThat(community.getDescription()).isEqualTo("커뮤니티 소개란 입니다.");
    }

이렇게 되면 CommunityCommandService에서 내부 Community.updateCommunity를 중복적으로 검증하는게 아닌지 생각이 듭니다.

객체지향 관점에서, Service는 Community에게 위임하여 상태를 변경시킵니다.

실제 내부에 어떠한 변화가 발생했는지에 집중하는게 아니라, 객체에게 위임했는지 여부에 좀 더 초점을 맞춰야하는 걸까요? (verify를 통해서)

질문

1. 저는 사실 상태검증을 더 좋아합니다. 상태검증을 사용한다면 위와 같은 중복검증이 발생하게 되는걸까요? 객체지향의 관점에서 보면 사실 verify를 통해서 호출여부를 판단하는게 더 좋아보입니다. 다만, 상태검증을 위해서라면 위와 같이 중복적으로 검증하는 것을 피할 수 없는건가요?

2. 강사님의 경우, 상태검증을 할 때 위와같이 Community 로직을 다른 곳에서 테스트하였다고 해도, Service에서 다시 테스트하나요?