[MIG] website_sale_secondary_unit: Migration to version 18.0

TT54395

Author: pilarvargas-tecnativa

website_sale_secondary_unit/README.rst

@@ -17,13 +17,13 @@ Website Sale Secondary Unit
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fe--commerce-lightgray.png?logo=github
-    :target: https://github.com/OCA/e-commerce/tree/17.0/website_sale_secondary_unit
+    :target: https://github.com/OCA/e-commerce/tree/18.0/website_sale_secondary_unit
     :alt: OCA/e-commerce
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/e-commerce-17-0/e-commerce-17-0-website_sale_secondary_unit
+    :target: https://translation.odoo-community.org/projects/e-commerce-18-0/e-commerce-18-0-website_sale_secondary_unit
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/e-commerce&target_branch=17.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/e-commerce&target_branch=18.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -47,23 +47,23 @@ Usage
 
 To use this module you need to:
 
-- Go to *'Website > Products > Products'*.
-- Select a template.
-- Set the secondary units that you need.
-- Go to Website Shop and buy this product, you will see a selectable
-  option with all secondary units defined in the product and visible in
-  website.
-- If you do not want to sell in a base product unit and only allow sell
-  in a secondary unit you can disable the option *'Allow to sell in unit
-  of measure'* in a product sale tab.
+-  Go to *'Website > Products > Products'*.
+-  Select a template.
+-  Set the secondary units that you need.
+-  Go to Website Shop and buy this product, you will see a selectable
+   option with all secondary units defined in the product and visible in
+   website.
+-  If you do not want to sell in a base product unit and only allow sell
+   in a secondary unit you can disable the option *'Allow to sell in
+   unit of measure'* in a product sale tab.
 
 Bug Tracker
 ===========
 
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/e-commerce/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/e-commerce/issues/new?body=module:%20website_sale_secondary_unit%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/e-commerce/issues/new?body=module:%20website_sale_secondary_unit%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -78,12 +78,12 @@ Authors
 Contributors
 ------------
 
-- `Tecnativa <https://www.tecnativa.com>`__:
+-  `Tecnativa <https://www.tecnativa.com>`__:
 
-  - Sergio Teruel
-  - Carlos Roca
-  - Pilar Vargas
-  - Carlos Lopez
+   -  Sergio Teruel
+   -  Carlos Roca
+   -  Pilar Vargas
+   -  Carlos Lopez
 
 Maintainers
 -----------
