[개발자 책읽기] 테스트 주도 개발 - 켄트 벡 (사과와 오렌지)

해당 글은 Kent-Beck의 테스트 주도 개발 By Example 이라는 책을 읽고 학습한 내용을 정리 및 회고하는 글 입니다. 자세한 사항은 YES 24 테스트 주도 개발 에서 확인해주세요.

테스트 주도 개발 - Test Driven Development By Example (Kent-Beck)

• 인사이트(insight)
• 지은이: Kent-Beck
• 엮은이: 김창준, 강규역
• 출간일: 2014.02.15

TDD 최종 목표

TDD 의 최종 목표는 Clean Code That Works 이다.
일반적인 프로그래머들은 Clean Code That Works를 한 번에 수행하는 것은 사실상 불가능하므로 Clean Code와 Code That Works를 서로 나눠서 생각하자는 것이다.
이는 우리가 흔히 아는 분할 정복의 기법이다.

사과와 오렌지

지난 시간에 우리는 상속을 이용해서 Money를 상속받는 Dollar 클래스와 Franc 클래스를 만들었다.

그리고 Dollar와 Franc에 중복되는 코드 equals() 를 Money로 옮겼다.

하지만 여기서 문제가 발생한다.

본질적으로 Dollar나 Franc나 같은 클래스를 상속받았지만 엄연히 서로 다른 클래스이다.

결국 Money에서 상속받는 equals()에는 오류가 있다.

왜? equals()에서는 단지 amount의 필드 값만 보고 비교를 하기 때문에.

그래서 이번에는 equals()를 바꿔보도록하자.

TDD 시작

실패하는 코드 작성

@Test
void testEquality() {
  assertEquals(new Dollar(5), new Dollar(5));
  assertNotEquals(new Franc(5), new Franc(1));
  assertNotEquals(new Dollar(5), new Franc(5));
}

class Money {
  protected int amount;

  @Override
  public boolean equals(Object object) {
    Money money = (Money) object;
    return amount == money.amount;
  }
}

당연히 실패한다.

우리의 목적은 금액도 같고, 클래스도 같을 때 true 반환하는 것이기 때문에 리팩토링하자.

@Override
public boolean equals(Object object) {
  Money money = (Money) object;
  return amount == money.amount // 금액이 같고
          && getClass.equals(object.getClass()); // 클래스도 같을 때
}

원래라면 이런식으 equals 는 적절하지 않다.
하지만 아직까지는 확실히 다른 디자인을 적용해야한다는 동기가 없으니 이대로 넘어가도 충분하다.