test: Network의 excute 메소드를 테스트하는 코드 추가 #5

- 의존성 주입을 위해서 Sessionable 프로토콜 추가
- Network의 session 타입을 Sessionable로 변경하고 이니셜라이저 추가
- URLSession 테스트를 위해 Sessionable 프로토콜 채택
- 테스트를 위한 MockSession, MockURLSessionDataTask 타입 추가
- Network의 excute 메소드 성공과 실패 케이스 작성

OpenMarket/OpenMarket.xcodeproj/project.pbxproj

@@ -17,6 +17,10 @@
  9A1CFDCC27842755003BEC69 /* ProductModification.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDCB27842755003BEC69 /* ProductModification.swift */; };
  9A1CFDCE27842E2E003BEC69 /* Currency.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDCD27842E2E003BEC69 /* Currency.swift */; };
  9A1CFDD1278469D5003BEC69 /* MultipartForm.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDD0278469D5003BEC69 /* MultipartForm.swift */; };
+ 9A1CFDD3278526B8003BEC69 /* NetworkTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDD2278526B8003BEC69 /* NetworkTests.swift */; };
+ 9A1CFDD527852721003BEC69 /* MockSession.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDD427852721003BEC69 /* MockSession.swift */; };
+ 9A1CFDD8278527CE003BEC69 /* Sessionable.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDD7278527CE003BEC69 /* Sessionable.swift */; };
+ 9A1CFDDA27852B2D003BEC69 /* MockURLSessionDataTask.swift in Sources */ = {isa = PBXBuildFile; fileRef = 9A1CFDD927852B2D003BEC69 /* MockURLSessionDataTask.swift */; };
  C70FB0FB25BEF61C00C9924E /* AppDelegate.swift in Sources */ = {isa = PBXBuildFile; fileRef = C70FB0FA25BEF61C00C9924E /* AppDelegate.swift */; };
  C70FB0FD25BEF61C00C9924E /* SceneDelegate.swift in Sources */ = {isa = PBXBuildFile; fileRef = C70FB0FC25BEF61C00C9924E /* SceneDelegate.swift */; };
  C70FB0FF25BEF61C00C9924E /* ViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = C70FB0FE25BEF61C00C9924E /* ViewController.swift */; };
@@ -54,6 +58,10 @@
  9A1CFDCB27842755003BEC69 /* ProductModification.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProductModification.swift; sourceTree = "<group>"; };
  9A1CFDCD27842E2E003BEC69 /* Currency.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Currency.swift; sourceTree = "<group>"; };
  9A1CFDD0278469D5003BEC69 /* MultipartForm.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = MultipartForm.swift; sourceTree = "<group>"; };
+ 9A1CFDD2278526B8003BEC69 /* NetworkTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = NetworkTests.swift; sourceTree = "<group>"; };
+ 9A1CFDD427852721003BEC69 /* MockSession.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = MockSession.swift; sourceTree = "<group>"; };
+ 9A1CFDD7278527CE003BEC69 /* Sessionable.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = Sessionable.swift; sourceTree = "<group>"; };
+ 9A1CFDD927852B2D003BEC69 /* MockURLSessionDataTask.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = MockURLSessionDataTask.swift; sourceTree = "<group>"; };
  C70FB0F725BEF61C00C9924E /* OpenMarket.app */ = {isa = PBXFileReference; explicitFileType = wrapper.application; includeInIndex = 0; path = OpenMarket.app; sourceTree = BUILT_PRODUCTS_DIR; };
  C70FB0FA25BEF61C00C9924E /* AppDelegate.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AppDelegate.swift; sourceTree = "<group>"; };
  C70FB0FC25BEF61C00C9924E /* SceneDelegate.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SceneDelegate.swift; sourceTree = "<group>"; };
@@ -95,10 +103,20 @@
  isa = PBXGroup;
  children = (
  9A1CFDD0278469D5003BEC69 /* MultipartForm.swift */,
+ 9A1CFDD7278527CE003BEC69 /* Sessionable.swift */,
  );
  path = Protocol;
  sourceTree = "<group>";
  };
