리팩터링을 제대로 하려면 불가피하게 저지르는 실수를 잡아주는 견고한 테스트 스위트(test suite)가 뒷받침돼야 한다.

자가 테스트 코드의 가치

모든 테스트를 완전히 자동화하고 그 결과까지 스스로 검사하게 만들자

• 자가 테스트 코드 자체뿐 아니라 테스트를 자주 수행하는 습관도 버그를 찾는 강력한 도구가 된다.
• 테스트 스위트는 강력한 버그 검출 도구로, 버그를 찾는 데 걸리는 시간을 대폭 줄여준다.
• 테스트를 작성하기 가장 좋은 시점은 프로그래밍을 시작하기 전이다.
  • 테스트를 작성하다 보면 원하는 기능을 추가하기 위해 무엇이 필요한지 고민하게 됨
  • 구현보다 인터페이스에 집중하게 됨
  • 코딩이 완료되는 시점을 정확하게 판단할 수 있음(테스트를 모두 통과한 시점이 코드를 완성한 시점)
• 켄트 벡은 테스트부터 작성하는 습관을 바탕으로 테스트 주도 개발(Test-Driven Development, **TDD**) 기법을 창시
  • 처음에는 통과하지 못할 테스트를 작성
  • 이 테스트를 통과하게 코드를 작성
  • 결과 코드를 최대한 깔끔하게 리팩터링

테스트 추가하기

자주 테스트하라. 작성 중인 코드는 최소한 몇 분 간격으로 테스트하고, 적어도 하루에 한 번은 전체 테스트를 돌려보자.

• describe와 it 블록에 부연 설명용 문자열을 써넣는 방식은 개발자마다 다름. 실패한 테스트가 무엇인지 식별할 수 있을 정도로만 작성해도 괜찮음
• 실패해야 할 상황에서는 반드시 실패하게 만들자
  • 테스트를 모두 통과하는 것은 좋은일이지만, 수많은 테스트를 실행했음에도 실패하는게 없다면 테스트가 내 의도와는 다른 방식으로 다뤄지는 건 아닌지 의심해보아야 함
  • 이는 코드에 일시적 오류를 주입해 실패하는지 확인해보면 된다.
• 테스트는 위험 요인을 중심으로 작성해야 한다! 테스트의 목적은 어디까지나 현재 혹은 향후에 발생하는 버그를 찾는데 있다.
• 완벽하게 만드느라 테스트를 수행하지 못하느니, 불완전한 테스트라도 작성해 실행하는 게 낫다.
  • 처음에는 임시값을 넣고 테스트 수행(임시 값 설정)
  • 프로그램이 내놓는 실제 값을 결과값에 대체하여 성공하는지 실행(실제 값으로 대체)
  • 코드를 수정하여 실패하는지 검토해본 후 다시 이전 값으로 대체(오류를 심었다가 되돌리기)
• 테스트 코드에서도 중복은 의심해봐야 한다.

  • 아래 코드에서 픽스처에 대한 코드가 중복 사용되고 있음

    describe("province", function() {
    	it("shortfall", function() {
    		const asia = new Province(sampleProvinceData()); 
    		expect(asia.shortfall).equal(5);
    	});
    	it("profit", function() {
    		const asia = new Province(sampleProvinceData()); 
    		expect(asia.profit).equal(230);
    	});
    });
    

  • 일반적으로 중복된 픽스처가 있다면 아래와 같이 바깥 범위로 끌어내 공통으로 사용할 수 있도록 수정하는 경우가 많다.(나도 이 방법을 주로 사용했었음..😅)

    describe("province", function() {
    	const asia = new Province(sampleProvinceData());
    
    	it("shortfall", function() {
    		expect(asia.shortfall).equal(5);
    	});
    	it("profit", function() {
    		expect(asia.profit).equal(230);
    	});
    });
    

    • 하지만, 이렇게 사용할 경우 테스트 관련 버그 중 가장 지저분한 유형인 테스트끼리 상호작용하게 하는 공유 픽스처를 생성하게 된다.
    • const는 asia 객체의 내용이 아니라 가리키는 참조가 상수임을 뜻하므로, 나중에 다른 테스트에서 공유 객체의 값을 수정하면 이 픽스처를 사용하는 또 다른 테스트가 실패할 수 있다.
    • 이렇게 되면 버그를 잡기가 어렵고 오래걸리게 된다.

  • 올바른 중복 픽스처 사용 코드

    describe("province", function() {
    	let asia;
    
    	beforeEach(function() {
    		asia = new Province(sampleProvinceData());
    	});
    
    	it("shortfall", function() {
    		expect(asia.shortfall).equal(5);
    	});
    	it("profit", function() {
    		expect(asia.profit).equal(230);
    	});
    });
    

    • beforeEach 구문은 각각의 테스트 바로 전에 실행되어 asia를 초기화하기 때문에 모든 테스트가 자신만의 새로운 asia를 사용하게 됨
    • 개별 테스트를 실행할 때마다 픽스처를 새로 만들면 모든 테스트를 독립적으로 구성할 수 있어서 결과를 예측할 수 없는 사태를 예방할 수 있다.

픽스처 수정하기

• 일반적으로 it 구문 하나당 검증도 하나씩만 하는게 좋다.