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

Allow options in objectToDocumentXML when calling a method #1096

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ndeniche
Copy link

@ndeniche ndeniche commented Nov 8, 2019

Allows overriding the InputMessage "Request" element for custom elements based on args param

  • Options is optional field for objectToDocumentXML function
  • Replicates _xml parameter functionality (sending a custom XML structure as a literal) with JSON arguments.

Rubén Alberto Reyes and others added 5 commits November 7, 2019 17:32
### Allows overriding the InputMessage "Request" element for custom elements based on args param

* Options is optional field for objectToDocumentXML function
* Replicates `_xml` parameter functionality (sending a custom XML structure as a literal) with JSON arguments.
* Options is optional field for objectToDocumentXML function
* Replicates `_xml` parameter functionality (sending a custom XML structure as a literal) with JSON arguments.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 94.529% when pulling 130c204 on ndeniche:ndeniche_override-base-element into af93a6f on vpulim:master.

@jsdevel
Copy link
Collaborator

jsdevel commented Nov 8, 2019

@ndeniche please add more tests to cover all scenarios.

@jsdevel
Copy link
Collaborator

jsdevel commented Nov 14, 2019

i'm closing this in 4 days if there's no more action

@ndeniche
Copy link
Author

All of my new code is covered. Should I improve coverage by adding tests to other features?

@jsdevel
Copy link
Collaborator

jsdevel commented Nov 18, 2019

@ndeniche if coverage has decreased, then not all the added code has been covered.

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.

3 participants