search

OCA / e-commerce

Odoo E-Commerce server automation addons
GNU Affero General Public License v3.0
170 stars 496 forks source link

[16.0][FIX] website_sale_vat_required: fix test #940

Closed pilarvargas-tecnativa closed 5 months ago

pilarvargas-tecnativa commented 5 months ago

The tests were using a demo data product so if at any time this product does not exist the tests start to fail. To avoid this, a new product is created and used for testing. Also to locate the checkout button it is better to locate it by its container element as the classes can change depending on the configuration and modules used in the instance and it can be a problem to locate it.

Example screenshot where the checkout button has changed its classes and is not found:

image

cc @Tecnativa

@CarlosRoca13 @chienandalu please review

OCA-git-bot commented 5 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-940-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

Congratulations, your PR was merged at a70b1f48fa94cd258e752f19599e00e592094735. Thanks a lot for contributing to OCA. ❤️