[CatalogUrlRewrite] Covering the CategoryProcessUrlRewriteMovingObserver by Unit Test#18004
Conversation
|
Hi @eduard13. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
|
@rogyar could you please have a look at these failed tests, maybe you can suggest me something here ? As I don't get what's the problem here. |
|
@eduard13 the number of class dependencies ideally should be less than 13. But for unit tests it's not a critical point |
Suppress PHPMD warning for test
|
Hi @eduard13. Thank you for your contribution. Please, consider to port this solution to 2.3 release line. |
1 similar comment
|
Hi @eduard13. Thank you for your contribution. Please, consider to port this solution to 2.3 release line. |
Description
This PR adds missing unit tests for CategoryProcessUrlRewriteMovingObserver class:
\Magento\CatalogUrlRewrite\Observer\CategoryProcessUrlRewriteMovingObserverFixed Issues (if relevant)
N/A
Manual testing scenarios
N/A
Contribution checklist