멘토링 데이 2주차 수요일 - donggeonoh/bithumb-techcamp-ios-1st GitHub Wiki

피드백

1. 코인 리스트의 API 호출을 2번 처리하는 코드에서 메서드의 depth가 너무 깊다. 이 부분 메서드를 나눠서 수정할 것. 나중에 이 코드를 수정하기가 까다로워지기 때문
2. 현재 잘하고 있지만 조금 더 노력했으면 좋겠다.

질문

Q. WebSocketNetworkService를 테스트 하려고 하였으나 URLSessionWebSocketTask를 상속한 Mock 객체를 만들 수 없어 (메서드가 public) 기존과 같은 동작을 하는 Mock WebSocketService를 만들어서 테스트를 구현하였는데, 기존 WebSocketService가 잘 동작하는지 보장해줄 수 있을지?

A. 일단 로직은 동일하게 구현을 했기 때문에 동작을 어느정도 보장할 수 있을 것이라고 생각한다. 하지만, 완벽하게 내부 로직을 테스트한건 아니기 때문에 신뢰성은 조금 떨어질 수 있다. 또, WebSocketService의 로직이 바뀌면 Mock도 같이 바꾸어야 한다는 문제는 고려해야 한다.