Minor improvements

Author: sebastianvarela

Sources/Publishers/Publishers.ExpiredScope.swift

@@ -4,7 +4,7 @@ import Foundation
 public extension Publisher {
     /// From a publisher, we can focus on a task and filter all expired and duplicated task. This publisher doesn't send a value if at the moment of subscription there is an expired task.
     @available(*, deprecated, renamed: "scope")
-    func expiredScope<T: Taskable & Equatable>(_ transform: @escaping (Self.Output) -> T, margin: TimeInterval = taskDefaultMargin) -> AnyPublisher<T, Failure> {
+    func expiredScope<T: Taskable>(_ transform: @escaping (Self.Output) -> T, margin: TimeInterval = taskDefaultMargin) -> AnyPublisher<T, Failure> {
         map(transform)
             .removeExpired(margin: margin)
             .removeDuplicates()

Sources/Publishers/Publishers.MapToLatestTask.swift

@@ -5,8 +5,8 @@ public extension Publisher {
         transform: @escaping ((Output.Payload.ID) -> (AnyPublisher<DownTask, Failure>))
     )
     -> AnyPublisher<DownTask, Failure>
-    where Output: Taskable & Equatable, Output.Payload: Identifiable,
-          DownTask: Taskable & Equatable, DownTask.Failure == Output.Failure {
+    where Output: Taskable, Output.Payload: Identifiable,
+          DownTask: Taskable, DownTask.Failure == Output.Failure {
         map { (task: Output) -> AnyPublisher<DownTask, Failure> in
             switch task.status {
             case .success(let payload):

Sources/Publishers/Publishers.RemoveExpired.swift

@@ -14,7 +14,7 @@ public extension Publishers {
         public typealias Output = Upstream.Output
         public typealias Failure = Upstream.Failure
 
-        public let upstream: Upstream
+        private let upstream: Upstream
         private let margin: TimeInterval
 
         public init(upstream: Upstream, margin: TimeInterval) {

Sources/Task/Task.swift

@@ -2,7 +2,7 @@ import Foundation
 
 public let taskDefaultMargin: TimeInterval = 0.250
 
-public class Task<T: Equatable, E: Error & Equatable>: Taskable, Equatable, CustomDebugStringConvertible {
+public class Task<T: Equatable, E: Error & Equatable>: Taskable, CustomDebugStringConvertible, CustomStringConvertible {
     public typealias Payload = T
     public typealias Failure = E
 
@@ -125,19 +125,14 @@ public class Task<T: Equatable, E: Error & Equatable>: Taskable, Equatable, Cust
               progress: progress)
     }
 
+    // MARK: - CustomStringConvertible
+    public var description: String {
+        "[TASK] \(status) - started: \(started)"
+    }
+
     // MARK: - CustomDebugStringConvertible
     public var debugDescription: String {
-        let tagPrint: String
-        if let tag = tag {
-            tagPrint = tag
-        } else {
-            tagPrint = "nil"
-        }
-
-        return """
-        🚀 Task: status: \(status), started: \(started), tag: \(tagPrint)
-        payload: \(String(describing: payload)), progress: \(String(describing: progress)) error: \(String(describing: error))
-        """
+        "[TASK] \(status) - started: \(started) - expiration \(expiration)"
     }
 
     // MARK: Equatable

Sources/Task/TaskStatus.swift

@@ -1,4 +1,4 @@
-public enum TaskStatus<Payload: Equatable, Failure: Error & Equatable>: Equatable {
+public enum TaskStatus<Payload: Equatable, Failure: Error & Equatable>: Equatable, CustomDebugStringConvertible, CustomStringConvertible {
     case idle
     case running
     case success(payload: Payload)
@@ -19,4 +19,23 @@ public enum TaskStatus<Payload: Equatable, Failure: Error & Equatable>: Equatabl
             return false
         }
     }
+
+    // MARK: - CustomStringConvertible
+    public var description: String {
+        switch self {
+        case .idle:
+            return "⚪️ idle"
+        case .running:
+            return "🌕 Running"
+        case .success(let payload):
+            return "🟢 Success - payload: \(payload)"
+        case .failure(let error):
+            return "🔴 Failure - error: \(error)"
+        }
+    }
+
+    // MARK: - CustomDebugStringConvertible
+    public var debugDescription: String {
+        description
+    }
 }

Sources/Task/Taskable.swift

@@ -1,6 +1,6 @@
 import Foundation
 
-public protocol Taskable {
+public protocol Taskable: Equatable {
     associatedtype Payload: Equatable
     associatedtype Failure: Error & Equatable
 

Tests/StoreTests.swift

@@ -27,7 +27,7 @@ class StoreTests: XCTestCase {
         // THIS PASSES
         store.state = TestState(testTask: .success(1), counter: 1)
 
-        // THIS DOES NOT, had the same success value as the previous one
+        // THIS NOT PASS, had the same success value as the previous one
         store.state = TestState(testTask: .success(1), counter: 2)
 
         // THIS PASSES