From 919869907de3945f02ac2dfb33190eb82c8edca5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Pigeon?= Date: Tue, 2 Sep 2014 16:11:37 +0200 Subject: [PATCH] [ADD] add new test for onchange on one2many field --- openerp/addons/test_new_api/models.py | 5 +++++ .../test_new_api/tests/test_onchange.py | 21 +++++++++++++++++++ 2 files changed, 26 insertions(+) diff --git a/openerp/addons/test_new_api/models.py b/openerp/addons/test_new_api/models.py index 263868c835b..84c897e051d 100644 --- a/openerp/addons/test_new_api/models.py +++ b/openerp/addons/test_new_api/models.py @@ -85,11 +85,16 @@ class Discussion(models.Model): 'test_new_api_discussion_category', 'discussion', 'category') participants = fields.Many2many('res.users') messages = fields.One2many('test_new_api.message', 'discussion') + message_changes = fields.Integer('Message changes') @api.onchange('moderator') def _onchange_moderator(self): self.participants |= self.moderator + @api.onchange('messages') + def _onchange_messages(self): + self.message_changes = len(self.messages) + class Message(models.Model): _name = 'test_new_api.message' diff --git a/openerp/addons/test_new_api/tests/test_onchange.py b/openerp/addons/test_new_api/tests/test_onchange.py index c69becf9263..cbd24f4daac 100644 --- a/openerp/addons/test_new_api/tests/test_onchange.py +++ b/openerp/addons/test_new_api/tests/test_onchange.py @@ -163,3 +163,24 @@ class TestOnChange(common.TransactionCase): self.assertLessEqual(set(['participants']), set(result['value'])) self.assertItemsEqual(result['value']['participants'], participants + [(4, demo.id)]) + + def test_onchange_one2many(self): + discussion = self.env.ref('test_new_api.discussion_0') + demo = self.env.ref('base.user_demo') + + field_onchange = self.Discussion._onchange_spec() + self.assertEqual(field_onchange.get('messages'), '1') + + self.assertEqual(len(discussion.messages), 3) + messages = [(4, msg.id) for msg in discussion.messages] + messages[0] = (1, messages[0][1], {'body': 'test onchange'}) + values = { + 'name': discussion.name, + 'moderator': demo.id, + 'categories': [(4, cat.id) for cat in discussion.categories], + 'messages': messages, + 'participants': [(4, usr.id) for usr in discussion.participants], + 'message_changes': len(discussion.messages) + } + result = discussion.onchange(values, 'messages', field_onchange) + self.assertEqual(result['value'].get('message_changes', 0), len(discussion.messages))