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

Worldpay: Support Unknown Card Type #3213

Conversation

tanyajajodia
Copy link
Contributor

No description provided.

@tanyajajodia
Copy link
Contributor Author

@dtykocki for your initial review and feedback

@tanyajajodia tanyajajodia force-pushed the worldpay-add-unknown-cardtype-support branch from 411209a to cf44a1e Compare May 10, 2019 16:03
@tanyajajodia tanyajajodia marked this pull request as ready for review May 10, 2019 16:04
@tanyajajodia tanyajajodia force-pushed the worldpay-add-unknown-cardtype-support branch from cf44a1e to fc58541 Compare May 10, 2019 16:06
Adds support for unknown card types for Worldpay which resolves
exception errors. The expected result for such unknown cards is that the
transactions fail with the error message “XML failed validation:
Invalid payment details : Card number not recognised:”

Closes activemerchant#3213

Remote:
40 tests, 171 assertions, 2 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
95% passed
The 2 failures listed above existed prior to this change.

Unit:
49 tests, 268 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@tanyajajodia tanyajajodia force-pushed the worldpay-add-unknown-cardtype-support branch from fc58541 to c2e1c65 Compare May 10, 2019 20:44
Copy link

@dtykocki dtykocki left a comment

Choose a reason for hiding this comment

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

👍

@dtykocki dtykocki closed this in a30594b May 13, 2019
whitby3001 pushed a commit to whitby3001/active_merchant that referenced this pull request Sep 3, 2019
Adds support for unknown card types for Worldpay which resolves
exception errors. The expected result for such unknown cards is that the
transactions fail with the error message “XML failed validation:
Invalid payment details : Card number not recognised:”

Closes activemerchant#3213

Remote:
40 tests, 171 assertions, 2 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
95% passed
The 2 failures listed above existed prior to this change.

Unit:
49 tests, 268 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
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