summaryrefslogtreecommitdiff
path: root/odoo
diff options
context:
space:
mode:
authorMartin Trigaux <mat@odoo.com>2020-05-19 10:29:16 +0200
committerMartin Trigaux <mat@odoo.com>2020-05-26 15:50:11 +0200
commit6db2a81ea2578615ccc17199dfc488745b2fc596 (patch)
tree0e0d8fc185987117922285f360b0728c5ae2cb95 /odoo
parent2f26eee981e6857fc0d9aeebe53ec85939bebfb2 (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.py15
-rw-r--r--odoo/addons/test_performance/tests/test_performance.py7
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'))