| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268 |
- //
- // RetryStrategyTests.swift
- // Kingfisher
- //
- // Created by onevcat on 2020/05/06.
- //
- // Copyright (c) 2020 Wei Wang <onevcat@gmail.com>
- //
- // Permission is hereby granted, free of charge, to any person obtaining a copy
- // of this software and associated documentation files (the "Software"), to deal
- // in the Software without restriction, including without limitation the rights
- // to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- // copies of the Software, and to permit persons to whom the Software is
- // furnished to do so, subject to the following conditions:
- //
- // The above copyright notice and this permission notice shall be included in
- // all copies or substantial portions of the Software.
- //
- // THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- // IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- // FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- // AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
- // THE SOFTWARE.
- import XCTest
- @testable import Kingfisher
- class RetryStrategyTests: XCTestCase {
- var manager: KingfisherManager!
- override class func setUp() {
- super.setUp()
- LSNocilla.sharedInstance().start()
- }
- override class func tearDown() {
- LSNocilla.sharedInstance().stop()
- super.tearDown()
- }
- override func setUpWithError() throws {
- try super.setUpWithError()
- let uuid = UUID()
- let downloader = ImageDownloader(name: "test.manager.\(uuid.uuidString)")
- let cache = ImageCache(name: "test.cache.\(uuid.uuidString)")
- manager = KingfisherManager(downloader: downloader, cache: cache)
- manager.defaultOptions = [.waitForCache]
- }
- override func tearDownWithError() throws {
- LSNocilla.sharedInstance().clearStubs()
- clearCaches([manager.cache])
- cleanDefaultCache()
- manager = nil
- try super.tearDownWithError()
- }
- func testCanCreateRetryStrategy() {
- let strategy = DelayRetryStrategy(maxRetryCount: 10, retryInterval: .seconds(5))
- XCTAssertEqual(strategy.maxRetryCount, 10)
- XCTAssertEqual(strategy.retryInterval.timeInterval(for: 0), 5)
- }
- func testDelayRetryIntervalCalculating() {
- let secondInternal = DelayRetryStrategy.Interval.seconds(10)
- XCTAssertEqual(secondInternal.timeInterval(for: 0), 10)
- let accumulatedInternal = DelayRetryStrategy.Interval.accumulated(3)
- XCTAssertEqual(accumulatedInternal.timeInterval(for: 0), 3)
- XCTAssertEqual(accumulatedInternal.timeInterval(for: 1), 6)
- XCTAssertEqual(accumulatedInternal.timeInterval(for: 2), 9)
- XCTAssertEqual(accumulatedInternal.timeInterval(for: 3), 12)
- let customInternal = DelayRetryStrategy.Interval.custom { TimeInterval($0 * 2) }
- XCTAssertEqual(customInternal.timeInterval(for: 0), 0)
- XCTAssertEqual(customInternal.timeInterval(for: 1), 2)
- XCTAssertEqual(customInternal.timeInterval(for: 2), 4)
- XCTAssertEqual(customInternal.timeInterval(for: 3), 6)
- }
- func testKingfisherManagerCanRetry() {
- let exp = expectation(description: #function)
- let brokenURL = URL(string: "brokenurl")!
- stub(brokenURL, data: Data())
- let retry = StubRetryStrategy()
- _ = manager.retrieveImage(
- with: .network(brokenURL),
- options: [.retryStrategy(retry)],
- completionHandler: { result in
- XCTAssertEqual(retry.count, 3)
- exp.fulfill()
- }
- )
- waitForExpectations(timeout: 1, handler: nil)
- }
- func testDelayRetryStrategyExceededCount() {
- let exp = expectation(description: #function)
- let blockCalled: ActorArray<Bool> = ActorArray([])
- let source = Source.network(URL(string: "url")!)
- let retry = DelayRetryStrategy(maxRetryCount: 3, retryInterval: .seconds(0))
- let group = DispatchGroup()
-
- group.enter()
- let context1 = RetryContext(
- source: source,
- error: .responseError(reason: .URLSessionError(error: E()))
- )
- retry.retry(context: context1) { decision in
- guard case RetryDecision.retry(let userInfo) = decision else {
- XCTFail("The decision should be `retry`")
- return
- }
- XCTAssertNil(userInfo)
- Task {
- await blockCalled.append(true)
- group.leave()
- }
- }
- group.enter()
- let context2 = RetryContext(
- source: source,
- error: .responseError(reason: .URLSessionError(error: E()))
- )
- context2.increaseRetryCount() // 1
- context2.increaseRetryCount() // 2
- context2.increaseRetryCount() // 3
- retry.retry(context: context2) { decision in
- guard case RetryDecision.stop = decision else {
- XCTFail("The decision should be `stop`")
- return
- }
- Task {
- await blockCalled.append(true)
- group.leave()
- }
- }
-
- group.notify(queue: .main) {
- Task {
- let result = await blockCalled.value
- XCTAssertEqual(result.count, 2)
- XCTAssertTrue(result.allSatisfy { $0 })
- exp.fulfill()
- }
- }
- waitForExpectations(timeout: 1, handler: nil)
- }
- func testDelayRetryStrategyNotRetryForErrorReason() {
- let exp = expectation(description: #function)
- // Only non-user cancel error && response error should be retied.
- let blockCalled: ActorArray<Bool> = ActorArray([])
- let source = Source.network(URL(string: "url")!)
- let retry = DelayRetryStrategy(maxRetryCount: 3, retryInterval: .seconds(0))
- let task = URLSession.shared.dataTask(with: URL(string: "url")!)
- let group = DispatchGroup()
- group.enter()
- let context1 = RetryContext(
- source: source,
- error: .requestError(reason: .taskCancelled(task: .init(task: task), token: .init()))
- )
- retry.retry(context: context1) { decision in
- guard case RetryDecision.stop = decision else {
- XCTFail("The decision should be `stop` if user cancelled the task.")
- return
- }
- Task {
- await blockCalled.append(true)
- group.leave()
- }
- }
- group.enter()
- let context2 = RetryContext(
- source: source,
- error: .cacheError(reason: .imageNotExisting(key: "any_key"))
- )
- retry.retry(context: context2) { decision in
- guard case RetryDecision.stop = decision else {
- XCTFail("The decision should be `stop` if the error type is not response error.")
- return
- }
- Task {
- await blockCalled.append(true)
- group.leave()
- }
- }
-
- group.notify(queue: .main) {
- Task {
- let result = await blockCalled.value
- XCTAssertEqual(result.count, 2)
- XCTAssertTrue(result.allSatisfy { $0 })
- exp.fulfill()
- }
- }
- waitForExpectations(timeout: 1, handler: nil)
- }
- func testDelayRetryStrategyDidRetried() {
- let called = ActorBox(false)
- let source = Source.network(URL(string: "url")!)
- let retry = DelayRetryStrategy(maxRetryCount: 3, retryInterval: .seconds(0))
- let context = RetryContext(
- source: source,
- error: .responseError(reason: .URLSessionError(error: E()))
- )
- retry.retry(context: context) { decision in
- guard case RetryDecision.retry = decision else {
- XCTFail("The decision should be `retry`.")
- return
- }
- Task {
- await called.setValue(true)
- }
- }
- Task {
- let result = await called.value
- XCTAssertTrue(result)
- }
-
- }
- }
- private struct E: Error {}
- final class StubRetryStrategy: RetryStrategy, @unchecked Sendable {
- let queue = DispatchQueue(label: "com.onevcat.KingfisherTests.StubRetryStrategy")
- var _count = 0
- var count: Int {
- get { queue.sync { _count } }
- set { queue.sync { _count = newValue } }
- }
- func retry(context: RetryContext, retryHandler: @escaping (RetryDecision) -> Void) {
- if count == 0 {
- XCTAssertNil(context.userInfo)
- } else {
- XCTAssertEqual(context.userInfo as! Int, count)
- }
- XCTAssertEqual(context.retriedCount, count)
- count += 1
- if count == 3 {
- retryHandler(.stop)
- } else {
- retryHandler(.retry(userInfo: count))
- }
- }
- }
|