Skip to content

Commit 226c1ba

Browse files
committed
reverted: HTTPURLResponse does not need to be part of Resource
1 parent a1775fb commit 226c1ba

File tree

6 files changed

+16
-16
lines changed

6 files changed

+16
-16
lines changed

Tests/DecodableResoureTest.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ class DecodableResoureTest: XCTestCase {
3333

3434
func testResource_withValidData() {
3535
//When
36-
let fetchedTrain = try? resource.parse(HTTPURLResponse.defaultMock, Train.validJSONData)
36+
let fetchedTrain = try? resource.parse(Train.validJSONData)
3737

3838
//Then
3939
XCTAssertEqual(fetchedTrain?.name, "ICE")
@@ -42,7 +42,7 @@ class DecodableResoureTest: XCTestCase {
4242
func testResource_withMAppedResult() {
4343
//When
4444
let nameResource = resource.map { $0.name }
45-
let fetchedTrainName = try? nameResource.parse(HTTPURLResponse.defaultMock, Train.validJSONData)
45+
let fetchedTrainName = try? nameResource.parse(Train.validJSONData)
4646

4747
//Then
4848
XCTAssertEqual(fetchedTrainName, "ICE")
@@ -51,7 +51,7 @@ class DecodableResoureTest: XCTestCase {
5151
func testResource_WithInvalidData() throws {
5252
//When
5353
do {
54-
_ = try resource.parse(HTTPURLResponse.defaultMock, Train.invalidJSONData)
54+
_ = try resource.parse(Train.invalidJSONData)
5555
XCTFail("Expected method to throws")
5656
} catch { }
5757
}

Tests/ModifyRequestNetworkService.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class ModifyRequestNetworkServiceTest: XCTestCase {
3535
]
3636
let networkService = ModifyRequestNetworkService(networkService: networkServiceMock, requestModifications: modification)
3737
let request = URLRequest(path: "/trains", baseURL: .defaultMock)
38-
let resource = Resource<Int, NetworkError>(request: request, parse: { _, _ in return 1 })
38+
let resource = Resource<Int, NetworkError>(request: request, parse: { _ in return 1 })
3939

4040
//When
4141
await networkService.requestResult(for: resource)

Tests/ResourceInspectTest.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,16 +30,16 @@ final class ResourceInspectTest: XCTestCase {
3030
let capturedParsingData = Container<Data?>(nil)
3131
let capturedInspectedData = Container<Data?>(nil)
3232

33-
let resource = Resource<Int, NetworkError>(request: URLRequest.defaultMock, parse: { response, data in
33+
let resource = Resource<Int, NetworkError>(request: URLRequest.defaultMock, parse: { data in
3434
capturedParsingData.setValue(data)
3535
return 1
3636
})
3737

3838
//When
39-
let inspectedResource = resource.inspectData({ response, data in
39+
let inspectedResource = resource.inspectData({ data in
4040
capturedInspectedData.setValue(data)
4141
})
42-
let result = try? inspectedResource.parse(HTTPURLResponse.defaultMock, data)
42+
let result = try? inspectedResource.parse(data)
4343

4444
//Then
4545
XCTAssertNotNil(result)

Tests/ResourceTest.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,10 @@ class ResourceTest: XCTestCase {
3131
//Given
3232
let validData: Data! = "ICE".data(using: .utf8)
3333

34-
let resource = Resource<String?, NetworkError>(request: URLRequest.defaultMock, parse: { String(data: $1, encoding: .utf8) })
34+
let resource = Resource<String?, NetworkError>(request: URLRequest.defaultMock, parse: { String(data: $0, encoding: .utf8) })
3535

3636
//When
37-
let name = try resource.parse(HTTPURLResponse.defaultMock, validData)
37+
let name = try resource.parse(validData)
3838

3939
//Then
4040
XCTAssertEqual(name, "ICE")
@@ -45,7 +45,7 @@ class ResourceTest: XCTestCase {
4545
let resource = Resource<Void, NetworkError>(request: URLRequest.defaultMock)
4646

4747
//When
48-
try resource.parse(HTTPURLResponse.defaultMock, Data())
48+
try resource.parse(Data())
4949
}
5050

5151
}

Tests/ResourceWithErrorTest.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ class ResourceWithErrorTest: XCTestCase {
3333

3434
let resource = Resource<String?, NetworkError>(
3535
request: URLRequest.defaultMock,
36-
parse: { String(data: $1, encoding: .utf8) },
36+
parse: { String(data: $0, encoding: .utf8) },
3737
mapError: { $0 }
3838
)
3939

4040
//When
41-
let name = try resource.parse(HTTPURLResponse.defaultMock, validData)
41+
let name = try resource.parse(validData)
4242

4343
//Then
4444
XCTAssertEqual(name, "ICE")
@@ -50,12 +50,12 @@ class ResourceWithErrorTest: XCTestCase {
5050

5151
let resource = Resource<String?, NetworkError>(
5252
request: URLRequest.defaultMock,
53-
parse: { String(data: $1, encoding: .utf8) },
53+
parse: { String(data: $0, encoding: .utf8) },
5454
mapError: { $0 }
5555
)
5656

5757
//When
58-
let numberOfCharacters = try resource.map(transform: { $0?.count }).parse(HTTPURLResponse.defaultMock, validData)
58+
let numberOfCharacters = try resource.map(transform: { $0?.count }).parse(validData)
5959

6060
//Then
6161
XCTAssertEqual(numberOfCharacters, 3)
@@ -86,7 +86,7 @@ class ResourceWithErrorTest: XCTestCase {
8686
)
8787

8888
//When
89-
try resource.parse(HTTPURLResponse.defaultMock, Data())
89+
try resource.parse(Data())
9090
}
9191

9292
}

Tests/RetryNetworkserviceTest.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import XCTest
2727
class RetryNetworkserviceTest: XCTestCase {
2828
var resource: Resource<Int, NetworkError> {
2929
let request = URLRequest(path: "/train", baseURL: .defaultMock)
30-
return Resource(request: request, parse: { _, _ in return 1})
30+
return Resource(request: request, parse: { _ in return 1})
3131
}
3232

3333
func testRetryRequest_shouldRetry() async throws {

0 commit comments

Comments
 (0)