Skip to content

ISSUE 60- increased code coverage for serializationinterceptor #65

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.

Sign up for GitHub

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

Jump to bottom
Merged
diana-derose merged 1 commit into from
Oct 19, 2019
Merged

ISSUE 60- increased code coverage for serializationinterceptor #65

diana-derose merged 1 commit into from
Oct 19, 2019

Conversation

@tarungulati1988
Copy link

@tarungulati1988 tarungulati1988 commented Oct 18, 2019 •
edited
Loading

Added unit tests for SerializeInterceptor.java for Issue#60

  • execute()
  • getSerializationRequestFormat()

@coveralls
Copy link

coveralls commented Oct 18, 2019

Coverage Status

Coverage increased (+0.4%) to 35.938% when pulling e31d891 on tarungulati1988:moreTests into 9100e92 on intuit:develop.

@tarungulati1988 tarungulati1988 changed the title ISSUE 60 - increased code coverage for serializationinterceptor ISSUE [60](https://github.com/intuit/QuickBooks-V3-Java-SDK/issues/60) - increased code coverage for serializationinterceptor Oct 19, 2019
@tarungulati1988 tarungulati1988 changed the title ISSUE [60](https://github.com/intuit/QuickBooks-V3-Java-SDK/issues/60) - increased code coverage for serializationinterceptor ISSUE https://github.com/intuit/QuickBooks-V3-Java-SDK/issues/60- increased code coverage for serializationinterceptor Oct 19, 2019
@tarungulati1988 tarungulati1988 changed the title ISSUE https://github.com/intuit/QuickBooks-V3-Java-SDK/issues/60- increased code coverage for serializationinterceptor ISSUE 60- increased code coverage for serializationinterceptor Oct 19, 2019
@diana-derose diana-derose merged commit cd7c214 into intuit:develop Oct 19, 2019
@diana-derose
Copy link
Collaborator

diana-derose commented Oct 19, 2019

Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diana-derose diana-derose diana-derose approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tarungulati1988 @coveralls @diana-derose