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

Fix #546 CapturingInterceptor will now buffer response #547

Merged
merged 1 commit into from Jan 31, 2017

Conversation

@daliboz
Copy link
Contributor

@daliboz daliboz commented Jan 27, 2017

Added tests and updated CapturingInterceptor to buffer the response if the entity was not repeatable.

Fixes #546

…is not repeatable.
@coveralls
Copy link

@coveralls coveralls commented Jan 28, 2017

Coverage Status

Coverage increased (+0.01%) to 81.551% when pulling c1e6f82 on daliboz:master into 97f1e55 on jamesagnew:master.

@jamesagnew
Copy link
Owner

@jamesagnew jamesagnew commented Jan 31, 2017

This is awesome! Thanks for the PR.

I kinda can't believe that's all it took to fix that... :)

@jamesagnew jamesagnew merged commit ea1ebb2 into jamesagnew:master Jan 31, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.01%) to 81.551%
Details
jamesagnew added a commit that referenced this pull request Jan 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.