issue-#25675 Added fix for #25675 issue to the 2.4 Magento version

[siraybit]

Description
This pull request should fix issue #25675 in Magento 2.4

Fixed Issues
#25675: Unable add product to cart in Magento 2.3.3 backend when stock quantity is 1 - "The requested qty is not available"
Manual testing scenarios
Described here #25675

[m2-assistant]

Hi @molneek. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento give me test instance - deploy test instance based on PR changes
• @magento give me 2.4-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

[siraybit]

@dmytro-ch
I see that some tests are failed. E.g ReorderUsingVaultBraintreeTestVariation1_0, but I don't see the place where my code fix or my unit test influence on these test fails. Could you please advise, how to prevent failing the issues and finally add this of the issue to Magento?

[rogyar]

Hi @molneek. The ReorderUsingVaultBraintreeTestVariation1_0 is a fluky test. It will be fixed soon after merging a corresponding fix into mainline. So, don't bother with this failing test, it's not related to your change somehow

[siraybit]

Hi @rogyar. So in this case, does it mean that all these tests "Functional Tests B2B, Functional Tests CE, Functional Tests EE and Static Tests" are not related to my changes?
Do I need manually rerun check when these are fixed, or this happen automatically?

[rogyar]

@magento run all tests

[siraybit]

Hello @rogyar, seems like all tests passed

[rogyar]

Just a hint for the future. If we don't need to share value between different methods, there's no need to create an additional class property. You may use just local variables inside of the methods.

So $this->itemQtyTestValue will be simply $itemQtyTestValue inside of every method (test) of the current class

[siraybit]

@rogyar Thank you, I agree, I had some ideas about using it, but then changed my mind and just used this as this is already created

[magento-engcom-team]

Hi @rogyar, thank you for the review.
ENGCOM-6910 has been created to process this Pull Request
✳️ @rogyar, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[engcom-Bravo]

✔️ QA Passed

An order could be created and proceed without any errors if the product quantity is 1

[m2-assistant]

Hi @molneek, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.