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

Elavon: Upgrade gateway integration [WIP] #3694

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leila-alderman
Copy link
Contributor

This is the start of the work necessary to upgrade the Elavon gateway
integration to move from the deprecated process.do endpoint to the
more current XML endpoint processxml.do.

This WIP commit includes the major updates to the gateway structure to
use XML and to achieve a successful remote purchase request.

Many of the tests still need to be updated, including updating all of
the stubbed responses in the unit tests.

CE-704

ruby -Itest test/remote/gateways/remote_elavon_test.rb -n test_successful_purchase

1 tests, 5 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications
100% passed

This is the start of the work necessary to upgrade the Elavon gateway
integration to move from the deprecated `process.do` endpoint to the
more current XML endpoint `processxml.do`.

This WIP commit includes the major updates to the gateway structure to
use XML and to achieve a successful remote purchase request.

Many of the tests still need to be updated, including updating all of
the stubbed responses in the unit tests.

CE-704

`ruby -Itest test/remote/gateways/remote_elavon_test.rb -n
test_successful_purchase`

1 tests, 5 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications
100% passed
@therufs therufs self-assigned this Jul 28, 2020
@curiousepic
Copy link
Contributor

Marking this "of interest" before a cleanup of stale PRs

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