-
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
MFTF: AdminAddingNewOptionsWithImagesAndPricesToConfigurableProductTest is failing in CI process #26610
Comments
Thanks for opening this issue! |
Hi @lbajsarowicz. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @lbajsarowicz do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Thanks for opening this issue! |
Hi @lbajsarowicz. Thank you for your report. The fix will be available with the upcoming 2.4.0 release. |
Summary (*)
When working with #26454 - I faced the failing test
AdminAddingNewOptionsWithImagesAndPricesToConfigurableProductTest
. Test outcome was completely unclear:Also, ScreenShot wasn't helpful at all:
I started to investigate with adding full Exception message to the Notification:
As you see - it's trying to remove Product with ID = 5. But (!) the test hasn't created any Product with ID = 5:
So I got deeper - why it's actually trying to remove product ID = 5 and noticed that it's actually filtering product to remove:
... by product name that is empty!
That bug is caused by invalid handler introduced by Magento QA team and exists on other tests too - but by coincidence, the it's not overlapping with other test operating on the same data:
Examples (*)
Proposed solution
C'mon, fix it :-)
The text was updated successfully, but these errors were encountered: