private 메서드는 어떻게 테스트 할까?

Service에 있는 메서드를 테스트하고 싶은데 public으로 되어있는 전체 로직이 아닌

private으로 된 기능이 나누어진 메서드의 기능검증을 하고 싶다.

 

그래서 public으로 변경해서 테스트코드를 완성했는데

찾아보니

테스트 코드를 작성하기 위해 ServiceImpl 클래스 내에 인터페이스에 정의되지 않은 public 메서드를 추가하는 것은 일반적으로 피해야 할 사항입니다. 인터페이스는 특정 계약(Contract)을 정의하고, 이를 구현하는 클래스들은 해당 계약을 지켜야 합니다. 만약 인터페이스에 정의되지 않은 메서드를 클래스 내에서 추가한다면, 코드의 일관성과 유지보수성이 저하될 수 있습니다.

 

라고 한다. 맞는 말인데 그렇다면 어떻게 테스트하는 게 좋을까?

 

Reflection

리플렉션 추천하는 글이 많이 보여서 사용했다. 문제는

private 메서드는 내부를 감추어 클라이언트와의 결합도를 낮춰주는데, 클라이언트인 테스트 클래스가 내부 메서드를 알고 있으니 결합도가 높아진다. 그리고 이는 유지보수할 때 테스트에 대한 비용을 증가시키는 요인이 될 수 있는데, 메서드 이름이나 파라미터 등을 변경할 때 실패하게 된다. 

 

인데, 내가 원하는 테스트는 public 메서드에서 사용되고 있는 private 메서드의 기능검증 하는 거였기 때문에

결합도와 무관하다고 생각하고 해당 클래스가 변경되었을 때 테스트가 깨지는 건 당연하다고(믿음직한 테스트 코드) 생각했기 때문에 사용했다.

 

스프링은 내부적으로 리플렉션을 사용하고 있기 때문에 고맙게도 ReflectionTestUtils를 제공해 준다.

@ExtendWith(MockitoExtension.class)
class UserSErviceImplTest {
	
    @InjectMocks
    UserServiceImpl userService;
    
    @Test
    void filterSignUpOverMonths() {
  	
    // testDataList 생성
    
    List<User> result = ReflectionTestUtils.invokeMethod(
    userService, "filterSignUpOverMonths", testDataList, 6);
    
    // 테스트 결과 검증
    
    }
}

 

실제 메서드 로직 테스트를 목적으로 Mockito를 사용하기 위해 Extend 해서 Service를 주입했다

리플렉션을 사용하면. invokeMethod 처음 인자값(target)에 UserServiceImpl.class 하면 되는데 왜 굳이 Mockito까지 사용했냐면UserServiceImpl.class 에 있는 filterSignUpOverMonths 메서드가 정적메서드가 아니기 때문에 인스턴스화가 필요했다.

(static 메서드 라면 안해도 된다)

 

실제 비즈니스 로직을 변경하지 않고 테스트해서 뿌듯

 

도움 받은 곳

https://mangkyu.tistory.com/235