테스트만을 위해서 운영 코드에 수정이 일어나도 괜찮은가요??

Question

DDD를 바탕으로 개발중입니다.User 엔티티에 비즈니스 로직을 구현하고, 이를 테스트 하고 있습니다.public void updateUpdatedAt() {this.updatedAt = LocalDateTime.now();}위 메서드에 대한 테스트를 작성중인데, then 부분에서 해당 메서드 호출 전의 updatedAt 값과 호출 후의 updatedAt 값이 같은 지를 비교하려고 합니다. 이를 위해서, 해당 메서드 호출 전 후 모두 updatedAt을 get해와야 하는데, 해당 컬럼에 대한 get 메서드가 없습니다. 즉, 테스트만을 위해 User 클래스에 getUpdatedAt() 메서드를 추가해야 하는 것이 맞는지에 대한 고민이 있습니다. 해당 메서드를 추가하지 않으면 테스트를 작성할 수 없고, 그렇다고 해당 테스트만을 위해서 운영 코드에 수정이 일어나도 괜찮은 지 고민이 됩니다..

dujulgi · Answer

몇몇 테스트 관련 책을 읽어보면 테스트 코드도 동등한 지위로 봐야한다고 써져있다보니 테스트를 위해 코드를 수정하는것도 얼마든지 가능한 일이라고 생각합니다.일반적으론 API를 잘 설계하면 테스트하기도 수월해요위 코드는public void setUpdatedAt(Clock clock) { this.updatedAt = LocalDateTime.now(clock); }로 바꾸면 테스트하기 더 수월할거 같네요그런데 이런 코드를 단위테스팅 하는건 고전파에선 별로 추천하지 않습니다.

또닉 · Answer

테스트대상이나 본문코드나 질문내용상 유추되는건 뭔가 설계가 좋지 않은 거 같습니다. 테스트코드는 테스트할 가치가 있는 것만 작성하세요.테스트만을 위한코드도 있기 마련이지만 ( 테스트코드에서만 접근 가능한 접근자를 쓰기도 합니다 )다만 테스트가 아니면 인스턴스가 만들어지지 않아야 할것입니다. 호출되지 않는 메서드는 메모리 차지를 거의 안할겁니다(메타정보 외에는 메모리에 로드 되지 않음)그리고, 유닛테스트가 가능한 코드를 작성하세요. 저런 시간을 다루는 코드는 시간 의존성을 줄인 코드를 작성하는 것이 좋습니다. .Now 를 저렇게 하드코딩하면 시간테스트가 안됩니다. 시간제공을 인터페이스로 추상화하고 테스트코드 작성시 시간을 마음대로 조작가능하게 만들어야합니다.

테스트만을 위해서 운영 코드에 수정이 일어나도 괜찮은가요??

2개의 답변

Q&A 목록