@leejun6694
# Step1-1 PR
쿠마 안녕하세요!
Step1이 아닌 Step1-1 PR을 보냅니다! 😅

## 궁금한 점

### 0. Mock 객체 구현을 맞게 했는지 궁금합니다.
네트워크와 통신하는 타입 NetworkManager를 구현했습니다.
- 그리고 URLSession의 Mock 객체를 만들어 NetworkManager에서 데이터를 받아오도록 했습니다.
- Mock 객체는 items url인지 item url인지에 따라 해당하는 JSON파일을 넘겨주도록 했습니다.
- 그리고 여기서 url과 JSON파일은 저희가 임의로 매칭했습니다. (items/1 로 끝나는 url이 들어오면 Items.json을, item/1로 끝나는 url이 들어오면 Item.json을 전달)

### 1. Model
NetworkManager와 JSOnParser 타입은 Model일까요, Controller일까요?

Soll은 Model 이라고 생각하고(데이터를 가공해서 Controller에서 호출되는 것이니까), Joey는 둘 다 아니라고 생각했어요. 쿠마는 어떻게 생각하시나요?

### 2. Test 대상을 다른 대상의 Test에 사용해야하는 경우
#### 예전 PR 피드백 질의응답
> `dequeue()`나 `clear()`를 테스트하기 위해서 `enqueue(_:)`를 통해 값을 세팅해주는데요,이럴 때 `enqueue(_:)`가 항상 성공할거라는 보장을 얻을 수 있나요? 이 메서드가 실패하면연쇄적으로 테스트가 실패할텐데, 이를 방지하고자 하는 방법이 있을까요? [깃헙](https://github.com/yagom-academy/ios-bank-manager/pull/70)

쿠마: 반대로 말하면 이런 메서드가 실패할 수 있을지 확인하기 위해, 우리가 테스트 코드를 짜고 있을 수도 있겠네요ㅎㅎ 이를 방지하기 위해선 enqueue method 에 에러 처리가 필요하겠네요...!? 저도 좀 더 생각해보겠습니다!

#### 이번 고민 내용
`fetchData()` 에서 `parse()` 를 사용하고 있는데 저번처럼 하나의 테스트가 다른 테스트를 품고있다고 생각이 들어서요. 혹시 더 좋은 방법이 있을까요?

### 3. 하드코딩 관련
테스트 코드에서 String 타입도 상수로 저장해서 사용하는 것이 좋을까요?

    let testData = try XCTUnwrap(NSDataAsset(name: "Items")?.data)

### 4. 접근 제어자
리팩토링을 하는 과정에서 은닉화를 위해 프로퍼티와 메서드들에 private을 붙여주려고 했는데요, NetworkManager의 프로퍼티와 메서드를 테스트 코드에서 접근해서 사용을 하고 있더라구요.

### 5. deprecated init()을 사용해도 되나요?
- 아래처럼 MockURLSessionDataTask가 URLSessionDataTask를 상속받는 경우 `override init()` 이 deprecated init이라는 경고가 떴습니다.

class MockURLSessionDataTask: URLSessionDataTask {
    override init() {}

    var resumeDidCall: () -> Void = {}

    override func resume() {
        resumeDidCall()
    }
}

- 작동이 되긴하지만 조금 찝찝해서 아래와 같이 구현했습니다. 다만 이 경우 URLSessionProtocol과 URLSession의 dataTask() 메서드까지 수정해줘야하는 번거로움이 있었습니다.
- Q. 위 방식과 다음 방식 중 어느 것이 나은지, deprecated init()은 사용해도 되는 것인지 쿠마의 조언을 구하고 싶습니다 🤓

class MockURLSessionDataTask: URLSessionDataTaskProtocol {
    var resumeDidCall: () -> Void = {}
    
    func resume() {
        resumeDidCall()
    }
}

protocol URLSessionProtocol {
    func dataTaskWithRequest(with request: URLRequest, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void)
    -> URLSessionDataTaskProtocol
}

extension URLSession: URLSessionProtocol {
    func dataTaskWithRequest(with request: URLRequest, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void)
    -> URLSessionDataTaskProtocol {
        dataTask(with: request, completionHandler: completionHandler) as URLSessionDataTaskProtocol
    }
}

# 잘 부탁드립니다 🤩 🙏 🥳