From 5db5f8e98119d5a8094e73099a23b12e27f07513 Mon Sep 17 00:00:00 2001
From: Romain Derie <rde@odoo.com>
Date: Fri, 15 Dec 2017 16:05:02 +0100
Subject: [PATCH 1/3] [IMP] sale_coupon, website_sale_coupon: add tests

Commit a42c27f99e0061b7c4f recently fixed some bugs on sale_coupon.
This commit add some tests to be sure theses bugs are fixed and won't appear
again.

This closes #1744
---
 static/src/js/website_sale_coupon.test.js | 120 ++++++++++++++++++++++
 tests/__init__.py                         |   1 +
 tests/test_shop_sale_coupon.py            |  11 ++
 views/website_sale_templates.xml          |  10 ++
 4 files changed, 142 insertions(+)
 create mode 100644 static/src/js/website_sale_coupon.test.js
 create mode 100644 tests/__init__.py
 create mode 100644 tests/test_shop_sale_coupon.py

diff --git a/static/src/js/website_sale_coupon.test.js b/static/src/js/website_sale_coupon.test.js
new file mode 100644
index 00000000000..1ce783c54c0
--- /dev/null
+++ b/static/src/js/website_sale_coupon.test.js
@@ -0,0 +1,120 @@
+odoo.define('website_sale_coupon.test', function (require) {
+'use strict';
+
+require("website_sale.tour");
+var tour = require("web_tour.tour");
+var base = require("web_editor.base");
+
+tour.register('shop_sale_coupon', {
+    test: true,
+    url: '/shop',
+    wait_for: base.ready()
+},
+    [
+        /* 1. Buy 1 iPad Mini, enable coupon code & insert 10% code */
+        {
+            content: "type iPad Mini in search",
+            trigger: 'form input[name="search"]',
+            run: "text iPad Mini",
+        },
+        {
+            content: "start search",
+            trigger: 'form:has(input[name="search"]) .oe_search_button',
+        },
+        {
+            content: "select iPad Mini",
+            extra_trigger: '.oe_website_sale_search', // Wait to be on search results or it sometimes throws concurent error (sent search form + click on product on /shop)
+            trigger: '.oe_product_cart a:contains("iPad Mini")',
+        },
+        {
+            content: "add 3 iPad Mini into cart",
+            trigger: '#product_details input[name="add_qty"]',
+            run: "text 3",
+        },
+        {
+            content: "click on 'Add to Cart' button",
+            trigger: "a:contains(Add to Cart)",
+        },
+        {
+            content: "open customize menu",
+            extra_trigger: '.oe_website_sale .oe_cart',
+            trigger: '#customize-menu > a',
+        },
+        {
+            content: "click on 'Coupon Code'",
+            trigger: "#customize-menu a:contains(Coupon Code)",
+        },
+        {
+            content: "insert promo code '10pc'",
+            extra_trigger: 'form[name="coupon_code"]',
+            trigger: 'form[name="coupon_code"] input[name="promo"]',
+            run: "text 10pc",
+        },
+        {
+            content: "validate the coupon",
+            trigger: 'form[name="coupon_code"] .a-submit',
+        },
+        {
+            content: "check reward product",
+            trigger: '.reward_product:contains("10.0 % discount on total amount")',
+            run: function () {}, // it's a check
+        },
+        /* 2. Add some iPad to get a free one, play with quantity */
+        {
+            content: "add more iPad Mini into cart",
+            trigger: '#cart_products input.js_quantity',
+            run: "text 4",
+        },
+        {
+            content: "check reduction amount got recomputed",
+            extra_trigger: '.oe_currency_value:contains("96.00")', // For some reason, "-96.00" won't work
+            trigger: '.reward_product:contains("Free Product - iPad Mini")',
+            run: function () {}, // it's a check
+        },
+        {
+            content: "remove one iPad from cart",
+            trigger: '#cart_products a.js_add_cart_json:first',
+        },
+        {
+            content: "check free product is removed",
+            trigger: '#wrap:not(:has(.reward_product:contains("Free Product - iPad Mini")))',
+            run: function () {}, // it's a check
+        },
+        /* 3. Empty cart and disable coupon */
+        {
+            content: "remove iPad Mini from cart",
+            trigger: '#cart_products input.js_quantity:first',
+            run: "text 0",
+        },
+        {
+            content: "check cart is empty",
+            trigger: '#wrap:not(:has(#cart_products))',
+            run: function () {}, // it's a check
+        },
+        /* 4. Disabled coupon box */
+        {
+            content: "open customize menu",
+            extra_trigger: '.oe_website_sale .oe_cart',
+            trigger: '#customize-menu > a',
+        },
+        {
+            content: "click on 'Coupon Code'",
+            trigger: "#customize-menu a:contains(Coupon Code)",
+        },
+    ]
+);
+
+//Check if website_sale_options is installed since its adding an extra step to add to cart
+if ('website_sale_options.website_sale' in odoo.__DEBUG__.services) {
+  var steps = tour.tours.shop_sale_coupon.steps;
+  for (var k=0; k<steps.length; k++) {
+      if (steps[k].content === "click on 'Add to Cart' button") {
+          steps.splice(k+1, 0, {
+              content: "click in modal on 'Proceed to checkout' button",
+              trigger: 'a:contains("Proceed to checkout")',
+          });
+          break;
+      }
+  }
+}
+});
diff --git a/tests/__init__.py b/tests/__init__.py
new file mode 100644
index 00000000000..9f48e6f73be
--- /dev/null
+++ b/tests/__init__.py
@@ -0,0 +1 @@
+from . import test_shop_sale_coupon
diff --git a/tests/test_shop_sale_coupon.py b/tests/test_shop_sale_coupon.py
new file mode 100644
index 00000000000..982cbd5de8b
--- /dev/null
+++ b/tests/test_shop_sale_coupon.py
@@ -0,0 +1,11 @@
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
+import odoo.tests
+
+class TestUi(odoo.tests.HttpCase):
+
+    post_install = True
+    at_install = False
+
+    def test_01_admin_shop_sale_coupon_tour(self):
+        self.phantom_js("/", "odoo.__DEBUG__.services['web_tour.tour'].run('shop_sale_coupon')", "odoo.__DEBUG__.services['web_tour.tour'].tours.shop_sale_coupon.ready", login="admin")
diff --git a/views/website_sale_templates.xml b/views/website_sale_templates.xml
index e3909b3be36..83203f81d79 100644
--- a/views/website_sale_templates.xml
+++ b/views/website_sale_templates.xml
@@ -1,10 +1,20 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
 
