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
add validation on complete order #12658
add validation on complete order #12658
Conversation
AdamKasp
commented
May 20, 2021
Q | A |
---|---|
Branch? | master |
New feature? | yes |
License | MIT |
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibilityValidator.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibilityValidator.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibility.php
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Resources/config/suites/api/inventory/checkout_inventory.yaml
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Resources/config/suites/api/inventory/checkout_inventory.yaml
Outdated
Show resolved
Hide resolved
src/Sylius/Behat/Resources/config/suites/api/inventory/checkout_inventory.yaml
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibility.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibilityValidator.php
Outdated
Show resolved
Hide resolved
b2ed565
to
de26201
Compare
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibilityValidator.php
Outdated
Show resolved
Hide resolved
de26201
to
e476f39
Compare
e476f39
to
2e27e12
Compare
Scenario: Placing an order with products that have sufficient quantity | ||
Given I have added 3 products "Iron Maiden T-Shirt" to the cart | ||
And I have proceeded selecting "Offline" payment method | ||
And I proceed through checkout process |
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.
And I proceed through checkout process | |
When I proceed through checkout process |
but why did you change that step?
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.
I had a problem with addresses, in behats they were empty, and I get SM error, with this step, the scenario is still valid and checkout works as I expect.
src/Sylius/Bundle/ApiBundle/Validator/Constraints/OrderProductInStockEligibility.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
.../Bundle/ApiBundle/spec/Validator/Constraints/OrderProductInStockEligibilityValidatorSpec.php
Outdated
Show resolved
Hide resolved
8e79960
to
6725511
Compare
6725511
to
d5302d6
Compare
Thanks, Adam! 🎉 |