search

Payum / PayumBundle

Payum offers everything you need to work with payments. From simplest use cases to very advanced ones.
https://payum.gitbook.io/payum
MIT License
569 stars 142 forks source link

Feature/update phpunit 8 to 9 #536

Closed Chris53897 closed 2 years ago

Chris53897 commented 2 years ago

All Changes are related to php 7.4 and phpunit 9 and to get tests running. This PR is related to https://github.com/Payum/PayumBundle/pull/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. https://github.com/Payum/Payum/pull/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.

Chris53897 commented 2 years ago

I have to thank you for maintaining payum. Thanks for the fast response. I made the suggested changes. Can you please review again.