summaryrefslogtreecommitdiff
path: root/addons/project/tests/test_project_base.py
diff options
context:
space:
mode:
authorXavier Morel <xmo@odoo.com>2015-06-23 13:14:07 +0200
committerXavier Morel <xmo@odoo.com>2015-06-23 16:38:14 +0200
commit11ba4689b1d9d602cc27270239fd96f2af840106 (patch)
treef5d4938616f1fcc4f5afc9c2f4ac44751971306e /addons/project/tests/test_project_base.py
parent386f76eb46deee167234edfb1f9b1d3bb6b86e5f (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.py57
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})