diff options
| author | Martin Trigaux <mat@odoo.com> | 2020-05-19 10:29:16 +0200 |
|---|---|---|
| committer | Martin Trigaux <mat@odoo.com> | 2020-05-26 15:50:11 +0200 |
| commit | 6db2a81ea2578615ccc17199dfc488745b2fc596 (patch) | |
| tree | 0e0d8fc185987117922285f360b0728c5ae2cb95 /odoo | |
| parent | 2f26eee981e6857fc0d9aeebe53ec85939bebfb2 (diff) | |
[IMP] *: use specific ir.property methods
Do not make direct CRUD to ir.property records, use appropriate
methods for that
Diffstat (limited to 'odoo')
| -rw-r--r-- | odoo/addons/base/tests/test_orm.py | 15 | ||||
| -rw-r--r-- | odoo/addons/test_performance/tests/test_performance.py | 7 |
2 files changed, 7 insertions, 15 deletions
diff --git a/odoo/addons/base/tests/test_orm.py b/odoo/addons/base/tests/test_orm.py index 2e0406e6b5b..285e4b67290 100644 --- a/odoo/addons/base/tests/test_orm.py +++ b/odoo/addons/base/tests/test_orm.py @@ -235,21 +235,18 @@ class TestORM(TransactionCase): 'groups_id': [(6, 0, [self.ref('base.group_partner_manager')])], }) p1 = self.env['res.partner'].with_user(user).create({'name': 'Zorro'}) - p1_prop = self.env['ir.property'].with_user(user).create({ - 'name': 'Slip en laine', - 'res_id': 'res.partner,{}'.format(p1.id), - 'fields_id': self.env['ir.model.fields'].search([ - ('model', '=', 'res.partner'), ('name', '=', 'ref')], limit=1).id, - 'value_text': 'Nain poilu', - 'type': 'char', - }) + self.env['ir.property'].with_user(user)._set_multi("ref", "res.partner", {p1.id: "Nain poilu"}) + p1_prop = self.env['ir.property'].with_user(user)._get("ref", "res.partner", res_id=p1.id) + self.assertEqual( + p1_prop, "Nain poilu", 'p1_prop should have been created') # Unlink with unprivileged user p1.unlink() # ir.property is deleted + p1_prop = self.env['ir.property'].with_user(user)._get("ref", "res.partner", res_id=p1.id) self.assertEqual( - p1_prop.exists(), self.env['ir.property'], 'p1_prop should have been deleted') + p1_prop, False, 'p1_prop should have been deleted') def test_create_multi(self): """ create for multiple records """ diff --git a/odoo/addons/test_performance/tests/test_performance.py b/odoo/addons/test_performance/tests/test_performance.py index 104fce93195..01bab7187e2 100644 --- a/odoo/addons/test_performance/tests/test_performance.py +++ b/odoo/addons/test_performance/tests/test_performance.py @@ -485,12 +485,7 @@ class TestIrPropertyOptimizations(TransactionCase): def test_with_truthy_default(self): eggs = self.Eggs.create({}) - field = self.env['ir.model.fields']._get('test_performance.bacon', 'property_eggs') - self.env['ir.property'].create({ - 'name': 'property_eggs_with_bacon', - 'fields_id': field.id, - 'value': eggs, - }) + self.env['ir.property']._set_default("property_eggs", "test_performance.bacon", eggs) self.assertEqual(eggs, self.env['ir.property']._get('property_eggs', 'test_performance.bacon')) |
