diff --git a/Examples/xx-messenger/Sources/AppCore/ReceiveFileHandler/ReceiveFileHandler.swift b/Examples/xx-messenger/Sources/AppCore/ReceiveFileHandler/ReceiveFileHandler.swift
index 84abe3c5edf058f205978bc836758a52bd379e0f..733a05229ed500f4996c43de01c4cef2fac47c43 100644
--- a/Examples/xx-messenger/Sources/AppCore/ReceiveFileHandler/ReceiveFileHandler.swift
+++ b/Examples/xx-messenger/Sources/AppCore/ReceiveFileHandler/ReceiveFileHandler.swift
@@ -5,14 +5,6 @@ import XXMessengerClient
import XXModels
public struct ReceiveFileHandler {
- public struct ProgressError: Error {
- public init(message: String) {
- self.message = message
- }
-
- public var message: String
- }
-
public typealias OnError = (Error) -> Void
public var run: (@escaping OnError) -> Cancellable
@@ -75,9 +67,6 @@ extension ReceiveFileHandler {
case .failed(.callbackError(let error)):
onError(error)
-
- case .failed(.progressError(let message)):
- onError(ProgressError(message: message))
}
}
} catch {
diff --git a/Examples/xx-messenger/Sources/AppCore/SendImage/SendImage.swift b/Examples/xx-messenger/Sources/AppCore/SendImage/SendImage.swift
index a457dcd49e11c3e06975a7151a6bd8a9dd839412..00720b568589effc2f56f2c10d20334036b9ba00 100644
--- a/Examples/xx-messenger/Sources/AppCore/SendImage/SendImage.swift
+++ b/Examples/xx-messenger/Sources/AppCore/SendImage/SendImage.swift
@@ -5,14 +5,6 @@ import XXMessengerClient
import XXModels
public struct SendImage {
- public struct ProgressError: Error, Equatable {
- public init(message: String) {
- self.message = message
- }
-
- public var message: String
- }
-
public typealias OnError = (Error) -> Void
public typealias Completion = () -> Void
@@ -78,9 +70,6 @@ extension SendImage {
case .failed(_, .error(let error)):
onError(error)
- case .failed(_, .progressError(let message)):
- onError(ProgressError(message: message))
-
case .failed(_, .close(let error)):
onError(error)
}
diff --git a/Examples/xx-messenger/Tests/AppCoreTests/ReceiveFileHandler/ReceiveFileHandlerTests.swift b/Examples/xx-messenger/Tests/AppCoreTests/ReceiveFileHandler/ReceiveFileHandlerTests.swift
index 6d54a9cfcc27db703a70f723b29bbc0833e1f5b1..ce6a2e223944754a2bba2c8b4cf5b6cbb8717e24 100644
--- a/Examples/xx-messenger/Tests/AppCoreTests/ReceiveFileHandler/ReceiveFileHandlerTests.swift
+++ b/Examples/xx-messenger/Tests/AppCoreTests/ReceiveFileHandler/ReceiveFileHandlerTests.swift
@@ -138,14 +138,6 @@ final class ReceiveFileHandlerTests: XCTestCase {
.didCatchError(receivingFileCallbackError)
])
- actions = []
- let receivingFileProgressError = "receiving-file-progress"
- receivingFileCallback?(.failed(.progressError(receivingFileProgressError)))
-
- XCTAssertNoDifference(actions, [
- .didCatchError(ReceiveFileHandler.ProgressError(message: receivingFileProgressError) as NSError)
- ])
-
actions = []
receivingFileCallback?(.progress(transmitted: 1, total: 2))
diff --git a/Examples/xx-messenger/Tests/AppCoreTests/SendImage/SendImageTests.swift b/Examples/xx-messenger/Tests/AppCoreTests/SendImage/SendImageTests.swift
index d1cb1741abe2ffdbd2f4481800ca8529943cb9df..1afbc1c8ba3d56f46b9aa46a8912aeea19a78b82 100644
--- a/Examples/xx-messenger/Tests/AppCoreTests/SendImage/SendImageTests.swift
+++ b/Examples/xx-messenger/Tests/AppCoreTests/SendImage/SendImageTests.swift
@@ -112,14 +112,6 @@ final class SendImageTests: XCTestCase {
.didFail(closeError),
])
- actions = []
- let progressError = "progress-error"
- sendFileCallback?(.failed(id: transferId, .progressError(progressError)))
-
- XCTAssertNoDifference(actions, [
- .didFail(SendImage.ProgressError(message: progressError) as NSError),
- ])
-
actions = []
sendFileCallback?(.progress(id: transferId, transmitted: 1, total: 2))
diff --git a/Sources/XXClient/Models/Progress.swift b/Sources/XXClient/Models/Progress.swift
index ba2e199db7fb504d790ef6ef514c1e25f9c794eb..5d4e13641a1dc9bba9170b42643b9e6c7dec9afd 100644
--- a/Sources/XXClient/Models/Progress.swift
+++ b/Sources/XXClient/Models/Progress.swift
@@ -4,19 +4,16 @@ public struct Progress: Equatable {
public init(
completed: Bool,
transmitted: Int,
- total: Int,
- error: String?
+ total: Int
) {
self.completed = completed
self.transmitted = transmitted
self.total = total
- self.error = error
}
public var completed: Bool
public var transmitted: Int
public var total: Int
- public var error: String?
}
extension Progress: Codable {
@@ -24,7 +21,6 @@ extension Progress: Codable {
case completed = "Completed"
case transmitted = "Transmitted"
case total = "Total"
- case error = "Err"
}
public static func decode(_ data: Data) throws -> Self {
diff --git a/Sources/XXMessengerClient/Messenger/Functions/MessengerReceiveFile.swift b/Sources/XXMessengerClient/Messenger/Functions/MessengerReceiveFile.swift
index ef4290f829d7811c4e0c94c31afa812c095edfbc..399de8c0dfa03bf0de43c8d61eb31f0181ff3d9e 100644
--- a/Sources/XXMessengerClient/Messenger/Functions/MessengerReceiveFile.swift
+++ b/Sources/XXMessengerClient/Messenger/Functions/MessengerReceiveFile.swift
@@ -19,7 +19,6 @@ public struct MessengerReceiveFile {
public enum CallbackInfo: Equatable {
public enum Failure: Equatable {
case callbackError(NSError)
- case progressError(String)
case receiveError(NSError)
}
@@ -56,9 +55,7 @@ extension MessengerReceiveFile {
callback: FileTransferProgressCallback { result in
switch result {
case .success(let info):
- if let error = info.progress.error {
- callback(.failed(.progressError(error)))
- } else if info.progress.completed {
+ if info.progress.completed {
do {
callback(.finished(try fileTransfer.receive(transferId: params.transferId)))
} catch {
diff --git a/Sources/XXMessengerClient/Messenger/Functions/MessengerSendFile.swift b/Sources/XXMessengerClient/Messenger/Functions/MessengerSendFile.swift
index fde8607c42e6e7f6d8d4c97da9beca79e588c589..8a0b17765655bd61d5319639935577f589f4b489 100644
--- a/Sources/XXMessengerClient/Messenger/Functions/MessengerSendFile.swift
+++ b/Sources/XXMessengerClient/Messenger/Functions/MessengerSendFile.swift
@@ -25,7 +25,6 @@ public struct MessengerSendFile {
public enum CallbackInfo: Equatable {
public enum Failure: Equatable {
case error(NSError)
- case progressError(String)
case close(NSError)
}
@@ -81,10 +80,7 @@ extension MessengerSendFile {
close(id: transferId)
case .success(let cb):
- if let error = cb.progress.error {
- callback(.failed(id: transferId, .progressError(error)))
- close(id: transferId)
- } else if cb.progress.completed {
+ if cb.progress.completed {
callback(.finished(id: transferId))
close(id: transferId)
} else {
diff --git a/Tests/XXClientTests/Models/ProgressTests.swift b/Tests/XXClientTests/Models/ProgressTests.swift
index 35a083c202a1e3d35b6c75860b9f8aa3b39d0d69..6b8342692e806898525a32db405760d8dbf2305a 100644
--- a/Tests/XXClientTests/Models/ProgressTests.swift
+++ b/Tests/XXClientTests/Models/ProgressTests.swift
@@ -11,8 +11,7 @@ final class ProgressTests: XCTestCase {
{
"Completed": \(completed),
"Transmitted": \(transmitted),
- "Total": \(total),
- "Err": null
+ "Total": \(total)
}
"""
let jsonData = jsonString.data(using: .utf8)!
@@ -21,8 +20,7 @@ final class ProgressTests: XCTestCase {
XCTAssertNoDifference(model, Progress(
completed: completed,
transmitted: transmitted,
- total: total,
- error: nil
+ total: total
))
let encodedModel = try model.encode()
@@ -30,28 +28,4 @@ final class ProgressTests: XCTestCase {
XCTAssertNoDifference(decodedModel, model)
}
-
- func testDecodingProgressWithError() throws {
- let completed = false
- let transmitted: Int = 128
- let total: Int = 2048
- let error = "something went wrong"
- let jsonString = """
- {
- "Completed": \(completed),
- "Transmitted": \(transmitted),
- "Total": \(total),
- "Err": "\(error)"
- }
- """
- let jsonData = jsonString.data(using: .utf8)!
- let model = try Progress.decode(jsonData)
-
- XCTAssertNoDifference(model, Progress(
- completed: completed,
- transmitted: transmitted,
- total: total,
- error: error
- ))
- }
}
diff --git a/Tests/XXMessengerClientTests/Messenger/Functions/MessengerReceiveFileTests.swift b/Tests/XXMessengerClientTests/Messenger/Functions/MessengerReceiveFileTests.swift
index 554b7f309114fbdaa2ea2679d290d09f238eaa52..b2fcefd858d2071d671117b10c9bb818a8ce7139 100644
--- a/Tests/XXMessengerClientTests/Messenger/Functions/MessengerReceiveFileTests.swift
+++ b/Tests/XXMessengerClientTests/Messenger/Functions/MessengerReceiveFileTests.swift
@@ -48,8 +48,7 @@ final class MessengerReceiveFileTests: XCTestCase {
progress: Progress(
completed: false,
transmitted: 1,
- total: 3,
- error: nil
+ total: 3
),
partTracker: .unimplemented
)
@@ -65,8 +64,7 @@ final class MessengerReceiveFileTests: XCTestCase {
progress: Progress(
completed: false,
transmitted: 2,
- total: 3,
- error: nil
+ total: 3
),
partTracker: .unimplemented
)
@@ -82,8 +80,7 @@ final class MessengerReceiveFileTests: XCTestCase {
progress: Progress(
completed: true,
transmitted: 3,
- total: 3,
- error: nil
+ total: 3
),
partTracker: .unimplemented
)
@@ -110,43 +107,6 @@ final class MessengerReceiveFileTests: XCTestCase {
}
}
- func testReceiveFileProgressError() throws {
- let error = "Something went wrong..."
-
- var receivedProgressCallback: FileTransferProgressCallback?
- var didReceiveCallback: [MessengerReceiveFile.CallbackInfo] = []
-
- var env: MessengerEnvironment = .unimplemented
- env.fileTransfer.get = {
- var fileTransfer: FileTransfer = .unimplemented
- fileTransfer.registerReceivedProgressCallback.run = { _, _, callback in
- receivedProgressCallback = callback
- }
- return fileTransfer
- }
- let receiveFile: MessengerReceiveFile = .live(env)
-
- try receiveFile(.stub) { info in
- didReceiveCallback.append(info)
- }
-
- receivedProgressCallback?.handle(.success(
- FileTransferProgressCallback.Callback(
- progress: Progress(
- completed: false,
- transmitted: 1,
- total: 3,
- error: error
- ),
- partTracker: .unimplemented
- )
- ))
-
- XCTAssertNoDifference(didReceiveCallback, [
- .failed(.progressError(error))
- ])
- }
-
func testReceiveFileCallbackError() throws {
let error = NSError(domain: "test", code: 123)
@@ -202,8 +162,7 @@ final class MessengerReceiveFileTests: XCTestCase {
progress: Progress(
completed: true,
transmitted: 3,
- total: 3,
- error: nil
+ total: 3
),
partTracker: .unimplemented
)
diff --git a/Tests/XXMessengerClientTests/Messenger/Functions/MessengerSendFileTests.swift b/Tests/XXMessengerClientTests/Messenger/Functions/MessengerSendFileTests.swift
index 4e1c1763b489e66206ca18668bd673d8d75c5570..54823eb87584a380eafd98f0b880f5913f20cf1f 100644
--- a/Tests/XXMessengerClientTests/Messenger/Functions/MessengerSendFileTests.swift
+++ b/Tests/XXMessengerClientTests/Messenger/Functions/MessengerSendFileTests.swift
@@ -47,8 +47,7 @@ final class MessengerSendFileTests: XCTestCase {
progress: Progress(
completed: false,
transmitted: 1,
- total: 10,
- error: nil
+ total: 10
),
partTracker: .unimplemented
)))
@@ -56,8 +55,7 @@ final class MessengerSendFileTests: XCTestCase {
progress: Progress(
completed: false,
transmitted: 6,
- total: 10,
- error: nil
+ total: 10
),
partTracker: .unimplemented
)))
@@ -65,8 +63,7 @@ final class MessengerSendFileTests: XCTestCase {
progress: Progress(
completed: true,
transmitted: 10,
- total: 10,
- error: nil
+ total: 10
),
partTracker: .unimplemented
)))
@@ -124,46 +121,6 @@ final class MessengerSendFileTests: XCTestCase {
XCTAssertNoDifference(didCloseSend, [transferId])
}
- func testSendFileProgressError() throws {
- var didCloseSend: [Data] = []
- var didReceiveCallback: [MessengerSendFile.CallbackInfo] = []
- var fileTransferProgressCallback: FileTransferProgressCallback!
-
- var env: MessengerEnvironment = .unimplemented
- env.fileTransfer.get = {
- var fileTransfer: FileTransfer = .unimplemented
- fileTransfer.send.run = { _, callback in
- fileTransferProgressCallback = callback
- return "transferId".data(using: .utf8)!
- }
- fileTransfer.closeSend.run = { id in
- didCloseSend.append(id)
- }
- return fileTransfer
- }
- let sendFile: MessengerSendFile = .live(env)
-
- let transferId = try sendFile(.stub) { info in
- didReceiveCallback.append(info)
- }
-
- let error = "something went wrong"
- fileTransferProgressCallback.handle(.success(.init(
- progress: .init(
- completed: false,
- transmitted: 0,
- total: 0,
- error: error
- ),
- partTracker: .unimplemented
- )))
-
- XCTAssertNoDifference(didReceiveCallback, [
- .failed(id: transferId, .progressError(error)),
- ])
- XCTAssertNoDifference(didCloseSend, [transferId])
- }
-
func testSendFileCloseError() throws {
let closeError = NSError(domain: "test", code: 1234)
@@ -192,8 +149,7 @@ final class MessengerSendFileTests: XCTestCase {
progress: .init(
completed: true,
transmitted: 1,
- total: 1,
- error: nil
+ total: 1
),
partTracker: .unimplemented
)))