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

Mundipagg: Append errors to message response field #3353

Merged

Conversation

jasonxp
Copy link
Contributor

@jasonxp jasonxp commented Sep 16, 2019

For purchase, authorize, and store, extract any error messages from the gateway response and append them to the "message" response feld. Mundipagg can return errors at either the top level of the response object, or under the last_transaction gateway_response key.

@jasonxp jasonxp force-pushed the ce-112-mundipagg-error-messages branch 2 times, most recently from b933144 to eb475ee Compare September 16, 2019 19:14
@@ -38,6 +38,8 @@ def setup
billing_address: address,
description: 'Store Purchase'
}

@gateway_response_error = 'Esta loja n??o possui um meio de pagamento configurado para a bandeira VR'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I haven't been able to reproduce one of these errors, as they likely require a configuration discrepancy. There is thus no corresponding remote test. However, the code and test were written based on examination of a real such error encountered by a user.

@jasonxp jasonxp requested review from therufs and a team September 16, 2019 19:18
@jasonxp jasonxp force-pushed the ce-112-mundipagg-error-messages branch from eb475ee to 438afc5 Compare September 16, 2019 19:38
Copy link
Contributor

@curiousepic curiousepic left a comment

Choose a reason for hiding this comment

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

Nice, LGTM 👍

For purchase, authorize, and store, extract any error messages from the
gateway response and append them to the "message" response feld. Mundipagg
can return errors at either the top level of the response object, or under
the last_transaction gateway_response key.

CE-114

Unit:
4268 tests, 70604 assertions, 0 failures, 0 errors, 0 pendings, 2 omissions, 0 notifications
100% passed

Remote:
37 tests, 93 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@jasonxp jasonxp force-pushed the ce-112-mundipagg-error-messages branch from 438afc5 to b1e0807 Compare September 18, 2019 20:34
@jasonxp jasonxp merged commit b1e0807 into activemerchant:master Sep 18, 2019
@jasonxp jasonxp deleted the ce-112-mundipagg-error-messages branch September 18, 2019 20:46
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.

2 participants