@@ -98,6 +98,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/e-commerce <https://github.com/OCA/e-commerce/tree/17.0/website_sale_secondary_unit>`_ project on GitHub.
+This module is part of the `OCA/e-commerce <https://github.com/OCA/e-commerce/tree/18.0/website_sale_secondary_unit>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

website_sale_secondary_unit/__manifest__.py

@@ -3,7 +3,7 @@
 {
     "name": "Website Sale Secondary Unit",
     "summary": "Allow manage secondary units in website shop",
-    "version": "17.0.1.0.0",
+    "version": "18.0.1.0.0",
     "development_status": "Beta",
     "category": "Website",
     "website": "https://github.com/OCA/e-commerce",

website_sale_secondary_unit/static/description/index.html

@@ -369,7 +369,7 @@ <h1 class="title">Website Sale Secondary Unit</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:4c9fbdc1746932986528b0f08a484c172d0bfafed5fc111c1342a3b645ec4a11
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/e-commerce/tree/17.0/website_sale_secondary_unit"><img alt="OCA/e-commerce" src="https://img.shields.io/badge/github-OCA%2Fe--commerce-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/e-commerce-17-0/e-commerce-17-0-website_sale_secondary_unit"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/e-commerce&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/e-commerce/tree/18.0/website_sale_secondary_unit"><img alt="OCA/e-commerce" src="https://img.shields.io/badge/github-OCA%2Fe--commerce-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/e-commerce-18-0/e-commerce-18-0-website_sale_secondary_unit"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/e-commerce&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module extends the functionality of saleorder_secondary_unit module
 to allow sell products in online store in secondary units defined.</p>
 <p><strong>Table of contents</strong></p>
@@ -402,16 +402,16 @@ <h1><a class="toc-backref" href="#toc-entry-2">Usage</a></h1>
 option with all secondary units defined in the product and visible in
 website.</li>
 <li>If you do not want to sell in a base product unit and only allow sell
-in a secondary unit you can disable the option <em>‘Allow to sell in unit
-of measure’</em> in a product sale tab.</li>
+in a secondary unit you can disable the option <em>‘Allow to sell in
+unit of measure’</em> in a product sale tab.</li>
 </ul>
 </div>
 <div class="section" id="bug-tracker">
 <h1><a class="toc-backref" href="#toc-entry-3">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/e-commerce/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/e-commerce/issues/new?body=module:%20website_sale_secondary_unit%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/e-commerce/issues/new?body=module:%20website_sale_secondary_unit%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -443,7 +443,7 @@ <h2><a class="toc-backref" href="#toc-entry-7">Maintainers</a></h2>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/e-commerce/tree/17.0/website_sale_secondary_unit">OCA/e-commerce</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/e-commerce/tree/18.0/website_sale_secondary_unit">OCA/e-commerce</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

website_sale_secondary_unit/static/tests/tours/website_sale_secondary_unit_tour.esm.js

@@ -1,4 +1,3 @@
-/** @odoo-module */
 /* Copyright 2019 Sergio Teruel
  * License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). */
 
@@ -9,47 +8,60 @@ registry.category("web_tour.tours").add("website_sale_secondary_unit", {
     steps: () => [
         {
             trigger: "a:contains('Test product')",
+            run: "click",
         },
         {
             trigger: "#secondary_uom",
-            run: "text(Box 5 Units)",
+            run: "selectByLabel Box 5 Units",
         },
         {
             trigger: "#add_to_cart",
-            extra_trigger:
-                ".js_product:has(input[name='add_qty']:propValueContains(5)):has(.price_uom)",
+            run: "click",
         },
         {
             trigger: "a[href='/shop/cart']",
+            run: "click",
+        },
+        {
+            trigger: "span:contains(Box 5 Units)",
         },
         {
             trigger: "a[href='/shop']",
-            extra_trigger: "span:contains(Box 5 Units)",
+            run: "click",
         },
         {
             trigger: "a:contains('Test product')",
+            run: "click",
         },
         {
             trigger: "#add_to_cart",
-            extra_trigger:
-                ".js_product:has(input[name='add_qty']:propValueContains(1))",
+            run: "click",
         },
         {
             trigger: "a[href='/shop/cart']",
+            run: "click",
+        },
+        {
+            trigger: "span:contains(Units)",
+        },
+        {
+            trigger: "a[name='website_sale_main_button']",
+            run: "click",
         },
         {
-            trigger: "a[href='/shop/checkout?express=1']",
-            extra_trigger: "span:containsExact(Units)",
+            trigger: "div[id='o_wsale_total_accordion_item'] button.accordion-button",
+            run: "click",
         },
         {
-            trigger: "div[id='o_wsale_total_accordion'] button.accordion-button",
+            trigger: "h6[name='secondary_uom_qty'] span:contains(Box 5)",
+            run: "click",
         },
         {
-            trigger: "h6[name='secondary_uom_qty'] span:containsExact(Box 5)",
+            trigger: "table:has(span:contains(Box 5)):has(span:contains(Units))",
         },
         {
             trigger: "a[href='/shop']",
-            extra_trigger: "table:has(span:contains(Box 5)):has(span:contains(Units))",
+            run: "click",
         },
     ],
 });

website_sale_secondary_unit/tests/test_website_sale_secondary_unit.py

@@ -1,10 +1,11 @@
 # Copyright 2019 Tecnativa - Sergio Teruel
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
-from odoo.tests.common import HttpCase
+from odoo.tests.common import HttpCase, tagged
 
 from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
 
 
+@tagged("post_install", "-at_install")
 class WebsiteSaleSecondaryUnitHttpCase(HttpCase):
     @classmethod
     def setUpClass(cls):
@@ -44,6 +45,8 @@ def setUpClass(cls):
         # Add group "Manage Multiple Units of Measure" to admin
         admin = cls.env.ref("base.user_admin")
         admin.groups_id |= cls.env.ref("uom.group_uom")
+        # Force VAT to avoid error in the module that makes it required.
+        admin.partner_id.vat = "TEST12345678"
 
     def test_ui_website(self):
         """Test frontend tour."""

website_sale_secondary_unit/views/product_secondary_unit_views.xml

@@ -10,7 +10,7 @@
             ref="product_secondary_unit.product_template_form_view"
         />
         <field name="arch" type="xml">
-            <xpath expr="//tree/field[@name='factor']" position="after">
+            <xpath expr="//list/field[@name='factor']" position="after">
                 <field name="website_published" />
             </xpath>
         </field>