From 42dcb2b01a9e7ecb223b78081e47bf15e61e5eb3 Mon Sep 17 00:00:00 2001 From: Fekete Mihai Date: Wed, 7 Mar 2018 12:16:15 +0200 Subject: [PATCH] [FIX] employee_firstname: test (#427) --- .../tests/test_hr_employee_firstname.py | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/hr_employee_firstname/tests/test_hr_employee_firstname.py b/hr_employee_firstname/tests/test_hr_employee_firstname.py index 847e059f632..c0d4996f4ba 100644 --- a/hr_employee_firstname/tests/test_hr_employee_firstname.py +++ b/hr_employee_firstname/tests/test_hr_employee_firstname.py @@ -46,12 +46,22 @@ def test_onchange(self): """ Validate the get_name method is not failing """ - field_onchange = self.employee1_id._onchange_spec() + field_onchange = self.employee_model.new({})._onchange_spec() self.assertEqual(field_onchange.get('firstname'), '1') self.assertEqual(field_onchange.get('lastname'), '1') - values = {'firstname': 'Antonio', 'lastname': 'Esposito'} - self.employee1_id.onchange(values, 'firstname', field_onchange) - self.employee1_id.onchange(values, 'lastname', field_onchange) + values = {'firstname': 'Antonio', + 'lastname': 'Esposito', + 'name': 'test employee'} + for field in self.employee_model._fields: + if field not in values: + values[field] = False + # we work on a temporary record + new_record = self.employee_model.new(values) + + updates = new_record.onchange( + values, ['firstname', 'lastname'], field_onchange) + values.update(updates.get('value', {})) + self.assertEqual(values['name'], 'Esposito Antonio') def test_auto_init_name(self): """