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

Add test gateway #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

sean-e-dietrich
Copy link

I wanted to have the ability for the test gateway to be built in so that when clients are trying to test out the system and how it is suppose to work they know exactly how the process should go, without needing to use a live credit card.

@mfauveau
Copy link
Owner

mfauveau commented Jan 25, 2017

Hi Sean,

Sorry for the delay. Can you make sure the build pass before I can merge?

See:

FILE: /home/travis/build/mfauveau/omnipay-nmi/src/Message/AbstractRequest.php
--------------------------------------------------------------------------------
FOUND 7 ERROR(S) AFFECTING 5 LINE(S)
--------------------------------------------------------------------------------
 337 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
 342 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
 342 | ERROR | No space found after comma in function call
 347 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
 352 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6
 352 | ERROR | No space found after comma in function call
 357 | ERROR | Line indented incorrectly; expected at least 8 spaces, found 6

@sean-e-dietrich
Copy link
Author

sean-e-dietrich commented Jan 25, 2017 via email

@mfauveau
Copy link
Owner

Looks like an issue with test cases: https://travis-ci.org/mfauveau/omnipay-nmi/jobs/195323320

@dcaswel
Copy link

dcaswel commented Feb 27, 2017

Just as an FYI, from the documentation, it seems like you wouldn't really need this. If you want to test things, just use the demo account credentials instead of your regular account.

Sean Dietrich and others added 4 commits May 15, 2017 16:20
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.

4 participants