[일반] 테스트 코드 작성 질문

[익명(218.153)]

리팩터링 내성을 생각을 해서 테스트를 진행하려면
테스트 대상의 구현에 의존하지않고 결과를 검증해야한다는데

예를 들어 특정 서비스의 삭제 메서드가 있고
레파지토리의 delete 메서드를 호출한다고 했을때

단위 테스트로 결과를 검증하려면 레파지토리 목 객체의
delete 메서드 호출 여부를 검증해야하는거 같은데
이러면 구현에 의존하게 되는거 아닌가?

단순히 delete 메서드로 삭제를 수행하다가
deleteById로 바꾸기만 해도 테스트는 실패되는데

반환 값이 없는 삭제 메서드의 경우에는 어떤식으로 단위 테스트를 진행해야 함?

[백갤러 1(106.101)]

mock은 당연히 구현체의 영향을 받음 fake쓰면 됨

[김대기(waitkim)]

원칙적으로는 그런데 외부 인프라에서 불러오는 값 세팅은 해줘야지 대신 그런 테스트를 최소화 할 수는 있음 인프라 관련 코드는 최대한 단순하고 멍청하게 하란대로만 하게 짜서 테케도 단순하게 가져는게 베스트 다른 로직이랑 얽히고 섥히면 아주 골치아파

[김대기(waitkim)]

아니면 mock 테스트 대신 통합 테스트를 돌리는 방법도 있지 전체 로직을 돌리라는건 아니고 최대한 분리를 해서 일부만 통합테스트 대상이 되도록 해야함 그리고 어떤 클래스군이 통합테스트로 검증될지 명확한 기준이 있어야함

[김대기(waitkim)]

다만 이렇게 분리를 하려면 비슷한 케이스가 많아서 동일한 인프라 관련 코드가 공용으로 쓰일 수 있거나, 테스트 난이도가 너무 높아서 분리가 필요하거나 해야함

[김대기(waitkim)]

정 답이 안나오면 아싸리 통합테스트 위주로 작성하는 것도 방법임 메서드 하나마다 mock 테스트 만들면 관리가 안되거든 코드 개선할때마다 테스트 수정량 씨발소리 절로 나오니 사람들이 코드 개선할 생각을 안함 로직 수정했는데 일은 바쁘고 테스트도 고쳐야되면 보통 제대로 안고치고 대충 돌아만 가게 만듬 이딴 테스트가 의미 있을 리 없잖아?

[김대기(waitkim)]

중요한 것은 실현 가능한 수단이라는걸 잊지 마 테스트는 생산성을 올리기 위한거지 신이 내린 법칙이 아님 테스트를 짜서 생산성이 감소하면 주객이 전도된거

[김대기(waitkim)]

여기서 생산성은 당장 코딩하는데 걸리는 시간이라기보다는 프로덕트 전체적인 관점에서의 생산성임

[익명(218.153)]

감사합니다