+<template id="assets_frontend" inherit_id="website.assets_frontend" name="Shop Sale Coupon">
+    <xpath expr="." position="inside">
+        <script type="text/javascript" src="/website_sale_coupon/static/src/js/website_sale_coupon.test.js"></script>
+    </xpath>
+</template>
 <template id="website_sale_coupon_cart_hide_qty" inherit_id="website_sale.cart_lines" active="True">
     <xpath expr="//td[hasclass('td-qty')]/div" position="attributes">
         <attribute name="t-att-style">line.is_reward_line and "display:none;"</attribute>
     </xpath>
+    <xpath expr="//table[@id='cart_products']/tbody//tr//td[hasclass('td-product_name')]//div" position="attributes">
+    <!-- Can't xpath "//table[@id='cart_products']/tbody//tr" since optional_product already xpath that one.
+         There will be the same xpath attribute class on the same element. Last installed module will only be taken into account -->
+        <attribute name="t-att-class">'reward_product' if line.is_reward_line else None</attribute>
+    </xpath>
     <xpath expr="//td[hasclass('td-qty')]/div" position="after">
         <div t-if="line.is_reward_line" class="css_quantity input-group oe_website_spinner" id="line_qty">
             <div t-attf-href="#" class="mb8 input-group-addon js_add_cart_json hidden-xs"><i class="fa fa-minus text-muted"/></div>

From e3ff8a11e4d5bd5e95162b61dbb214f5ca5052a8 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@odoo.com>
Date: Fri, 2 Mar 2018 10:51:52 +0100
Subject: [PATCH 2/3] [FIX] website_sale_coupon: adapt test to new term.

Option has been renamed with odoo/odoo@fff0314623f9c7d0680d197c875db61087e5282e
---
 static/src/js/website_sale_coupon.test.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/static/src/js/website_sale_coupon.test.js b/static/src/js/website_sale_coupon.test.js
index 1ce783c54c0..4740a0a3f76 100644
--- a/static/src/js/website_sale_coupon.test.js
+++ b/static/src/js/website_sale_coupon.test.js
@@ -41,8 +41,8 @@ tour.register('shop_sale_coupon', {
             trigger: '#customize-menu > a',
         },
         {
-            content: "click on 'Coupon Code'",
-            trigger: "#customize-menu a:contains(Coupon Code)",
+            content: "click on 'Promo Code'",
+            trigger: "#customize-menu a:contains(Promo Code)",
         },
         {
             content: "insert promo code '10pc'",
@@ -98,8 +98,8 @@ tour.register('shop_sale_coupon', {
             trigger: '#customize-menu > a',
         },
         {
-            content: "click on 'Coupon Code'",
-            trigger: "#customize-menu a:contains(Coupon Code)",
+            content: "click on 'Promo Code'",
+            trigger: "#customize-menu a:contains(Promo Code)",
         },
     ]
 );

From ee065efddf66c9dfb4b56157c956df36634851f9 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@odoo.com>
Date: Fri, 2 Mar 2018 16:13:24 +0100
Subject: [PATCH 3/3] [FIX] website_sale_coupon: adapt test to capitalized text

---
 static/src/js/website_sale_coupon.test.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/static/src/js/website_sale_coupon.test.js b/static/src/js/website_sale_coupon.test.js
index 4740a0a3f76..3bf9a0fd2bd 100644
--- a/static/src/js/website_sale_coupon.test.js
+++ b/static/src/js/website_sale_coupon.test.js
@@ -110,8 +110,8 @@ if ('website_sale_options.website_sale' in odoo.__DEBUG__.services) {
   for (var k=0; k<steps.length; k++) {
       if (steps[k].content === "click on 'Add to Cart' button") {
           steps.splice(k+1, 0, {
-              content: "click in modal on 'Proceed to checkout' button",
-              trigger: 'a:contains("Proceed to checkout")',
+              content: "click in modal on 'Proceed to Checkout' button",
+              trigger: 'a:contains("Proceed to Checkout")',
           });
           break;
       }