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

Shift4_v2: Inherit securionPay API to enable Shift4v2 #4860

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

javierpedrozaing
Copy link
Collaborator

@javierpedrozaing javierpedrozaing commented Aug 17, 2023

SER-653
SER-654
SER-655
SER-661
SER-662
SER-663

Shift4 purchased Securion Pay and is now using their API, that's why
this commit enables a new shift4_v2 gateway

WIP: Pending to include general credit support

Unit test

Finished in 0.150258 seconds.
34 tests, 191 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
226.28 tests/s, 1271.15 assertions/s

Remote test

Finished in 27.855679 seconds.
33 tests, 116 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Rubocop

760 files inspected, no offenses detected

Copy link
Contributor

@sinourain sinourain left a comment

Choose a reason for hiding this comment

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

good test coverage, left some comments for your consideration

lib/active_merchant/billing/gateways/shift4_v2.rb Outdated Show resolved Hide resolved
test/remote/gateways/remote_shift4_v2_test.rb Outdated Show resolved Hide resolved
test/remote/gateways/remote_shift4_v2_test.rb Outdated Show resolved Hide resolved
test/fixtures.yml Outdated Show resolved Hide resolved
test/unit/gateways/shift4_v2_test.rb Outdated Show resolved Hide resolved
@Heavyblade Heavyblade force-pushed the SER-653_shift4_v2 branch 4 times, most recently from 53686bb to d6c61de Compare August 22, 2023 16:28
@Heavyblade Heavyblade marked this pull request as ready for review August 22, 2023 16:28
@Heavyblade Heavyblade force-pushed the SER-653_shift4_v2 branch 2 times, most recently from 58a0afc to f2aecbe Compare August 22, 2023 16:52
Copy link
Contributor

@naashton naashton left a comment

Choose a reason for hiding this comment

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

This all looks good to me! Can we stub an entry in the fixtures file for this gateway and add a CHANGELOG entry?

test/remote/gateways/remote_shift4_v2_test.rb Show resolved Hide resolved
@Heavyblade Heavyblade force-pushed the SER-653_shift4_v2 branch 4 times, most recently from 25a1251 to 4a17fe9 Compare August 23, 2023 15:41
Description
-------------------------
[SER-653](https://spreedly.atlassian.net/browse/SER-653)
[SER-654](https://spreedly.atlassian.net/browse/SER-654)
[SER-655](https://spreedly.atlassian.net/browse/SER-655)
[SER-662](https://spreedly.atlassian.net/browse/SER-662)

Shift4 purchased Securion Pay and is now using their API, that's why
this commit enable a new shift4_v2 gateway

Unit test
-------------------------
Finished in 0.150258 seconds.
34 tests, 191 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote test
-------------------------
Finished in 28.137188 seconds.
30 tests, 103 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Rubocop
-------------------------
760 files inspected, no offenses detected
@Heavyblade
Copy link
Collaborator

Latest changes applied @naashton.

@naashton naashton merged commit 461ff34 into master Aug 29, 2023
5 checks 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.

None yet

6 participants