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

Decidir: Add Cabal card #3322

Merged
merged 1 commit into from
Sep 4, 2019
Merged

Decidir: Add Cabal card #3322

merged 1 commit into from
Sep 4, 2019

Conversation

leila-alderman
Copy link
Contributor

Adds the Cabal card type to the Decidir gateway.

With our current test gateway implementation, it does not seem possible
to test Cabal cards. When attempting to run a remote test with a Cabal
test card number, the error message states that the request contains an
invalid BIN (first 6 digits of the card number). Therefore, the addition
of Cabal cards has not been tested on the Decidir gateway.

CE-94 / CE-100

Unit:
22 tests, 112 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions,
0 notifications
100% passed

Remote:
17 tests, 60 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions,
0 notifications
100% passed

@leila-alderman leila-alderman requested review from a team September 3, 2019 13:54
@jknipp
Copy link
Member

jknipp commented Sep 3, 2019

@leila-alderman Did you reach out to Decidir to see if they could enable Cabal test cards? They may be able to give us an API key to test this functionality.

@jasonxp
Copy link
Contributor

jasonxp commented Sep 3, 2019

@jknipp Good question. Since we already launched Cabal support for a subset of gateways, Production is currently in an inconsistent state in that we have successful Cabal transactions with only some of the gateways that support the card. The current docs do not account for card types not working on all supported gateways. As a result, we decided to go with a strategy of getting support for Cabal released on the remaining gateways as quickly as possible, and following up with the gateways after release to get our test accounts working properly for additional remote tests. Does that seem reasonable?

@jknipp
Copy link
Member

jknipp commented Sep 3, 2019

@jasonxp Makes sense, thanks for the clarification that the decision to skip tests was intentional 👍

Adds the Cabal card type to the Decidir gateway.

With our current test gateway implementation, it does not seem possible
to test Cabal cards. When attempting to run a remote test with a Cabal
test card number, the error message states that the request contains an
invalid BIN (first 6 digits of the card number). Therefore, the addition
of Cabal cards has not been tested on the Decidir gateway.

CE-94 / CE-100

Unit:
22 tests, 112 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions,
0 notifications
100% passed

Remote:
17 tests, 60 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions,
0 notifications
100% passed
@leila-alderman leila-alderman force-pushed the CE-100_decidir_add_cabal branch from ee2c075 to 487399b Compare September 4, 2019 19:48
@leila-alderman leila-alderman deleted the CE-100_decidir_add_cabal branch September 4, 2019 19:49
@leila-alderman leila-alderman merged commit 487399b into master Sep 4, 2019
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