Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Add XMLParser/XMLDocument serializer tests #3753

Merged
merged 3 commits into from Nov 23, 2017

Conversation

skyline75489
Copy link
Contributor

Inspired by 290ec35, this adds tests for AFXMLParserResponseSerializer and AFXMLDocumentResponseSerializer. This also fixes an error in AFXMLDocumentResponseSerializer documentation.

@@ -166,7 +166,7 @@ NS_ASSUME_NONNULL_BEGIN
- (instancetype)init;

/**
Input and output options specifically intended for `NSXMLDocument` objects. For possible values, see the `NSJSONSerialization` documentation section "NSJSONReadingOptions". `0` by default.
Input and output options specifically intended for `NSXMLDocument` objects. For possible values, see the `NSXMLDocument` documentation section "Input and Output Options". `0` by default.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This looks like a copy-and-paste error for me.

@codecov-io
Copy link

codecov-io commented Oct 15, 2016

Codecov Report

Merging #3753 into master will increase coverage by 0.27%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3753      +/-   ##
==========================================
+ Coverage   86.95%   87.23%   +0.27%     
==========================================
  Files          45       46       +1     
  Lines        5505     5539      +34     
  Branches      439      439              
==========================================
+ Hits         4787     4832      +45     
+ Misses        715      704      -11     
  Partials        3        3
Impacted Files Coverage Δ
AFNetworking/AFURLResponseSerialization.h 90% <ø> (ø) ⬆️
Tests/Tests/AFXMLParserResponseSerializerTests.m 100% <100%> (ø)
AFNetworking/AFURLResponseSerialization.m 88.75% <0%> (+3.25%) ⬆️

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 ec5c955...158dc56. Read the comment docs.

Copy link
Contributor

@SlaunchaMan SlaunchaMan left a comment

Choose a reason for hiding this comment

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

Looks good! More tests are always appreciated!

@skyline75489
Copy link
Contributor Author

@SlaunchaMan Would you mind merging this PR & #3750 which were drafted a long time ago

@SlaunchaMan SlaunchaMan merged commit 9d5a894 into AFNetworking:master Nov 23, 2017
@SlaunchaMan SlaunchaMan added this to the 3.2.0 milestone Dec 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants