Jagdish Panchal (Open ERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-clean_yml_hr_expense-jap into 
lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-clean_yml_hr_expense-jap/+merge/83114

Hr_expense: Clean up the test case
---------------------------------

1) Removed hr_expense_report.yml
2) test_hr_expense.yml rename this file to confirm2heldinvoiced.yml and also 
modify 
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-clean_yml_hr_expense-jap/+merge/83114
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-clean_yml_hr_expense-jap.
=== modified file 'hr_expense/__openerp__.py'
--- hr_expense/__openerp__.py	2011-10-11 20:05:49 +0000
+++ hr_expense/__openerp__.py	2011-11-23 06:06:36 +0000
@@ -62,8 +62,8 @@
     'demo_xml': [
         'hr_expense_demo.xml',
         ],
-    'test': ['test/test_hr_expense.yml',
-             'test/hr_expense_report.yml',
+    'test': [
+             'test/confirm2heldinvoiced.yml',
              ],
     'installable': True,
     'active': False,

=== renamed file 'hr_expense/test/test_hr_expense.yml' => 'hr_expense/test/confirm2heldinvoiced.yml'
--- hr_expense/test/test_hr_expense.yml	2011-11-08 09:33:29 +0000
+++ hr_expense/test/confirm2heldinvoiced.yml	2011-11-23 06:06:36 +0000
@@ -1,17 +1,23 @@
 -
-  In order to test hr_expenses for OpenERP, I create expenses for employee and manage employee's expenses.
--
-  I create an unit of measure category "Working Time".
--
-  !record {model: product.uom.categ, id: product_uom_categ_workingtime0}:
-    name: Working Time.
-
--
-  I create a new employee.
--
-  !record {model: hr.employee, id: hr.employee}:
-    address_home_id: base.res_partner_address_1
-    address_id: base.main_address
+  In Order to test process of Employee Expenses.
+-
+  I create this expense.
+-
+  !workflow {model: hr.expense.expense, action: confirm, ref: hr_expense_expense_septemberexpenses0}
+-
+  I change the state of expense.
+-
+  !python {model: hr.expense.expense}: |
+    self.expense_confirm(cr, uid, [ref('hr_expense_expense_septemberexpenses0')])
+-
+  I check the the expense is created or not
+-
+  !assert {model: hr.expense.expense, id: hr_expense_expense_septemberexpenses0}:
+    - state == 'confirm'
+-
+  I accept this expense.
+-
+  !workflow {model: hr.expense.expense, action: validate, ref: hr_expense_expense_septemberexpenses0}
 -
   I assign home address to employee.
 -
@@ -19,103 +25,47 @@
     address_home_id: base.res_partner_address_1
     address_id: base.main_address
 -
-  I create product unit of measure "Hour".
--
-  !record {model: product.uom, id: product_uom_hour0}:
-    category_id: 'product_uom_categ_workingtime0'
-    factor: 8.0
-    name: Hour
-    rounding: 0.01
--
-   I Create a product "travel".
--
-  !record {model: product.product, id: product_product_travel0}:
-    categ_id: product.product_category_services
-    cost_method: standard
-    list_price: 800.0
-    mes_type: fixed
-    name: Travel
-    procure_method: make_to_stock
-    standard_price: 700.0
-    supply_method: buy
-    type: product
-    uom_id: 'product_uom_hour0'
-    uom_po_id: 'product_uom_hour0'
-    volume: 0.0
-    warranty: 0.0
-    weight: 0.0
-    weight_net: 0.0
--
-  I mark this product for expenses.
--
-  !python {model: product.product}: |
-    self.on_change_hr_expense_ok(cr, uid, [ref('product_product_travel0')], True)
--
-  I create an expense.
--
-  !record {model: hr.expense.expense, id: hr_expense_expense_september0}:
-    company_id: base.main_company
-    date: !eval "'%s-05-05' %(datetime.now().year)"
-    name: September Expenses
-    line_ids:
-      - date_value: !eval "'%s-05-27' %(datetime.now().year)"
-        name: Travel
-        product_id: 'product_product_travel0'
-        sequence: 0.0
-        uom_id: product.product_uom_unit
-        unit_amount: 700.0
-    user_id: base.user_root
--
-  I check that expense is in "Draft" state.
--
-  !assert {model: hr.expense.expense, id: hr_expense_expense_september0}:
-    - state == 'draft'
--
-  I change the employee on expense form.
--
-  !python {model: hr.expense.expense}: |
-    res = self.onchange_employee_id(cr, uid, [ref('hr_expense_expense_september0')], ref('hr.employee1'), None)
-    values = dict([('employee_id', ref('hr.employee1'))] + res['value'].items())
-    self.write(cr, uid, [ref('hr_expense_expense_september0')], values, None)
--
-  Now I change the related product on expense line.
--
-  !python {model: hr.expense.expense}: |
-    exp_line_obj = self.pool.get('hr.expense.line')
-    exp_obj = self.browse(cr, uid, ref('hr_expense_expense_september0'), None)
-    line_ids = [line.id for line in exp_obj.line_ids]
-    res = exp_line_obj.onchange_product_id(cr, uid, line_ids, ref('product_product_expense_air'), False, ref('hr.employee1'), None)
-    values = dict([('product_id', ref('product_product_expense_air'))] + res['value'].items())
-    exp_line_obj.write(cr, uid, line_ids, values, None)
--
-  I confirm this expenses by click on "Confirm" button.
--
-  !workflow {model: hr.expense.expense, action: confirm, ref: hr_expense_expense_september0}
-
--
-  I check that state is "Waiting Confirmation".
--
-  !assert {model: hr.expense.expense, id: hr_expense_expense_september0}:
-    - state == 'confirm'
--
-  I accept this expense by click on "Accept" button.
--
-  !workflow {model: hr.expense.expense, action: validate, ref: hr_expense_expense_september0}
-
--
-  I invoiced this expense by click on "Invoice" button.
--
-  !python {model: hr.expense.expense}: |
-    self.invoice(cr, uid, [ref('hr_expense_expense_september0')], None)
--
-  I check that state is "Invoiced".
--
-  !assert {model: hr.expense.expense, id: hr_expense_expense_september0}:
+  I create the Invoiced for the expense.
+-
+  !python {model: hr.expense.expense}: |
+    self.invoice(cr, uid, [ref('hr_expense_expense_septemberexpenses0')])
+    
+-
+  I check that expense is "Invoiced".
+-
+  !assert {model: hr.expense.expense, id: hr_expense_expense_septemberexpenses0}:
     - state == 'invoiced'
 -
   I check that Invoice is created for the expense.
 -
   !python {model: hr.expense.expense}: |
-    exp = self.browse(cr, uid, [ref('hr_expense_expense_september0')])[0]
+    exp = self.browse(cr, uid, [ref('hr_expense_expense_septemberexpenses0')])[0]
     assert exp.invoice_id, "Invoice should be created"
-
+-
+  I paid the expense
+-
+  !python {model: hr.expense.expense}: |
+    self.expense_paid(cr, uid, [ref('hr_expense_expense_septemberexpenses0')])
+-
+  I check the expense is paid.
+-
+  !assert {model: hr.expense.expense, id: hr_expense_expense_septemberexpenses0}:
+    - state == 'paid'
+-
+  I cancel the expense
+-
+  !python {model: hr.expense.expense}: |
+    self.expense_canceled(cr, uid, [ref('hr_expense_expense_septemberexpenses0')])
+-
+  I check the expense is cancel.
+-
+  !assert {model: hr.expense.expense, id: hr_expense_expense_septemberexpenses0}:
+    - state == 'cancelled' 
+-
+  we will print a PDF report defined on a HR Expense.
+- 
+  !python {model: hr.expense.expense}: |
+    import netsvc, tools, os
+    (data, format) = netsvc.LocalService('report.hr.expense').create(cr, uid, [ref('hr_expense.hr_expense_expense_septemberexpenses0'),ref('hr_expense.hr_expense_expense_septemberexpenses1')], {}, {})
+    if tools.config['test_report_directory']:
+        file(os.path.join(tools.config['test_report_directory'], 'hr_expense-report.'+format), 'wb+').write(data)
\ No newline at end of file

=== removed file 'hr_expense/test/hr_expense_report.yml'
--- hr_expense/test/hr_expense_report.yml	2011-01-14 00:11:01 +0000
+++ hr_expense/test/hr_expense_report.yml	1970-01-01 00:00:00 +0000
@@ -1,8 +0,0 @@
--
-  In order to test the PDF reports defined on a HR Expense, we will print a HR Expense
-- 
-  !python {model: hr.expense.expense}: |
-    import netsvc, tools, os
-    (data, format) = netsvc.LocalService('report.hr.expense').create(cr, uid, [ref('hr_expense.hr_expense_expense_septemberexpenses0'),ref('hr_expense.hr_expense_expense_septemberexpenses1')], {}, {})
-    if tools.config['test_report_directory']:
-        file(os.path.join(tools.config['test_report_directory'], 'hr_expense-report.'+format), 'wb+').write(data)
\ No newline at end of file

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to