우아한테크코스 BE 5기 3주차 회고

POII·2023년 2월 26일

우아한테크코스 우테코

우아한테크코스 BE 5기

목록 보기

11/15

이번 주 미션은 2주차때 페어와 함께 만들었던 모양만 있는 사다리를 진짜 사다리 게임으로 바꾸는 것!

레이싱 미션과 비교했을 때 생각해야 될 게 많아서 꽤 시간이 오래 걸렸다.

겨우겨우 금요일 아침에 PR을 날리고 리뷰가 올 때 까지 책을 읽어야지 하는 결심을 세웠었다.

그런데 사다리게임 리뷰어인 찰리가 당일에 캠퍼스에 직접 와서 오프라인 피드백을 해주었다.. ㄷㄷ

피드백 하러 오신 건 아니고 브라운 코치를 만나러 왔다가 겸사겸사?

덕분에 글로 정리하기 힘들어서 풀리퀘스트에 보내지 못했던 질문들을 옆에서 직접 할 수 있었다.

추상적으로 이건 어떨까?하고 생각만하던 것들을 꽤 구체적으로 정리할 수 있었던 시간이었다.

배운 것

테스트의 검증부는 하드코딩으로 하는 편이 좋을 수 있다.

만약 프로덕션 코드의 어떤 로직을 통해 값을 가져와 체크한다면 무의미한 검증이 될 수 있다.

void sum_test() {
        int a = 1;
        int b = 2;
        assertThat(sum(a, b)).isEqualsTo(a+b);
    }

위는 좀 극단적인 예시지만 isEqualsTo의 인자를 3으로 직접 코딩하는 편이 누가봐도 a+b 를 실행하는 sum 메소드의 결과를 의미있게 테스트하는 길일 것이다.

무의미함을 넘어 더 치명적인 상황이 발생할 수도 있다. 만약 a+b 라는 기능 자체에 문제가 있고, sum 이 그 로직을 사용하고 있을 경우 테스트는 언제나 거짓된 성공을 반환한다.

이는 프로그램 자체가 오작동 했을 때, 개발자가 이 기능은 테스트 성공했으니 다른 쪽에 문제가 있을거야라고 생각하게 만들 수 있다.

다음으로 생길 수 있는 문제점은 테스트를 먼저 작성하기 어렵게 한다는 점이다.

void get_name_and_level() {
	var name = new Name("포이");
	var level = new Level("레벨 1");
	var player = new Player(name, position);
	assertThat(player.getNameAndLevelString())
		.isEqualsTo(player.getName().value() + player.getLevel().value());
}

예시를 들기 위해 게터를 테스트하는 이상한 코드를 작성해보았다.

위 처럼 검증부에 로직이 들어가게 되면 결과에 따라 로직을 작성하고 리팩터링하는게 아닌 프로덕션 코드를 먼저 구상한 후 테스트를 작성하는 것이 된다. 이는 getName과 getLevel이라는 메서드의 리팩터링을 제한적으로 만들 수 있다.

도메인과도 강결합 되어있기 때문에 도메인의 로직이 리팩토링으로인해 변경되면 테스트는 그대로인데 갑자기 실패하는 테스트가 되는 결과를 낳을 수도 있다.

ParameterizedTest에 이름을 표시하는 방법

별 거 아닐 수 있지만, DIsplayName 어노테이션을 이용해도 ParameterlizedTest를 이용하면 각 인자들엔 이름이 표시되지 않는다.

		@DisplayName("알맞은 사다리 길이를 입력하면 정상적으로 사다리가 생성된다.")
    @ParameterizedTest()
    @ValueSource(ints = {5, 100})
    void create_success(int rightLadderHeight) {
        assertThatNoException().isThrownBy(() -> new LadderHeight(rightLadderHeight, 3));
    }

이는 ParameterizedTest 안에 name 속성으로 해결할 수 있다. 인자가 여러개라라면 name안에 {0}, {1} 처럼 이의 순서를 활용해 표시하고 싶은 인자를 선택할 수 있다.

@DisplayName("알맞은 사다리 길이를 입력하면 정상적으로 사다리가 생성된다.")
@ParameterizedTest(name = "알맞은 사다리 길이를 입력하면 정상적으로 사다리가 생성된다. 입력 : {0}")
@ValueSource(ints = {5, 100})
void create_success(intrightLadderHeight) {
	assertThatNoException().isThrownBy(() -> new LadderHeight(rightLadderHeight, 3));
}