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
Feature/update phpunit 8 to 9 #536
Feature/update phpunit 8 to 9 #536
Conversation
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.
Thanks for the massive effort here, just left a few small comments
Co-authored-by: Pierre du Plessis <pierre@pcservice.co.za>
Co-authored-by: Pierre du Plessis <pierre@pcservice.co.za>
I have to thank you for maintaining payum. |
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.
Thanks for the massive effort here @Chris53897
All Changes are related to php 7.4 and phpunit 9 and to get tests running.
This PR is related to #535
Serveral changes are needed in composer.json
"league/uri-schemes": "^1.1", => Otherwise it would not install on php 8.x
"payum/payum": "^1.6.2", => Otherwise --prefer-lowest Test will not be an installable set
"symfony/phpunit-bridge": "6.0", => Only one version is needed for require-dev
"psr/log": "^1 || ^2", is more a temporary fix. I guesss the real problem is in payum itself. Payum/Payum#926
"symfony/form": "^4.4.20 || ^5.0", => Required for the test (Annotated Validations) CreditCardTypeTest->shouldRequireExpireAtInFuture()
Update: All Tests are migrated to phpUnit 9.
Symfony4 needs some extra config to work with Symfony4, 5 and 6 in the near future.
Symfony6 will need extra Work, as there are deprecated functions.
Should be handled in an different PR, after support of Symfony6 is added.