+ 9A1CFDD627852793003BEC69 /* Test Doubles */ = {
+ isa = PBXGroup;
+ children = (
+ 9A1CFDD427852721003BEC69 /* MockSession.swift */,
+ 9A1CFDD927852B2D003BEC69 /* MockURLSessionDataTask.swift */,
+ );
+ name = "Test Doubles";
+ sourceTree = "<group>";
+ };
  C70FB0EE25BEF61C00C9924E = {
  isa = PBXGroup;
  children = (
@@ -138,7 +156,9 @@
  F35B1A372782EC300031A755 /* OpenMarketTests */ = {
  isa = PBXGroup;
  children = (
+ 9A1CFDD627852793003BEC69 /* Test Doubles */,
  F35B1A382782EC300031A755 /* ParserTests.swift */,
+ 9A1CFDD2278526B8003BEC69 /* NetworkTests.swift */,
  );
  path = OpenMarketTests;
  sourceTree = "<group>";
@@ -355,6 +375,7 @@
  F35B1A73278442F40031A755 /* ImageFile.swift in Sources */,
  9A1CFDC82783EE09003BEC69 /* NetworkConstant.swift in Sources */,
  9A1CFDD1278469D5003BEC69 /* MultipartForm.swift in Sources */,
+ 9A1CFDD8278527CE003BEC69 /* Sessionable.swift in Sources */,
  9A1CFD7A2782E683003BEC69 /* Parser.swift in Sources */,
  9A1CFDC62783DD63003BEC69 /* Vendors.swift in Sources */,
  9A1CFD7C2782EB88003BEC69 /* ParserError.swift in Sources */,
@@ -369,6 +390,9 @@
  isa = PBXSourcesBuildPhase;
  buildActionMask = 2147483647;
  files = (
+ 9A1CFDD527852721003BEC69 /* MockSession.swift in Sources */,
+ 9A1CFDDA27852B2D003BEC69 /* MockURLSessionDataTask.swift in Sources */,
+ 9A1CFDD3278526B8003BEC69 /* NetworkTests.swift in Sources */,
  F35B1A392782EC300031A755 /* ParserTests.swift in Sources */,
  );
  runOnlyForDeploymentPostprocessing = 0;

OpenMarket/OpenMarket/Model/Network/Network.swift

@@ -8,7 +8,11 @@
 import Foundation
 
 final class Network {
- let session = URLSession.shared
+ let session: Sessionable
+
+ init(session: Sessionable = URLSession.shared) {
+ self.session = session
+ }
 
  func execute(request: URLRequest, completion: @escaping (Result<Data?, Error>) -> Void) {
  session.dataTask(with: request) { data, response, error in

OpenMarket/OpenMarket/Protocol/Sessionable.swift

@@ -0,0 +1,14 @@
+//
+// Sessionable.swift
+// OpenMarket
+//
+// Created by Ari on 2022/01/05.
+//
+
+import Foundation
+
+protocol Sessionable {
+ func dataTask(with request: URLRequest, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void) -> URLSessionDataTask
+}
+
+extension URLSession: Sessionable {}

OpenMarket/OpenMarketTests/MockSession.swift

@@ -0,0 +1,32 @@
+//
+// MockNetwork.swift
+// OpenMarketTests
+//
+// Created by Ari on 2022/01/05.
+//
+
+import Foundation
+@testable import OpenMarket
+
+struct DummyData {
+ let data: Data?
+ let response: URLResponse?
+ let error: Error?
+ var completionHandler: ((Data?, URLResponse?, Error?) -> Void?)? = nil
+
+ func completion() {
+ completionHandler?(data, response, error)
+ }
+}
+
+class MockSession: Sessionable {
+ var dummyData: DummyData?
+
+ init(dummyData: DummyData) {
+ self.dummyData = dummyData
+ }
+
+ func dataTask(with request: URLRequest, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void) -> URLSessionDataTask {
+ return MockURLSessionDataTask(dummy: dummyData, completionHandler: completionHandler)
+ }
+}

OpenMarket/OpenMarketTests/MockURLSessionDataTask.swift

@@ -0,0 +1,21 @@
+//
+// MockURLSessionDataTask.swift
+// OpenMarketTests
+//
+// Created by Ari on 2022/01/05.
+//
+
+import Foundation
+
+class MockURLSessionDataTask: URLSessionDataTask {
+ var dummyData: DummyData?
+
+ init(dummy: DummyData?, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void?) {
+ self.dummyData = dummy
+ self.dummyData?.completionHandler = completionHandler
+ }
+
+ override func resume() {
+ dummyData?.completion()
+ }
+}

OpenMarket/OpenMarketTests/NetworkTests.swift

@@ -0,0 +1,68 @@
+//
+// NetworkTests.swift
+// OpenMarketTests
+//
+// Created by Ari on 2022/01/05.
+//
+
+import XCTest
+@testable import OpenMarket
+
+class NetworkTests: XCTestCase {
+
+ func test_Success() {
+ // given
+ let data = "test".data(using: .utf8)!
+ let url = URL(string: "testURL")!
+ let responce = HTTPURLResponse(url: url, statusCode: 200, httpVersion: nil, headerFields: nil)
+ let dummyData = DummyData(data: data, response: responce, error: nil)
+ let session = MockSession(dummyData: dummyData)
+ let network = Network(session: session)
+ let request = URLRequest(url: url)
+ let expectation = XCTestExpectation(description: "네트워크 실행")
+
+ // when
+ network.execute(request: request) { result in
+
+ // then
+ switch result {
+ case .success(let sampleData):
+ XCTAssertEqual(sampleData, data)
+ case .failure:
+ XCTFail()
+ }
+
+ expectation.fulfill()
+ }
+
+ wait(for: [expectation], timeout: 5)
+ }
+
+ func test_Failure() {
+ // given
+ let data = "test".data(using: .utf8)!
+ let url = URL(string: "testURL")!
+ let responce = HTTPURLResponse(url: url, statusCode: 404, httpVersion: nil, headerFields: nil)
+ let dummyData = DummyData(data: data, response: responce, error: nil)
+ let session = MockSession(dummyData: dummyData)
+ let network = Network(session: session)
+ let request = URLRequest(url: url)
+ let expectation = XCTestExpectation(description: "네트워크 실행")
+
+ // when
+ network.execute(request: request) { result in
+
+ // then
+ switch result {
+ case .success:
+ XCTFail()
+ case .failure:
+ XCTAssert(true)
+ }
+
+ expectation.fulfill()
+ }
+
+ wait(for: [expectation], timeout: 5)
+ }
+}