diff options
| author | Xavier Morel <xmo@odoo.com> | 2015-06-23 13:14:07 +0200 |
|---|---|---|
| committer | Xavier Morel <xmo@odoo.com> | 2015-06-23 16:38:14 +0200 |
| commit | 11ba4689b1d9d602cc27270239fd96f2af840106 (patch) | |
| tree | f5d4938616f1fcc4f5afc9c2f4ac44751971306e /addons/project/tests/test_project_base.py | |
| parent | 386f76eb46deee167234edfb1f9b1d3bb6b86e5f (diff) | |
[IMP] running speed of some tests & new testcase type
Some tests (e.g. mail) have expensive and significant DB setup for a
number of small and cheap tests. Using a TransactionCase, the DB setup
far dominates the tests themselves, by up to 10x (mail unit tests take
~130s on my machine, the tests themselves take ~15s).
The SavepointCase introduced here is an hybrid of SingleTransactionCase
and TransactionCase: it uses a single transaction for all tests in a
class, but each test case is isolated by a rollbacked savepoint. This
allows a common DB setup (via setUpClass) while keeping independent
tests.
TransactionCase should remain the primary test case superclass, but
SavepointCase can be a fair optimisation when setup costs far dominate.
Diffstat (limited to 'addons/project/tests/test_project_base.py')
| -rw-r--r-- | addons/project/tests/test_project_base.py | 57 |
1 files changed, 28 insertions, 29 deletions
diff --git a/addons/project/tests/test_project_base.py b/addons/project/tests/test_project_base.py index 70f8b59812f..fc788de5c6c 100644 --- a/addons/project/tests/test_project_base.py +++ b/addons/project/tests/test_project_base.py @@ -24,76 +24,75 @@ from openerp.addons.mail.tests.common import TestMail class TestProjectBase(TestMail): - def setUp(self): - super(TestProjectBase, self).setUp() - cr, uid = self.cr, self.uid + @classmethod + def setUpClass(cls): + super(TestProjectBase, cls).setUpClass() + cr, uid = cls.cr, cls.uid # Usefull models - self.project_project = self.registry('project.project') - self.project_task = self.registry('project.task') - self.project_task_delegate = self.registry('project.task.delegate') + cls.project_project = cls.registry('project.project') + cls.project_task = cls.registry('project.task') + cls.project_task_delegate = cls.registry('project.task.delegate') # Find Project User group - group_project_user_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'project', 'group_project_user') - self.group_project_user_id = group_project_user_ref and group_project_user_ref[1] or False + cls.group_project_user_id = cls.env.ref('project.group_project_user').id or False # Find Project Manager group - group_project_manager_ref = self.registry('ir.model.data').get_object_reference(cr, uid, 'project', 'group_project_manager') - self.group_project_manager_id = group_project_manager_ref and group_project_manager_ref[1] or False + cls.group_project_manager_id = cls.env.ref('project.group_project_manager').id or False # Test partners to use through the various tests - self.project_partner_id = self.res_partner.create(cr, uid, { + cls.project_partner_id = cls.res_partner.create(cr, uid, { 'name': 'Gertrude AgrolaitPartner', 'email': 'gertrude.partner@agrolait.com', }) - self.email_partner_id = self.res_partner.create(cr, uid, { + cls.email_partner_id = cls.res_partner.create(cr, uid, { 'name': 'Patrick Ratatouille', 'email': 'patrick.ratatouille@agrolait.com', }) # Test users to use through the various tests - self.user_projectuser_id = self.res_users.create(cr, uid, { + cls.user_projectuser_id = cls.res_users.create(cr, uid, { 'name': 'Armande ProjectUser', 'login': 'Armande', 'alias_name': 'armande', 'email': 'armande.projectuser@example.com', - 'groups_id': [(6, 0, [self.group_employee_id, self.group_project_user_id])] + 'groups_id': [(6, 0, [cls.group_employee_id, cls.group_project_user_id])] }) - self.user_projectmanager_id = self.res_users.create(cr, uid, { + 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, [self.group_employee_id, self.group_project_manager_id])] + 'groups_id': [(6, 0, [cls.group_employee_id, cls.group_project_manager_id])] }) - self.user_none_id = self.res_users.create(cr, uid, { + 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, [])] }) - self.user_projectuser = self.res_users.browse(cr, uid, self.user_projectuser_id) - self.user_projectmanager = self.res_users.browse(cr, uid, self.user_projectmanager_id) - self.partner_projectuser_id = self.user_projectuser.partner_id.id - self.partner_projectmanager_id = self.user_projectmanager.partner_id.id + 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 + cls.partner_projectmanager_id = cls.user_projectmanager.partner_id.id # Test 'Pigs' project - self.project_pigs_id = self.project_project.create(cr, uid, { + cls.project_pigs_id = cls.project_project.create(cr, uid, { 'name': 'Pigs', 'privacy_visibility': 'public', 'alias_name': 'project+pigs', - 'partner_id': self.partner_raoul_id, + 'partner_id': cls.partner_raoul_id, }, {'mail_create_nolog': True}) # Already-existing tasks in Pigs - self.task_1_id = self.project_task.create(cr, uid, { + cls.task_1_id = cls.project_task.create(cr, uid, { 'name': 'Pigs UserTask', - 'user_id': self.user_projectuser_id, - 'project_id': self.project_pigs_id, + 'user_id': cls.user_projectuser_id, + 'project_id': cls.project_pigs_id, }, {'mail_create_nolog': True}) - self.task_2_id = self.project_task.create(cr, uid, { + cls.task_2_id = cls.project_task.create(cr, uid, { 'name': 'Pigs ManagerTask', - 'user_id': self.user_projectmanager_id, - 'project_id': self.project_pigs_id, + 'user_id': cls.user_projectmanager_id, + 'project_id': cls.project_pigs_id, }, {'mail_create_nolog': True}) |
