-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Nuvei: Add store and stored credentials #5214
Conversation
8ec9503
to
21198c7
Compare
44bb639
to
a3ebb99
Compare
a3ebb99
to
d0d28ed
Compare
9e3f80d
to
8c40bda
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implementation generally looks good, but I'm getting this error Ambiguous payment data. Both UPO and card data present. Only one should be provided!
on the remote tests for SC.
24 tests, 69 assertions, 3 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
87.5% passed
I pulled the creds from 1pass. Maybe something with the account config?
8c40bda
to
5f5bab9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Could you add a changelog entry?
Description ------------------------- This commit add store and stored credentials for Nuvei Gateway [SER-1357](https://spreedly.atlassian.net/browse/SER-1357) Unit test ------------------------- Finished in 0.034584 seconds. 9 tests, 30 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed 260.24 tests/s, 867.45 assertions/s Remote test ------------------------- Finished in 59.543749 seconds. 23 tests, 69 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed 0.39 tests/s, 1.16 assertions/s Rubocop ------------------------- 801 files inspected, no offenses detected
5f5bab9
to
b1dfcbb
Compare
Description
This commit add store and stored credentials for Nuvei Gateway
SER-1357
Unit test
Finished in 0.034584 seconds.
9 tests, 30 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
260.24 tests/s, 867.45 assertions/s
Remote test
Finished in 59.543749 seconds.
23 tests, 69 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
0.39 tests/s, 1.16 assertions/s
Rubocop
801 files inspected, no offenses detected