[FIX] payment_authorize/paypal: Fix tests

2 files changed, 44 insertions, 39 deletions

diff --git a/addons/payment_authorize/tests/test_authorize.py b/addons/payment_authorize/tests/test_authorize.py
index 59fb401b091..98d87b04116 100644
--- a/addons/payment_authorize/tests/test_authorize.py
+++ b/addons/payment_authorize/tests/test_authorize.py
@@ -89,10 +89,13 @@ class AuthorizeForm(AuthorizeCommon):
         res = self.authorize.render('SO004', 320.0, self.currency_usd.id, values=self.buyer_values)
         # check form result
         tree = objectify.fromstring(res)
-        self.assertEqual(tree.get('action'), 'https://test.authorize.net/gateway/transact.dll', 'Authorize: wrong form POST url')
+
+        data_set = tree.xpath("//input[@name='data_set']")
+        self.assertEqual(len(data_set), 1, 'Authorize: Found %d "data_set" input instead of 1' % len(data_set))
+        self.assertEqual(data_set[0].get('data-action-url'), 'https://test.authorize.net/gateway/transact.dll', 'Authorize: wrong data-action-url POST url')
         for el in tree.iterfind('input'):
             values = list(el.attrib.values())
-            if values[1] in ['submit', 'x_fp_hash', 'return_url', 'x_state', 'x_ship_to_state']:
+            if values[1] in ['submit', 'x_fp_hash', 'return_url', 'x_state', 'x_ship_to_state', 'data_set']:
                 continue
             self.assertEqual(
                 values[2],
diff --git a/addons/payment_authorize/views/payment_authorize_templates.xml b/addons/payment_authorize/views/payment_authorize_templates.xml
index b4ea25275ee..b5e5c8d3515 100644
--- a/addons/payment_authorize/views/payment_authorize_templates.xml
+++ b/addons/payment_authorize/views/payment_authorize_templates.xml
@@ -48,43 +48,45 @@
         </template>
 
         <template id="authorize_form">
-            <input type="hidden" name="data_set" t-att-data-action-url="tx_url"/>
-            <input type='hidden' name='x_login' t-att-value='x_login'/>
-            <input type='hidden' name='x_fp_hash' t-att-value='x_fp_hash'/>
-            <input type='hidden' name='x_amount' t-att-value='x_amount'/>
-            <input type='hidden' name='x_show_form' t-att-value="x_show_form"/>
-            <input type='hidden' name='x_type' t-att-value="x_type"/>
-            <input type='hidden' name='x_method' t-att-value="x_method"/>
-            <input type='hidden' name='x_fp_sequence' t-att-value='x_fp_sequence'/>
-            <input type='hidden' name='x_version' t-att-value="x_version"/>
-            <input type="hidden" name="x_relay_response" t-att-value="x_relay_response"/>
-            <input type="hidden" name="x_relay_url" t-att-value="x_relay_url"/>
-            <input type='hidden' name="x_fp_timestamp" t-att-value="x_fp_timestamp"/>
-            <input type="hidden" name='return_url' t-att-value="returndata"/>
-            <input type="hidden" name='x_cancel_url' t-att-value="x_cancel_url"/>
-            <!--Order Information -->
-            <input type='hidden' name='x_invoice_num' t-att-value='reference'/>
-            <input type='hidden' name='x_currency_code' t-att-value='x_currency_code'/>
-            <!-- Billing Information-->
-            <input type='hidden' name='x_first_name' t-att-value="billing_first_name"/>
-            <input type='hidden' name='x_last_name' t-att-value="billing_last_name"/>
-            <input type='hidden' name='x_address' t-att-value="billing_address"/>
-            <input type='hidden' name='x_city' t-att-value="billing_city"/>
-            <input type='hidden' name='x_zip' t-att-value="billing_zip_code"/>
-            <input type='hidden' name='x_country' t-att-value="billing_country"/>
-            <input type='hidden' name='x_phone' t-att-value='billing_phone'/>
-            <input type='hidden' name='x_email' t-att-value="billing_email"/>
-            <input type='hidden' name='x_state' t-att-value="billing_state"/>
-            <!-- Shipping Information-->
-            <input type='hidden' name='x_ship_to_first_name' t-att-value="first_name"/>
-            <input type='hidden' name='x_ship_to_last_name' t-att-value="last_name"/>
-            <input type='hidden' name='x_ship_to_address' t-att-value="address"/>
-            <input type='hidden' name='x_ship_to_city' t-att-value="city"/>
-            <input type='hidden' name='x_ship_to_zip' t-att-value="zip_code"/>
-            <input type='hidden' name='x_ship_to_country' t-att-value="country"/>
-            <input type='hidden' name='x_ship_to_phone' t-att-value='phone'/>
-            <input type='hidden' name='x_ship_to_email' t-att-value="email"/>
-            <input type='hidden' name='x_ship_to_state' t-att-value="state"/>
+            <div>
+                <input type="hidden" name="data_set" t-att-data-action-url="tx_url"/>
+                <input type='hidden' name='x_login' t-att-value='x_login'/>
+                <input type='hidden' name='x_fp_hash' t-att-value='x_fp_hash'/>
+                <input type='hidden' name='x_amount' t-att-value='x_amount'/>
+                <input type='hidden' name='x_show_form' t-att-value="x_show_form"/>
+                <input type='hidden' name='x_type' t-att-value="x_type"/>
+                <input type='hidden' name='x_method' t-att-value="x_method"/>
+                <input type='hidden' name='x_fp_sequence' t-att-value='x_fp_sequence'/>
+                <input type='hidden' name='x_version' t-att-value="x_version"/>
+                <input type="hidden" name="x_relay_response" t-att-value="x_relay_response"/>
+                <input type="hidden" name="x_relay_url" t-att-value="x_relay_url"/>
+                <input type='hidden' name="x_fp_timestamp" t-att-value="x_fp_timestamp"/>
+                <input type="hidden" name='return_url' t-att-value="returndata"/>
+                <input type="hidden" name='x_cancel_url' t-att-value="x_cancel_url"/>
+                <!--Order Information -->
+                <input type='hidden' name='x_invoice_num' t-att-value='reference'/>
+                <input type='hidden' name='x_currency_code' t-att-value='x_currency_code'/>
+                <!-- Billing Information-->
+                <input type='hidden' name='x_first_name' t-att-value="billing_first_name"/>
+                <input type='hidden' name='x_last_name' t-att-value="billing_last_name"/>
+                <input type='hidden' name='x_address' t-att-value="billing_address"/>
+                <input type='hidden' name='x_city' t-att-value="billing_city"/>
+                <input type='hidden' name='x_zip' t-att-value="billing_zip_code"/>
+                <input type='hidden' name='x_country' t-att-value="billing_country"/>
+                <input type='hidden' name='x_phone' t-att-value='billing_phone'/>
+                <input type='hidden' name='x_email' t-att-value="billing_email"/>
+                <input type='hidden' name='x_state' t-att-value="billing_state"/>
+                <!-- Shipping Information-->
+                <input type='hidden' name='x_ship_to_first_name' t-att-value="first_name"/>
+                <input type='hidden' name='x_ship_to_last_name' t-att-value="last_name"/>
+                <input type='hidden' name='x_ship_to_address' t-att-value="address"/>
+                <input type='hidden' name='x_ship_to_city' t-att-value="city"/>
+                <input type='hidden' name='x_ship_to_zip' t-att-value="zip_code"/>
+                <input type='hidden' name='x_ship_to_country' t-att-value="country"/>
+                <input type='hidden' name='x_ship_to_phone' t-att-value='phone'/>
+                <input type='hidden' name='x_ship_to_email' t-att-value="email"/>
+                <input type='hidden' name='x_ship_to_state' t-att-value="state"/>
+            </div>
         </template>
 
         <template id="payment_authorize_redirect" name="Payment Authorize">