단위 테스트 계획(코드) 및 테스트 작업에 대한 리뷰 부탁드립니다.

Question

제가 단위테스트를 작성하고 있습니다. 1. add()(21줄)가 제대로 동작하고 있는지 테스트는 것이 목적입니다. 우선 기술적으로 다음과 같은 3개의 메소드를 목업할 예정인데요, 25번줄: groupRepository.nextId()32번줄: GroupFactory.create - *파워뫀키토 이용33번줄: groupoRepository.save() << 우선 목업 대상이 맞는지 좀 조언 부탁드립니다. 혹시 목업이 빠졌을까요? 2. 그리고 지금 서비스는 group을 반환하기로 되어 있거든요? 이것이 테스트 대상인지 아닌지 좀 기준이 잘 서지 않습니다. 만약 테스트 대상이라면 32번줄의 GroupFactory.create()를 목업하면 안 되거든요. 그런데 목업을 안 하면 만들어지는 규칙이 좀 까다로워서 테스트하기가 어려운 상황입니다. 그러면 34번줄의 최종적으로 반환되는 group이 제대로 된 것이라고 확인하는 작업은 어디서 해야 하나요...?? 3. 마지막으로 테스트에서 중복코드 다루는 것에 대해 의견을 여쭤보겠습니다. 보통 저는 DRY 원칙을 지키지만, 구글 검색해본 결과 테스트코드에는 코드 중복이 무방하다는 의견도 만만치 않더라구요. 사실 제가 어제 오늘 중복코드 리팩토링하다가 지쳐서 찾아보니까, 모든 테스트케이스를 커버할 수 있는 리팩토링은 할 수 없다는 주장도 있고, 가독성이 더 중요하다고 하는 사람도 있었습니다. 개인적으로 쭉 읽어보니까 공감이 가는 부분이 많았습니다. 그래서 일단 아래처럼 막 중복시켜놨는데요, 이 부분에 대해서 의견 부탁드립니다. 아래는 제가 읽은 글 중 가장 논리적인 주장 출처입니다.http://blog.jayfields.com/2008/05/testing-duplicate-code-in-your-tests.html @Test public void should_success_when_addUserGroup_by_commonUserInAdminGroup(){ when(mockUser.superUser()).thenReturn(false); when(mockUser.admin()).thenReturn(true); when(mockGroupRepo.nextId()).thenReturn(mockGroupId); when(mockUserRepo.find(isA(UserId.class))).thenReturn(mockUser); AddGroupRequest request = AddGroupRequest .builder("requester01") .name("테스트그룹") .description("테스트그룹 설명입니다.") .groupType(GroupType.USER) .build(); facade.add(request); ArgumentCaptor groupCaptor = ArgumentCaptor.forClass(Group.class); InOrder mockUserOrder = inOrder(mockUser); InOrder mockGroupRepoOrder = inOrder(mockGroupRepo); mockUserOrder.verify(mockUser, times(1)).admin(); mockUserOrder.verify(mockUser, never()).superUser(); mockGroupRepoOrder.verify(mockGroupRepo).nextId(); mockGroupRepoOrder.verify(mockGroupRepo).save(groupCaptor.capture()); } @Test public void should_success_when_addUserGroup_by_superUser(){ when(mockUser.superUser()).thenReturn(true); when(mockUser.admin()).thenReturn(true); when(mockGroupRepo.nextId()).thenReturn(mockGroupId); when(mockUserRepo.find(isA(UserId.class))).thenReturn(mockUser); AddGroupRequest request = AddGroupRequest .builder("requester01") .name("테스트그룹") .description("테스트그룹 설명입니다.") .groupType(GroupType.USER) .build(); facade.add(request); ArgumentCaptor groupCaptor = ArgumentCaptor.forClass(Group.class); InOrder mockUserOrder = inOrder(mockUser); InOrder mockGroupRepoOrder = inOrder(mockGroupRepo); mockUserOrder.verify(mockUser).admin(); mockUserOrder.verify(mockUser, never()).superUser(); mockGroupRepoOrder.verify(mockGroupRepo).nextId(); mockGroupRepoOrder.verify(mockGroupRepo).save(groupCaptor.capture()); } @Test public void should_success_when_addDefaultGroup_by_superUser(){ when(mockUser.superUser()).thenReturn(true); when(mockUser.admin()).thenReturn(true); when(mockGroupRepo.nextId()).thenReturn(mockGroupId); when(mockUserRepo.find(isA(UserId.class))).thenReturn(mockUser); AddGroupRequest request = AddGroupRequest .builder("requester01") .name("테스트그룹") .description("테스트그룹 설명입니다.") .groupType(GroupType.DEFAULT) .build(); facade.add(request); ArgumentCaptor groupCaptor = ArgumentCaptor.forClass(Group.class); InOrder mockUserOrder = inOrder(mockUser); InOrder mockGroupRepoOrder = inOrder(mockGroupRepo); mockUserOrder.verify(mockUser, never()).admin(); mockUserOrder.verify(mockUser).superUser(); mockGroupRepoOrder.verify(mockGroupRepo).nextId(); mockGroupRepoOrder.verify(mockGroupRepo).save(groupCaptor.capture()); }

unthinkall · Answer

GroupFactory는 따로 유닛테스트 적용시켜야 될거 같은데요?

단위 테스트 계획(코드) 및 테스트 작업에 대한 리뷰 부탁드립니다.

1개의 답변

#mockito 목록