<aside> 💡 11장에서 다루는 내용

• 비공개 메서드 단위 테스트
• 단위 테스트를 하기 위한 비공개 메서드 노출
• 테스트로 유출된 도메인 지식
• 구체 클래스 목 처리

</aside>

• 11장에서는 책의 앞부분과 내용이 전혀 달라 따로 보는 편이 좋을 정도로 관련성이 없는 주제를 모았다.

11.1비공개 메서드 단위 테스트

• 비공개 메서드를 어떻게 테스트 하는가? → 전혀 하지 말아야 한다.

11.1.1 비공개 메서드와 테스트 취약성

• 단위 테스트를 하기위해 비공개 메서드를 노출하는 경우에는 5장에서 다룬 기본 원칙 중 하나인 ‘식별할 수 있는 동작만 테스트 하는 것’을 위반한다.
  • 비공개 메서드를 노출하면 테스트가 구현 세부 사항과 결합되어 리팩터링 내성이 떨어진다.
  • 비공개 메서드를 직접 테스트하는 대신 포괄적인 식별할 수 있는 동작으로서 간접적으로 테스트하는 것이 좋다.

11.1.2 비공개 메서드와 불필요한 커버리지

• 비공개 메서드가 너무 복잡해서 식별할 수 있는 동작으로 간접적으로 테스트하기에 충분히 커버리지를 얻을 수 없는 경우 → 식별할 수 있는 동작에 이미 합리적인 테스트 커버리지가 있다고 가정하면 두 가지 문제가 발생할 수 있다.

  • 테스트에서 벗어난 코드가 어디에도 사용되지 않는다면 리팩터링 후 남은 죽은 코드 → 삭제 필요

  • 추상화가 누락돼 있다. 비공개 메서드가 너무 복잡하면 별도의 클래스로 도출해야 하는 추상화가 누락됐다는 징후

    // 예제 11.1
    
    public class Order {
    		private Customer _customer;
        private List<Product> _products;
    
        public String generateDescription() {
            return "Customer name : " + _customer.name +
                "total number of products : " + _products.size() +
                "total price : " + getPrice();
        }
    
        private BigDecimal getPrice() {
            BigDecimal basePrice = /* _products 기반한 계산 */
            BigDecimal discounts = /* _customer 기반한 계산 */
            BigDecimal taxes = /* _products 기반한 계산 */
    
            return basePrice.subtract(discounts).add(taxes);
        }
    }
    

    • generateDescription() 메서드는 매우 간단하며, 주문에 대한 일반적인 설명을 반환
    • getPrice() 메서드는 훨씬 더 복잡한 비공개 메서드 → 중요한 비즈니스 로직이 있기 때문에 테스트를 철저히 해야한다.
    • 이 로직은 추상화가 누락됐다. getPrice() 메서드를 노출하기보단 다음과 같이 추상화를 통해 별도의 클래스로 도출해서 명시적으로 작성하는 것이 좋다.

    // 예제 11.2
    
    public class Order {
    		private Customer _customer;
        private List<Product> _products;
    
        public String generateDescription() {
    				PriceCalculator calc = new PriceCalculator();
    
            return "Customer name : " + _customer.name +
                "total number of products : " + _products.size() +
                "total price : " + calc.calculator(_customer, _products);
        }
    }
    
    public class PriceCalculator {
        public BigDecimal calculator(Customer customer, List<Product> products) {
            BigDecimal basePrice = /* _products 기반한 계산 */
            BigDecimal discounts = /* _customer 기반한 계산 */
            BigDecimal taxes = /* _products 기반한 계산 */
    
            return basePrice.subtract(discounts).add(taxes);
        }
    }
    

    • 이렇게 되면 Order 클래스와 별개로 PriceCalculator를 테스트할 수 있다.

11.1.3 비공개 메서드 테스트가 타당한 경우

• 비공개 메서드를 절대 테스트하지 말라는 규칙에도 예외가 있다. → 5장에서 살펴본 코드의 공개 여부와 목적 간의 관계를 다시 봐야한다.

표 11.1 코드의 공개 여부와 목적의 관계, 구현 세부 사항을 공개하지 말라.

• 5장에서 살펴봤듯이, 식별할 수 있는 동작을 공개로 하고 구현 세부 사항을 비공개로 하면 API가 잘 설계됐다고 할 수 있다.
  • 반면에 구현 세부 사항이 유출되면 코드 캡슐화를 해친다.