From d4a1cec260ffa9fb1c10430df6e8c38880ecaf6a Mon Sep 17 00:00:00 2001
From: "sgo@tinyerp.com" <>
Date: Tue, 19 Feb 2013 18:49:21 +0530
Subject: [PATCH] [IMP]improve code and yml

bzr revid: sgo@tinyerp.com-20130219131921-nbfs273xujvriomu
---
 addons/sale/__openerp__.py                    |  4 +-
 addons/sale/test/create_sale_users.yml        | 20 +++++++++
 addons/sale/test/sale_order_demo.yml          | 41 +++++++++++++++++++
 .../test/sale_order_salesmanager_demo.yml     | 31 --------------
 4 files changed, 63 insertions(+), 33 deletions(-)
 create mode 100644 addons/sale/test/create_sale_users.yml
 create mode 100644 addons/sale/test/sale_order_demo.yml
 delete mode 100644 addons/sale/test/sale_order_salesmanager_demo.yml

diff --git a/addons/sale/__openerp__.py b/addons/sale/__openerp__.py
index 6ba3a02bb00..2bea86b41af 100644
--- a/addons/sale/__openerp__.py
+++ b/addons/sale/__openerp__.py
@@ -80,8 +80,8 @@ The Dashboard for the Sales Manager will include
     ],
     'demo': ['sale_demo.xml'],
     'test': [
-        'test/sale_order_salesmanager_demo.yml',
-        'test/sale_order_salesman_demo.yml',
+        'test/create_sale_users.yml',
+        'test/sale_order_demo.yml',
         'test/manual_order_policy.yml',
         'test/cancel_order.yml',
         'test/delete_order.yml',
diff --git a/addons/sale/test/create_sale_users.yml b/addons/sale/test/create_sale_users.yml
new file mode 100644
index 00000000000..a61c4bd7c99
--- /dev/null
+++ b/addons/sale/test/create_sale_users.yml
@@ -0,0 +1,20 @@
+-
+   Create a user as 'Salesmanager'
+-
+  !record {model: res.users, id: res_users_salesmanager}:
+    company_id: base.main_company
+    name: Sales manager
+    login: sm
+    password: sm
+    groups_id:
+      - base.group_sale_manager
+-
+   Create a user as 'Salesman'
+-
+  !record {model: res.users, id: res_users_salesman}:
+    company_id: base.main_company
+    name: Salesman
+    login: su
+    password: su
+    groups_id:
+      - base.group_sale_salesman_all_leads
diff --git a/addons/sale/test/sale_order_demo.yml b/addons/sale/test/sale_order_demo.yml
new file mode 100644
index 00000000000..84aaa43a3cf
--- /dev/null
+++ b/addons/sale/test/sale_order_demo.yml
@@ -0,0 +1,41 @@
+-
+  Test the data with salesman,
+-
+  !context
+    uid: 'res_users_salesman'
+-
+  In order to test process of the Sale Order, I create sale order
+-
+  !record {model: sale.order, id: sale_order_test1}:
+    partner_id: base.res_partner_2
+    note: Invoice after delivery
+    payment_term: account.account_payment_term
+    order_line: 
+      - product_id: product.product_product_7
+        product_uom_qty: 8
+-
+  I verify that the onchange was correctly triggered
+-
+  !assert {model: sale.order, id: sale.sale_order_test1, string: The onchange function of product was not correctly triggered}:
+    - order_line[0].name == u'[LCD17] 17\u201d LCD Monitor'
+    - order_line[0].price_unit == 1350.0
+    - order_line[0].product_uom_qty == 8
+    - order_line[0].product_uom.id == ref('product.product_uom_unit')
+
+-
+  I create another sale order
+-
+  !record {model: sale.order, id: sale_order_test2}:
+    partner_id: base.res_partner_2
+    order_line: 
+      - product_id: product.product_product_7
+        product_uom_qty: 16
+        product_uom: product.product_uom_dozen
+-
+  I verify that the onchange was correctly triggered
+-
+  !assert {model: sale.order, id: sale.sale_order_test2, string: The onchange function of product was not correctly triggered}:
+    - order_line[0].name == u'[LCD17] 17\u201d LCD Monitor'
+    - order_line[0].price_unit == 1350.0 * 12
+    - order_line[0].product_uom.id == ref('product.product_uom_dozen')
+    - order_line[0].product_uom_qty == 16
\ No newline at end of file
diff --git a/addons/sale/test/sale_order_salesmanager_demo.yml b/addons/sale/test/sale_order_salesmanager_demo.yml
deleted file mode 100644
index f09e1327a75..00000000000
--- a/addons/sale/test/sale_order_salesmanager_demo.yml
+++ /dev/null
@@ -1,31 +0,0 @@
--
-  In order to test process of the Sale Order, I create sale order
--
-    Create a user as 'Salesmanager'
--
-  !record {model: res.users, id: res_users_salesmanager}:
-    company_id: base.main_company
-    name: Sales manager
-    login: sm
-    password: sm
-    groups_id:
-      - base.group_sale_manager
--
-  Now, check the data with sales Manager
--
-  !context
-    uid: 'res_users_salesmanager'
--
-  !record {model: sale.order, id: sale_order_test1}:
-    partner_id: base.res_partner_2
-    note: Invoice after delivery
-    payment_term: account.account_payment_term
-    order_line: 
-      - product_id: product.product_product_7
-        product_uom_qty: 8
--
-  I verify that the onchange was correctly triggered
--
-  !assert {model: sale.order, id: sale.sale_order_test1, string: The onchange function of product was not correctly triggered}:
-    - order_line[0].name == u'[LCD17] 17\u201d LCD Monitor'
-    - order_line[0].price_unit == 1350.0