Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preparation for 2.0 #93

Merged
merged 8 commits into from Sep 23, 2019
Merged

Conversation

lightsprint09
Copy link
Member

Fixes issues:

  • Removes URLRequestConvertible
  • Removes deprecated methods

New feature:

Make sure to check all boxes before merging

  • Method/Class documentation
  • README.md documentation
  • Unit tests for new features/regressions

Source/ModifyRequestNetworkService.swift Outdated Show resolved Hide resolved
Source/ModifyRequestNetworkService.swift Outdated Show resolved Hide resolved
Source/ModifyRequestNetworkService.swift Outdated Show resolved Hide resolved
Source/ModifyRequestNetworkService.swift Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Apr 3, 2019

Codecov Report

Merging #93 into develop will decrease coverage by 0.46%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #93      +/-   ##
===========================================
- Coverage    93.24%   92.78%   -0.47%     
===========================================
  Files           17       17              
  Lines          311      291      -20     
===========================================
- Hits           290      270      -20     
  Misses          21       21
Impacted Files Coverage Δ
Source/ContainerNetworkTask.swift 100% <ø> (+33.33%) ⬆️
Source/NetworkServiceMock.swift 79.31% <ø> (-15.82%) ⬇️
Source/NetworkTaskMock.swift 100% <ø> (ø) ⬆️
Source/URLRequest+Init.swift 94.54% <ø> (ø)
Source/Resource.swift 100% <100%> (ø) ⬆️
Source/Resource+Decodable.swift 100% <100%> (ø) ⬆️
Source/URLRequestConvertible+Modifications.swift 90.9% <100%> (ø) ⬆️
Source/ModifyRequestNetworkService.swift 100% <100%> (ø) ⬆️
Source/BasicNetworkService.swift 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43d25cc...2bc5e79. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Apr 3, 2019

Codecov Report

Merging #93 into develop will decrease coverage by 0.46%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #93      +/-   ##
===========================================
- Coverage    93.24%   92.78%   -0.47%     
===========================================
  Files           17       17              
  Lines          311      291      -20     
===========================================
- Hits           290      270      -20     
  Misses          21       21
Impacted Files Coverage Δ
Source/ContainerNetworkTask.swift 100% <ø> (+33.33%) ⬆️
Source/NetworkServiceMock.swift 79.31% <ø> (-15.82%) ⬇️
Source/NetworkTaskMock.swift 100% <ø> (ø) ⬆️
Source/URLRequest+Init.swift 94.54% <ø> (ø)
Source/Resource.swift 100% <100%> (ø) ⬆️
Source/Resource+Decodable.swift 100% <100%> (ø) ⬆️
Source/URLRequestConvertible+Modifications.swift 90.9% <100%> (ø) ⬆️
Source/ModifyRequestNetworkService.swift 100% <100%> (ø) ⬆️
Source/BasicNetworkService.swift 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43d25cc...2bc5e79. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 3, 2019

Codecov Report

Merging #93 into develop will increase coverage by 1.13%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #93      +/-   ##
===========================================
+ Coverage    93.59%   94.73%   +1.13%     
===========================================
  Files           18       18              
  Lines          328      304      -24     
===========================================
- Hits           307      288      -19     
+ Misses          21       16       -5
Impacted Files Coverage Δ
Source/ContainerNetworkTask.swift 100% <ø> (+33.33%) ⬆️
Source/NetworkTaskMock.swift 100% <ø> (ø) ⬆️
Source/URLRequest+Init.swift 94.54% <ø> (ø)
Source/Resource.swift 100% <100%> (ø) ⬆️
Source/NetworkServiceMock.swift 96% <100%> (+0.87%) ⬆️
Source/Resource+Decodable.swift 100% <100%> (ø) ⬆️
Source/URLRequestConvertible+Modifications.swift 90.9% <100%> (ø) ⬆️
Source/ModifyRequestNetworkService.swift 100% <100%> (ø) ⬆️
Source/BasicNetworkService.swift 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a89d606...2a48097. Read the comment docs.

//Then
XCTAssertEqual(networkServiceMock.lastRequests, [resource.request, resource.request])
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

//When
networkServiceMock.request(resource, onCompletion: { _ in }, onError: { _ in })
networkServiceMock.request(resource, onCompletion: { _ in }, onError: { _ in })

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)


/// All executed requests.
public private(set) var lastRequests: [URLRequest] = []

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

public var lastRequest: URLRequest? {
return lastRequests.last
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

public var requestCount: Int {
return lastRequests.count
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

@@ -104,10 +97,20 @@ struct NetworkServiceMockCallback {
- seealso: `NetworkService`
*/
public final class NetworkServiceMock: NetworkService {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)

@lightsprint09 lightsprint09 merged commit c5c7c1d into develop Sep 23, 2019
@lightsprint09 lightsprint09 deleted the feature/remove-deprecated-code branch September 6, 2021 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants