-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
magento/magento2#27500 - Replace deprecated constructs from Payment P… #27596
magento/magento2#27500 - Replace deprecated constructs from Payment P… #27596
Conversation
Hi @cristiano-pacheco. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
…o phpunit8/module-payment
Hi @lbajsarowicz could you review this PR? please. I made the changes for the tests of the Magento_Payment module, and tests of other modules are broken and therefore this PR did not pass. How can I solve this? |
Please take a look at Static Tests. I will take care about CustomerBalance module. |
@engcom-Foxtrot could you please taske a look at the failing Unit tests |
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.
@engcom-Foxtrot could you please taske a look at the failing Unit tests
@magento run all tests |
@cristiano-pacheco Could you please resolve the conflicts |
@cristiano-pacheco I am closing it since it was implemented in another PR. Thank you for your efforts! |
Hi @cristiano-pacheco, thank you for your contribution! |
…HPUnit.
This PR is about the issue #27500 on item app/code/Magento/Payment
Description (*)
Changes were made to the payment module to be compatible with PHPunit in version 8.
Related Pull Requests
Contribution checklist (*)