From 78dc8f68718a3f753a92bae3164ccdd2e3837936 Mon Sep 17 00:00:00 2001 From: Christophe Simonis Date: Fri, 9 Jun 2017 18:56:07 +0200 Subject: [PATCH] [FIX] website_crm_partner_assign: correctly check value changes Since fdc87498880b19754faf7fc57f456d6383bfbbf7, leads may have a default team set. Correctly check fields against previous values. --- .../tests/test_partner_assign.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/addons/website_crm_partner_assign/tests/test_partner_assign.py b/addons/website_crm_partner_assign/tests/test_partner_assign.py index d38f908ac4c..2efdfc76d0b 100644 --- a/addons/website_crm_partner_assign/tests/test_partner_assign.py +++ b/addons/website_crm_partner_assign/tests/test_partner_assign.py @@ -78,11 +78,14 @@ class TestPartnerLeadPortal(TestCrmCases): def test_partner_lead_accept(self): """ Test an integrating partner accepting the lead """ + team_before = self.lead.team_id + user_before = self.lead.user_id + self.lead.sudo(self.portal_user.id).partner_interested(comment="Oh yeah, I take that lead !") self.assertEqual(self.lead.type, 'opportunity', 'Bad Type: accepted lead by portal user should become an opportunity.') - self.assertFalse(self.lead.team_id, 'Accepting lead does not change the sales team.') - self.assertFalse(self.lead.user_id, 'Accepting lead does not change the salesman.') + self.assertEqual(self.lead.team_id, team_before, 'Accepting lead does not change the sales team.') + self.assertEqual(self.lead.user_id, user_before, 'Accepting lead does not change the salesman.') def test_partner_lead_decline(self): """ Test an integrating partner decline the lead """