[IMP] minor perf improvement during tests
During tests, some creation of user records would unnecessarily trigger password reset or set a password, both of which would trigger password hashing which takes some time (for good reasons). Fix by: * passing no_reset_password in YAML tests and some Python tests still missing it (a number of Python tests already used it) * removing passwords from YAML records as they're never necessary, the test user records are not expected to ever log in
This commit is contained in:
parent
6cb3fe43ed
commit
f5b88f6309
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Accountant
|
||||
login: acc
|
||||
password: acc
|
||||
email: accountuser@yourcompany.com
|
||||
-
|
||||
I added groups for Accountant.
|
||||
|
@ -21,7 +20,6 @@
|
|||
company_id: base.main_company
|
||||
name: Financial Manager
|
||||
login: fm
|
||||
password: fm
|
||||
email: accountmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Financial Manager.
|
||||
|
@ -29,4 +27,4 @@
|
|||
!record {model: res.users, id: res_users_account_manager}:
|
||||
groups_id:
|
||||
- account.group_account_manager
|
||||
- base.group_partner_manager
|
||||
- base.group_partner_manager
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Voucher Accountant
|
||||
login: vacc
|
||||
password: acc
|
||||
email: accountant@yourcompany.com
|
||||
-
|
||||
I added groups to Accountant for account voucher.
|
||||
|
@ -21,7 +20,6 @@
|
|||
company_id: base.main_company
|
||||
name: Financial Manager for voucher
|
||||
login: fmv
|
||||
password: fmv
|
||||
email: finmanager@yourcompany.com
|
||||
-
|
||||
I added groups to Financial Manager for account voucher.
|
||||
|
|
|
@ -6,8 +6,6 @@
|
|||
<record id="default_template_user" model="res.users">
|
||||
<field name="name">Template User</field>
|
||||
<field name="login">portaltemplate</field>
|
||||
<!-- No password: this template user doesn't need to log in -->
|
||||
<field name="password"></field>
|
||||
<field name="active" eval="False"/>
|
||||
<!-- Avoid auto-including this user in any default group, just like a typical portal member -->
|
||||
<field name="groups_id" eval="[(6, 0, [ref('base.group_portal')])]"/>
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Crm Sales manager
|
||||
login: csm
|
||||
password: csm
|
||||
email: crmmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Crm Salesmanager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: Crm Salesman
|
||||
login: csu
|
||||
password: csu
|
||||
email: crmuser@yourcompany.com
|
||||
-
|
||||
I added groups for Crm Salesman.
|
||||
|
|
|
@ -4,22 +4,18 @@
|
|||
!record {model: res.users, id: test_res_user_01}:
|
||||
name: 'Test user A'
|
||||
login: 'tua@example.com'
|
||||
new_password: 'tua'
|
||||
-
|
||||
!record {model: res.users, id: test_res_user_02}:
|
||||
name: 'Test user B'
|
||||
login: 'tub@example.com'
|
||||
new_password: 'tub'
|
||||
-
|
||||
!record {model: res.users, id: test_res_user_03}:
|
||||
name: 'Test user C'
|
||||
login: 'tuc@example.com'
|
||||
new_password: 'tuc'
|
||||
-
|
||||
!record {model: res.users, id: test_res_user_04}:
|
||||
name: 'Test user D'
|
||||
login: 'tud@example.com'
|
||||
new_password: 'tud'
|
||||
-
|
||||
Salesman also creates lead so giving access rights of salesman.
|
||||
-
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Event manager
|
||||
login: em
|
||||
password: em
|
||||
email: eventmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Event manager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: User
|
||||
login: eu
|
||||
password: eu
|
||||
email: eventuser@yourcompany.com
|
||||
-
|
||||
I added groups for Event user.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR manager
|
||||
login: hrm
|
||||
password: hrm
|
||||
-
|
||||
I added groups for HR Manager.
|
||||
-
|
||||
|
@ -19,7 +18,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR Officer
|
||||
login: hro
|
||||
password: hro
|
||||
email: hro@example.com
|
||||
-
|
||||
I added groups for HR Officer.
|
||||
|
@ -34,7 +32,6 @@
|
|||
company_id: base.main_company
|
||||
name: Employee
|
||||
login: emp
|
||||
password: emp
|
||||
-
|
||||
I added groups for Employee.
|
||||
-
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR Officer
|
||||
login: ao
|
||||
password: ao
|
||||
-
|
||||
I added groups for HR Attendance Officer.
|
||||
-
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR Recruitment Officer
|
||||
login: hrro
|
||||
password: hrro
|
||||
email: hrofcr@yourcompany.com
|
||||
-
|
||||
I added groups for HR Recruitment Officer.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR timesheet manager
|
||||
login: hrtm
|
||||
password: hrtm
|
||||
-
|
||||
I added groups for HR timesheet Manager.
|
||||
-
|
||||
|
@ -19,7 +18,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR timesheet Officer
|
||||
login: hrto
|
||||
password: hrto
|
||||
-
|
||||
I added groups for HR timesheet Officer.
|
||||
-
|
||||
|
@ -33,7 +31,6 @@
|
|||
company_id: base.main_company
|
||||
name: Timesheet Employee
|
||||
login: empt
|
||||
password: empt
|
||||
-
|
||||
I added groups for Timesheet Employee.
|
||||
-
|
||||
|
|
|
@ -100,7 +100,7 @@ class TestMail(common.SavepointCase):
|
|||
'login': 'raoul',
|
||||
'alias_name': 'raoul',
|
||||
'groups_id': [(6, 0, [cls.group_employee_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_bert_id = cls.res_users.create(cr, uid, {
|
||||
'name': 'Bert Tartignole',
|
||||
'signature': 'SignBert',
|
||||
|
@ -108,7 +108,7 @@ class TestMail(common.SavepointCase):
|
|||
'login': 'bert',
|
||||
'alias_name': 'bert',
|
||||
'groups_id': [(6, 0, [])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_raoul = cls.res_users.browse(cr, uid, cls.user_raoul_id)
|
||||
cls.user_bert = cls.res_users.browse(cr, uid, cls.user_bert_id)
|
||||
cls.user_admin = cls.res_users.browse(cr, uid, uid)
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: MRP Manager
|
||||
login: mam
|
||||
password: mam
|
||||
email: mrp_manager@yourcompany.com
|
||||
-
|
||||
I added groups for MRP Manager.
|
||||
|
@ -21,11 +20,10 @@
|
|||
company_id: base.main_company
|
||||
name: MRP User
|
||||
login: mau
|
||||
password: mau
|
||||
email: mrp_user@yourcompany.com
|
||||
-
|
||||
I added groups for MRP User.
|
||||
-
|
||||
!record {model: res.users, id: res_users_mrp_user}:
|
||||
groups_id:
|
||||
- mrp.group_mrp_user
|
||||
- mrp.group_mrp_user
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: MRP User
|
||||
login: maou
|
||||
password: maou
|
||||
email: mrp_operation_user@yourcompany.com
|
||||
-
|
||||
I added groups for MRP User.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: MRP Manager
|
||||
login: marm
|
||||
password: marm
|
||||
email: mrp_repair_manager@yourcompany.com
|
||||
-
|
||||
I added groups for MRP Repair Manager.
|
||||
|
@ -20,11 +19,10 @@
|
|||
company_id: base.main_company
|
||||
name: MRP User
|
||||
login: maru
|
||||
password: maru
|
||||
email: mrp_repair_user@yourcompany.com
|
||||
-
|
||||
I added groups for MRP Repair User.
|
||||
-
|
||||
!record {model: res.users, id: res_mrp_repair_user}:
|
||||
groups_id:
|
||||
- mrp.group_mrp_user
|
||||
- mrp.group_mrp_user
|
||||
|
|
|
@ -36,7 +36,12 @@ class test_portal(TestMail):
|
|||
cls.group_portal_id = cls.env.ref('base.group_portal').id
|
||||
|
||||
# Create Chell (portal user)
|
||||
cls.user_chell_id = cls.res_users.create(cr, uid, {'name': 'Chell Gladys', 'login': 'chell', 'email': 'chell@gladys.portal', 'groups_id': [(6, 0, [cls.group_portal_id])]})
|
||||
cls.user_chell_id = cls.res_users.create(cr, uid, {
|
||||
'name': 'Chell Gladys',
|
||||
'login': 'chell',
|
||||
'email': 'chell@gladys.portal',
|
||||
'groups_id': [(6, 0, [cls.group_portal_id])]
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_chell = cls.res_users.browse(cr, uid, cls.user_chell_id)
|
||||
cls.partner_chell_id = cls.user_chell.partner_id.id
|
||||
|
||||
|
|
|
@ -45,19 +45,19 @@ class TestPortalProjectBase(TestProjectBase):
|
|||
'login': 'chell',
|
||||
'alias_name': 'chell',
|
||||
'groups_id': [(6, 0, [self.group_portal_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
self.user_public_id = self.res_users.create(cr, uid, {
|
||||
'name': 'Donovan Public',
|
||||
'login': 'donovan',
|
||||
'alias_name': 'donovan',
|
||||
'groups_id': [(6, 0, [self.group_public_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
self.user_manager_id = self.res_users.create(cr, uid, {
|
||||
'name': 'Eustache Manager',
|
||||
'login': 'eustache',
|
||||
'alias_name': 'eustache',
|
||||
'groups_id': [(6, 0, [self.group_project_manager_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
|
||||
# Test 'Pigs' project
|
||||
self.project_pigs_id = self.project_project.create(cr, uid, {
|
||||
|
|
|
@ -57,21 +57,21 @@ class TestProjectBase(TestMail):
|
|||
'alias_name': 'armande',
|
||||
'email': 'armande.projectuser@example.com',
|
||||
'groups_id': [(6, 0, [cls.group_employee_id, cls.group_project_user_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_projectmanager_id = cls.res_users.create(cr, uid, {
|
||||
'name': 'Bastien ProjectManager',
|
||||
'login': 'bastien',
|
||||
'alias_name': 'bastien',
|
||||
'email': 'bastien.projectmanager@example.com',
|
||||
'groups_id': [(6, 0, [cls.group_employee_id, cls.group_project_manager_id])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_none_id = cls.res_users.create(cr, uid, {
|
||||
'name': 'Charlie Avotbonkeur',
|
||||
'login': 'charlie',
|
||||
'alias_name': 'charlie',
|
||||
'email': 'charlie.noone@example.com',
|
||||
'groups_id': [(6, 0, [])]
|
||||
})
|
||||
}, {'no_reset_password': True})
|
||||
cls.user_projectuser = cls.res_users.browse(cr, uid, cls.user_projectuser_id)
|
||||
cls.user_projectmanager = cls.res_users.browse(cr, uid, cls.user_projectmanager_id)
|
||||
cls.partner_projectuser_id = cls.user_projectuser.partner_id.id
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Project Manager
|
||||
login: prim
|
||||
password: prim
|
||||
email: issuemanager@yourcompany.com
|
||||
-
|
||||
I added groups for Project manager.
|
||||
|
@ -20,11 +19,10 @@
|
|||
company_id: base.main_company
|
||||
name: Project User
|
||||
login: priu
|
||||
password: priu
|
||||
email: issueuser@yourcompany.com
|
||||
-
|
||||
I added groups for Project user.
|
||||
-
|
||||
!record {model: res.users, id: res_users_project_issue_user}:
|
||||
groups_id:
|
||||
- project.group_project_user
|
||||
- project.group_project_user
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: HR Manager
|
||||
login: hr
|
||||
password: hr
|
||||
groups_id:
|
||||
- base.group_hr_manager
|
||||
-
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Purchase Manager
|
||||
login: pm
|
||||
password: pm
|
||||
email: purchasemanager@yourcompany.com
|
||||
-
|
||||
I added groups for Purchase manager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: Purchase User
|
||||
login: pu
|
||||
password: pu
|
||||
email: purchaseuser@yourcompany.com
|
||||
-
|
||||
I added groups for Purchase user.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Purchase requisition Manager
|
||||
login: prm
|
||||
password: prm
|
||||
email: requisition_manager@yourcompany.com
|
||||
-
|
||||
I added groups for Purchase Requisition Manager.
|
||||
|
@ -20,11 +19,10 @@
|
|||
company_id: base.main_company
|
||||
name: Purchase requisition User
|
||||
login: pru
|
||||
password: pru
|
||||
email: requisition_user@yourcompany.com
|
||||
-
|
||||
I added groups for Purchase Requisition User.
|
||||
-
|
||||
!record {model: res.users, id: res_users_purchase_requisition_user}:
|
||||
groups_id:
|
||||
- purchase_requisition.group_purchase_requisition_user
|
||||
- purchase_requisition.group_purchase_requisition_user
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Sales manager
|
||||
login: sm
|
||||
password: sm
|
||||
email: salesmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Salesmanager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: Salesman
|
||||
login: su
|
||||
password: su
|
||||
email: salesman@yourcompany.com
|
||||
-
|
||||
I added groups for Salesman.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock Sales manager
|
||||
login: ssm
|
||||
password: ssm
|
||||
email: ss_salesmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Salesmanager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock Salesman
|
||||
login: ssu
|
||||
password: ssu
|
||||
email: ss_salesman@yourcompany.com
|
||||
-
|
||||
I added groups for Stock Salesman.
|
||||
|
@ -36,7 +34,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock User
|
||||
login: sau
|
||||
password: sau
|
||||
email: stock_user@yourcompany.com
|
||||
-
|
||||
I added groups for Stock User.
|
||||
|
@ -51,8 +48,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock Manager
|
||||
login: sam
|
||||
password: sam
|
||||
email: admin@portal.example.com
|
||||
email: stock_manager@yourcompany.com
|
||||
-
|
||||
I added groups for Stock Manager.
|
||||
|
|
|
@ -5,7 +5,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock Manager
|
||||
login: sam
|
||||
password: sam
|
||||
email: stockmanager@yourcompany.com
|
||||
-
|
||||
I added groups for Stock Manager.
|
||||
|
@ -20,7 +19,6 @@
|
|||
company_id: base.main_company
|
||||
name: Stock User
|
||||
login: sau
|
||||
password: sau
|
||||
email: stockuser@yourcompany.com
|
||||
-
|
||||
I added groups for Stock User.
|
||||
|
|
|
@ -316,10 +316,10 @@ class YamlInterpreter(object):
|
|||
if not self._coerce_bool(record.forcecreate):
|
||||
return None
|
||||
|
||||
|
||||
#context = self.get_context(record, self.eval_context)
|
||||
#TOFIX: record.context like {'withoutemployee':True} should pass from self.eval_context. example: test_project.yml in project module
|
||||
context = record.context
|
||||
# FIXME: record.context like {'withoutemployee':True} should pass from self.eval_context. example: test_project.yml in project module
|
||||
# TODO: cleaner way to avoid resetting password in auth_signup (makes user creation costly)
|
||||
context = dict(record.context or {}, no_reset_password=True)
|
||||
view_info = False
|
||||
if view_id:
|
||||
varg = view_id
|
||||
|
|
Loading…
Reference in New Issue