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

Fix failing AuthorizeRequestTest #5

Closed
wants to merge 7 commits into from
Closed

Fix failing AuthorizeRequestTest #5

wants to merge 7 commits into from

Conversation

sa3dany
Copy link

@sa3dany sa3dany commented Oct 16, 2020

This fixes the failing tests for AuthorizeRequestTest by adding the missing payment_type param.

This pull request also adds the following:

  • Adds description field to composer.json
  • Adds script to run tests in composer.json (allows running tests using composer test)
  • Adds .gitignore with /vendor

sa3dany and others added 7 commits October 16, 2020 21:11
This allows simply running `composer test` to run the unit tests instead of  `./vendor/bin/phpunit tests`.
php-cs-fixer (PHP code style fixer) is added as a dev dependency.

Two composer.json scripts are added:
  - **fix**: Runs `php-cs-fixer` fix command to auto-fix all source
    files.
  - **lint**: Runs `php-cs-fixer` in dry run mode to check for code
    styel violations.

The default style rules for `php-cs-fixer` are used which are PSR1 and PSR2.
This pull request was closed.
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

2 participants