reverted: HTTPURLResponse does not need to be part of Resource

Author: hoereth

Source/NetworkServices/BasicNetworkService.swift

@@ -84,7 +84,7 @@ public final class BasicNetworkService: NetworkService {
             }
 
             do {
-                return .success((try resource.parse(response, data), response))
+                return .success((try resource.parse(data), response))
             } catch let error {
                 return .failure(.serializationError(error: error, response: response, data: data))
             }

Source/NetworkServices/NetworkServiceMock.swift

@@ -182,7 +182,7 @@ public final class NetworkServiceMock: NetworkService, @unchecked Sendable  {
             switch scheduled {
             case .success((let data, let httpURLResponse)):
                 do {
-                    let result = try resource.parse(httpURLResponse, data)
+                    let result = try resource.parse(data)
                     return .success((result, httpURLResponse))
                 } catch {
                     fatalError("Not able to parse data. Error: \(error)")

Source/Resource+Decodable.swift

@@ -34,7 +34,7 @@ extension Resource where Model: Decodable {
     ///   - mapError: a closure which maps to Error
     public init(request: URLRequest, decoder: JSONDecoder, mapError: @escaping @Sendable  (_ networkError: NetworkError) -> E) {
         self.init(request: request, parse: {
-                try decoder.decode(Model.self, from: $1)
+                try decoder.decode(Model.self, from: $0)
             },
             mapError: mapError
         )
@@ -52,7 +52,7 @@ extension Resource where Model: Decodable, E: NetworkErrorConvertible {
     ///   - mapError: a closure which maps to Error
     public init(request: URLRequest, decoder: JSONDecoder) {
         self.init(request: request, parse: {
-                try decoder.decode(Model.self, from: $1)
+                try decoder.decode(Model.self, from: $0)
             },
             mapError: {
                 E(networkError: $0)

Source/Resource+Inspect.swift

@@ -36,10 +36,10 @@ extension Resource {
      - parameter inspector: closure which gets passed the data
      - returns: a new resource which gets instepcted before parsing
      */
-    public func inspectData(_ inspector: @escaping @Sendable  (HTTPURLResponse, Data) -> Void) -> Resource<Model, E> {
-        let parse: @Sendable (HTTPURLResponse, Data) throws -> Model = { response, data in
-            inspector(response, data)
-            return try self.parse(response, data)
+    public func inspectData(_ inspector: @escaping @Sendable (Data) -> Void) -> Resource<Model, E> {
+        let parse: @Sendable (Data) throws -> Model = { data in
+            inspector(data)
+            return try self.parse(data)
         }
         return Resource<Model, E>(request: request, parse: parse, mapError: mapError)
     }

Source/Resource+Map.swift

@@ -30,8 +30,8 @@ extension Resource {
     public func map<T>(transform: @escaping @Sendable  (Model) throws -> T) -> Resource<T, E> {
         return Resource<T, E>(
             request: request,
-            parse: { response, data in
-                return try transform(try self.parse(response, data))
+            parse: { data in
+                return try transform(try self.parse(data))
             },
             mapError: mapError
         )

Source/Resource+Void.swift

@@ -15,7 +15,7 @@ public extension Resource where Model == Void {
     ///   - request: The request to get the remote data payload
     ///   - mapError: a closure which maps to Error
     init(request: URLRequest, mapError: @escaping @Sendable  (_ networkError: NetworkError) -> E) {
-        self.init(request: request, parse: { _, _ in }, mapError: mapError)
+        self.init(request: request, parse: { _ in }, mapError: mapError)
     }
 }
 
@@ -26,7 +26,7 @@ public extension Resource where Model == Void, E: NetworkErrorConvertible {
     /// - Parameters:
     ///   - request: The request to get the remote data payload
     init(request: URLRequest) {
-        self.init(request: request, parse: { _, _ in }, mapError: { E(networkError: $0) })
+        self.init(request: request, parse: { _ in }, mapError: { E(networkError: $0) })
     }
 
 }

Source/Resource.swift

@@ -40,7 +40,7 @@ public struct Resource<Model, E: Error>: Sendable {
     public let request: URLRequest
 
     /// Parses data into given model.
-    public let parse: @Sendable (_ response: HTTPURLResponse, _ data: Data) throws -> Model
+    public let parse: @Sendable (_ data: Data) throws -> Model
     public let mapError: @Sendable (_ networkError: NetworkError) -> E
 
     /// Creates a type safe resource, which can be used to fetch it with NetworkService
@@ -49,7 +49,7 @@ public struct Resource<Model, E: Error>: Sendable {
     /// - request: The request to get the remote data payload
     /// - parse: Parses data fetched with the request into given Model
 
-    public init(request: URLRequest, parse: @escaping @Sendable (HTTPURLResponse, Data) throws -> Model, mapError: @escaping @Sendable  (_ networkError: NetworkError) -> E) {
+    public init(request: URLRequest, parse: @escaping @Sendable  (Data) throws -> Model, mapError: @escaping @Sendable  (_ networkError: NetworkError) -> E) {
         self.request = request
         self.parse = parse
         self.mapError = mapError

Source/ResourceWithError+NetworkErrorConvertible.swift

@@ -9,7 +9,7 @@ import Foundation
 
 public extension Resource where E: NetworkErrorConvertible {
 
-    init(request: URLRequest, parse: @escaping @Sendable (HTTPURLResponse, Data) throws -> Model) {
+    init(request: URLRequest, parse: @escaping @Sendable (Data) throws -> Model) {
         self.request = request
         self.parse = parse
         self.mapError = { E(networkError: $0) }