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
Resolved Issue #24785 #24800
Resolved Issue #24785 #24800
Conversation
Hi @drpayyne. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
@magento give me test instance |
Hi @drpayyne. Thank you for your request. I'm working on Magento instance for you |
Hi @drpayyne, here is your new Magento instance. |
16239b6
to
eaa54db
Compare
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.
Hi @dryadav. Thanks for collaboration. Due to Magento Definition of Done all code must be covered by tests. Please cover your fix by automated tests.
@VladimirZaets Thank you. Although, I'm confused a bit. What test should I write? I have only added a method in a jQuery widget. 🤔 |
@drpayyne I think in this case we can write a functional test and check what is displayed, or at least JS unit test, that will check the new method that you created. If you need documentation how to do it, or some additional information feels free to ask. |
@VladimirZaets yes, can I have some references or documentations? I've written only PHPUnit tests. Thanks a lot! |
@drpayyne please, refer to Magento devdocs: |
@sidolov @VladimirZaets Hello, I added an MFTF test to test the changing of the label. Please review my code when you can. And thanks for helping me with the documentation reference. Also, my Static Tests build is failing but says no errors in the Allure dashboard. |
Hi @sidolov, thank you for the review. |
Changed let to var. Added 'use strict';
Remove empty line
Hi @drpayyne, thank you for your contribution! |
Description
Resolved issue #24785
Added a JS method to handle the state of checked links in a downloadable product. The state is now managed and reflected in the label. Also added MFTF test to test changing of label.
Unselect all
is displayed only when all links are checkedSelect all
is displayed otherwiseFixed Issues
Manual testing scenarios
Contribution checklist