Merge pull request #7 from crelies/dev

more unit tests, fixed xc test manifests
crelies · Feb 23, 2020 · 51ed9bf · 51ed9bf
2 parents 6fa244d + f0f6c4a
commit 51ed9bf
Show file tree

Hide file tree

Showing 10 changed files with 102 additions and 1 deletion.
diff --git a/Tests/RemoteImageTests/Extensions/URLSession+RemoteImageURLDataPublisherTests.swift b/Tests/RemoteImageTests/Extensions/URLSession+RemoteImageURLDataPublisherTests.swift
@@ -21,4 +21,8 @@ final class URLSession_RemoteImageURLDataPublisherTests: XCTestCase {
         let dataPublisher = urlSession.dataPublisher(for: urlRequest)
         XCTAssertEqual(dataPublisher.description, dataTaskPublisher.description)
     }
+
+    static var allTests = [
+        ("testDataPublisher", testDataPublisher)
+    ]
 }
diff --git a/Tests/RemoteImageTests/Models/PhotoKitServiceErrorTests.swift b/Tests/RemoteImageTests/Models/PhotoKitServiceErrorTests.swift
@@ -61,4 +61,16 @@ final class PhotoKitServiceErrorTests: XCTestCase {
     func testErrorDomain() {
         XCTAssertEqual(PhotoKitServiceError.errorDomain, String(describing: PhotoKitService.self))
     }
+
+    static var allTests = [
+        ("testMissingDataErrorDescription", testMissingDataErrorDescription),
+        ("testMissingDataFailureReason", testMissingDataFailureReason),
+        ("testMissingDataRecoverySuggestion", testMissingDataRecoverySuggestion),
+        ("testMissingDataErrorCode", testMissingDataErrorCode),
+        ("testPhAssetNotFoundErrorDescription", testPhAssetNotFoundErrorDescription),
+        ("testPhAssetNotFoundFailureReason", testPhAssetNotFoundFailureReason),
+        ("testPhAssetNotFoundRecoverySuggestion", testPhAssetNotFoundRecoverySuggestion),
+        ("testPhAssetNotFoundErrorCode", testPhAssetNotFoundErrorCode),
+        ("testErrorDomain", testErrorDomain)
+    ]
 }
diff --git a/Tests/RemoteImageTests/Models/RemoteImageServiceErrorTests.swift b/Tests/RemoteImageTests/Models/RemoteImageServiceErrorTests.swift
@@ -14,4 +14,8 @@ final class RemoteImageServiceErrorTests: XCTestCase {
         let expectedDescription = "Could not create image from received data"
         XCTAssertEqual(description, expectedDescription)
     }
+
+    static var allTests = [
+        ("testCouldNotCreateImageDescription", testCouldNotCreateImageDescription)
+    ]
 }
diff --git a/Tests/RemoteImageTests/Models/RemoteImageStateTests.swift b/Tests/RemoteImageTests/Models/RemoteImageStateTests.swift
@@ -0,0 +1,43 @@
+//
+//  RemoteImageStateTests.swift
+//  RemoteImageTests
+//
+//  Created by Christian Elies on 23.02.20.
+//
+
+#if canImport(UIKit)
+@testable import RemoteImage
+import UIKit
+import XCTest
+
+final class RemoteImageStateTests: XCTestCase {
+    func testErrorValue() {
+        let error = NSError(domain: "MockDomain", code: 1, userInfo: nil)
+        let state: RemoteImageState = .error(error)
+        XCTAssertEqual(state.error, error)
+    }
+
+    func testNoErrorValue() {
+        let state: RemoteImageState = .loading
+        XCTAssertNil(state.error)
+    }
+
+    func testImageValue() {
+        let image = UIImage()
+        let state: RemoteImageState = .image(image)
+        XCTAssertEqual(state.image, image)
+    }
+
+    func testNoImageValue() {
+        let state: RemoteImageState = .loading
+        XCTAssertNil(state.image)
+    }
+
+    static var allTests = [
+        ("testErrorValue", testErrorValue),
+        ("testNoErrorValue", testNoErrorValue),
+        ("testImageValue", testImageValue),
+        ("testNoImageValue", testNoImageValue)
+    ]
+}
+#endif
diff --git a/Tests/RemoteImageTests/Services/PhotoKitServiceTests.swift b/Tests/RemoteImageTests/Services/PhotoKitServiceTests.swift
@@ -87,4 +87,10 @@ final class PhotoKitServiceTests: XCTestCase {
             XCTFail("Invalid photo data result")
         }
     }
+
+    static var allTests = [
+        ("testPhotoDataNotFound", testPhotoDataNotFound),
+        ("testPhotoDataFailure", testPhotoDataFailure),
+        ("testPhotoDataSuccess", testPhotoDataSuccess)
+    ]
 }
diff --git a/Tests/RemoteImageTests/Services/RemoteImageServiceDependenciesTests.swift b/Tests/RemoteImageTests/Services/RemoteImageServiceDependenciesTests.swift
@@ -15,4 +15,8 @@ final class RemoteImageServiceDependenciesTests: XCTestCase {
         XCTAssertTrue(dependencies.photoKitService is PhotoKitService)
         XCTAssertTrue(dependencies.remoteImageURLDataPublisher is URLSession)
     }
+
+    static var allTests = [
+        ("testInitialization", testInitialization)
+    ]
 }
diff --git a/Tests/RemoteImageTests/Services/RemoteImageServiceFactoryTests.swift b/Tests/RemoteImageTests/Services/RemoteImageServiceFactoryTests.swift
@@ -14,5 +14,9 @@ final class RemoteImageServiceFactoryTests: XCTestCase {
         let service = RemoteImageServiceFactory.makeRemoteImageService()
         XCTAssertEqual(service.state, .loading)
     }
+
+    static var allTests = [
+        ("testMakeRemoteImageService", testMakeRemoteImageService)
+    ]
 }
 #endif
diff --git a/Tests/RemoteImageTests/Services/RemoteImageServiceTests.swift b/Tests/RemoteImageTests/Services/RemoteImageServiceTests.swift
@@ -285,5 +285,16 @@ final class RemoteImageServiceTests: XCTestCase {
             XCTFail("Invalid fetch ph asset cached result")
         }
     }
+
+    static var allTests = [
+        ("testFetchImageURLSuccess", testFetchImageURLSuccess),
+        ("testFetchImageURLFailure", testFetchImageURLFailure),
+        ("testFetchImageURLFailureCompletion", testFetchImageURLFailureCompletion),
+        ("testFetchImageURLCached", testFetchImageURLCached),
+        ("testFetchPHAssetSuccess", testFetchPHAssetSuccess),
+        ("testFetchPHAccessInvalidData", testFetchPHAccessInvalidData),
+        ("testFetchPHAccessFailure", testFetchPHAccessFailure),
+        ("testFetchPHAssetCached", testFetchPHAssetCached)
+    ]
 }
 #endif
diff --git a/Tests/RemoteImageTests/Views/RemoteImageTests.swift b/Tests/RemoteImageTests/Views/RemoteImageTests.swift
@@ -34,5 +34,9 @@ final class RemoteImageTests: XCTestCase {
             XCTFail("\(error)")
         }
     }
+
+    static var allTests = [
+        ("testLoadingState", testLoadingState)
+    ]
 }
 #endif
diff --git a/Tests/RemoteImageTests/XCTestManifests.swift b/Tests/RemoteImageTests/XCTestManifests.swift
@@ -3,7 +3,16 @@ import XCTest
 #if !canImport(ObjectiveC)
 public func allTests() -> [XCTestCaseEntry] {
     return [
-        testCase(DefaultRemoteImageCacheTests.allTests)
+        testCase(DefaultRemoteImageCacheTests.allTests),
+        testCase(PhotoKitServiceErrorTests.allTests),
+        testCase(PhotoKitServiceTests.allTests),
+        testCase(RemoteImageTests.allTests),
+        testCase(RemoteImageServiceDependenciesTests.allTests),
+        testCase(RemoteImageServiceErrorTests.allTests),
+        testCase(RemoteImageServiceFactoryTests.allTests),
+        testCase(RemoteImageServiceTests.allTests),
+        testCase(RemoteImageStateTests.allTests),
+        testCase(URLSession_RemoteImageURLDataPublisherTests.allTests)
     ]
 }
 #endif