Open kishe89 opened 1 year ago
https://github.com/Hyeeeein/TDD_Jest_TS/blob/9492ec0aedaf041d746fa56cf55e1bd1dc81f856/src/request/test/request.test.ts#L44-L47 위 와같은 코드도 함수 결과 데이터 mocking 한게 중복되므로 별도 변수로 빼서 테스트 변경시 변경점을 한곳으로 적용하는게 좋겠습니다.
결과 값을 mocking 할때는 결과 타입 정의하신것과 최대한 동일하게 타입까지 mocking 해주시면 좋습니다. 실제 함수 결과가 Promise 인경우 mocking data 는 Promise.resolve(T), Promise.reject(T) 등으로요.
Promise.resolve(T), Promise.reject(T)
coverage 같은 경우는 협업시 conflict 이 날 수 있기에 ignore에 추가하고 실제 테스트 coverage 데이터가 필요한 경우에 그때 그때 환경에서(주로 build 나 test 환경) 뽑아서 사용할 수 있도록 합니다.
전반적으로 요구한대로 다 잘 하셨어요. 혹시 진문 사항 있으시면 메일로 전달해주시면 시간 될 떄 보고 이야기 드리겠습니다.
추가적으로 저장소 관련해서도 mocking 연습 해보시면 좋을거같습니다.
넵 리뷰 감사드립니다 말씀해주신대로 기존 코드 수정하고 저장소 mocking 도 한번 시도해보도록 하겠습니다!
https://github.com/Hyeeeein/TDD_Jest_TS/blob/9492ec0aedaf041d746fa56cf55e1bd1dc81f856/src/request/test/request.test.ts#L44-L47 위 와같은 코드도 함수 결과 데이터 mocking 한게 중복되므로 별도 변수로 빼서 테스트 변경시 변경점을 한곳으로 적용하는게 좋겠습니다.
결과 값을 mocking 할때는 결과 타입 정의하신것과 최대한 동일하게 타입까지 mocking 해주시면 좋습니다. 실제 함수 결과가 Promise 인경우 mocking data 는
Promise.resolve(T), Promise.reject(T)
등으로요.coverage 같은 경우는 협업시 conflict 이 날 수 있기에 ignore에 추가하고 실제 테스트 coverage 데이터가 필요한 경우에 그때 그때 환경에서(주로 build 나 test 환경) 뽑아서 사용할 수 있도록 합니다.
전반적으로 요구한대로 다 잘 하셨어요. 혹시 진문 사항 있으시면 메일로 전달해주시면 시간 될 떄 보고 이야기 드리겠습니다.
추가적으로 저장소 관련해서도 mocking 연습 해보시면 좋을거같습니다.