-
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
Fix unit tests with PHPUnit 9 errors #29244
Fix unit tests with PHPUnit 9 errors #29244
Conversation
Hi @AntonEvers. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, please join the Community Contributions Triage session to discuss the appropriate ticket. 🎥 You can find the recording of the previous Community Contributions Triage on the Magento Youtube Channel ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
68c75b4
to
be6e45d
Compare
Data providers are executed before setUp. In previous phpunit versions $this->formIds[0] would just convert to null, explaining why the logAttempt method was never called. Fixing this issue leads to different expectations of the test result in testAroundExecuteCaptchaIsNotRequired. I have adjusted the necessary. No need to test if this method returns nothing if the return type is already void Fix quote unit tests by explicitely declaring methods that are vital to quote operations. I think it is better to declare these methods explicitely than to depend on DataObject methods Update extensionattribute mock repair partial mock with non-existing method mocking Revert "Fix quote unit tests by explicitely declaring methods that are vital" This reverts commit e29506ce3b353d55cbe64d064e863c4969c3d6cd. fix mock object with non existing method mocks replace partial mocks with non-existing method mocking
be6e45d
to
86000ce
Compare
@magento run all tests |
@ihor-sviziev can I ask for your review on this? Starting from today I saw a lot of failing tests. |
Hi @ihor-sviziev, thank you for the review. |
QA not applicable |
Hi @ihor-sviziev, thank you for the review. |
QA not applicable |
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!
Hi @lbajsarowicz, thank you for the review. |
@magento create issue |
Hi @AntonEvers, thank you for your contribution! |
Description (*)
Most of the failing tests are due to:
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)
Resolved issues: