[클린코드 7기 원건우] 자동차 경주 미션 STEP 5

[rjsdnql123]

마지막 날이라 STEP5 리뷰 전에 제출합니다!

기능 요구사항

• 도메인 로직은 domain/ 하위로, UI 관련 로직은 view/ 하위에서 관리한다.
• domain/ 하위의 모듈은 view/ 하위의 모듈을 의존하지 않아야 한다.
• 도메인 로직 내에서 테스트하기 어려운 부분을 분리하고, 테스트 가능한 부분에 대해서만 단위 테스트를 작성한다.
• 테스트 코드에서 jest.fn()을 사용하지 않는다.

신경쓴점

Domain 로직이 view 로직에 의존 하지 않도록 작성 하였습니다.

궁금한점

과제 요구 사항에 jest.fn을 사용하지 않는 이유가 있을까요??

[igy95]

안녕하세요 건우님! 마지막 미션까지 고생 많으셨습니다 ㅎㅎ

최대한 view, domain의 목적을 분리하고 참조 방향을 잘 맞추려고 해주신 듯 하네요.

코멘트 남겨두었으니 확인 부탁드리고 재요청 주세요~~!

과제 요구 사항에 jest.fn을 사용하지 않는 이유가 있을까요??

이 부분은 확인 후 답변 드릴게요 !

[igy95]

과제 요구 사항에 jest.fn을 사용하지 않는 이유가 있을까요??

자동차 경주 피드백(2)에서 “가장 작은 단위부터 테스트하기“와 “단위 테스트하기 어려운 코드를 단위 테스트하기“라는 부분이 있습니다:blush:
모킹을 잘하기 위함이 아니라, 기초적인 단위 테스트 그 자체를 훈련하기 위함인데 본질에 집중하지 못할 것을 우려해서 미리 설정해 둔 제약사항이었습니다! 교안 링크를 보시면 좀 더 의도가 선명하게 이해되실 거예요 ㅎㅎ
이외에도 의문이 있으시면 얼마든지 문의주세요! 감사합니다.

테스트 하기 어려운 부분을 모킹으로 해결해버리면, 어떤 것이 단위 테스트로 해결 가능한 부분인지 판단이 흐려질 것을 우려하여 설정해둔 제약사항으로 보이네요!

[rjsdnql123]

늦었네요 ㅠㅠ 수정 모두 되었습니다!

[rjsdnql123]

@igy95

테스트 하기 어려운 부분을 모킹으로 해결해버리면, 어떤 것이 단위 테스트로 해결 가능한 부분인지 판단이 흐려질 것을 우려하여 설정해둔 제약사항으로 보이네요!

그렇군요.. 잘 사용할 수 있을 때 사용하는것이 좋겟네요!

[igy95]

안녕하세요 건우님~! 수정 후 요청 주신 걸 제가 놓쳐 approve 시점이 늦어졌네요 🥲 리뷰 반영된 부분 모두 확인하였습니다! 5단계까지 진행하시느라 고생많으셨어요 ~~