diff --git a/base_phone/tests/__init__.py b/base_phone/tests/__init__.py deleted file mode 100644 index 03639ac..0000000 --- a/base_phone/tests/__init__.py +++ /dev/null @@ -1,3 +0,0 @@ -# -*- coding: utf-8 -*- - -from . import test_phone diff --git a/base_phone/tests/test_phone.py b/base_phone/tests/test_phone.py deleted file mode 100644 index 5afb37e..0000000 --- a/base_phone/tests/test_phone.py +++ /dev/null @@ -1,69 +0,0 @@ -# -*- coding: utf-8 -*- -# Copyright 2016-2018 Akretion France -# @author: Alexis de Lattre -# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). - - -from odoo.tests.common import TransactionCase - - -class TestPhone(TransactionCase): - - def test_phone(self): - company = self.env.ref('base.main_company') - fr_country_id = self.env.ref('base.fr').id - company.country_id = fr_country_id - rpo = self.env['res.partner'] - # Create an existing partner without country - partner1 = rpo.create({ - 'name': u'Pierre Paillet', - 'phone': '04-72-08-87-32', - 'mobile': '06.42.77.42.66', - 'fax': '(0) 1 45 42 12 42', - }) - self.assertEquals(partner1.phone, u'+33 4 72 08 87 32') - self.assertEquals(partner1.mobile, u'+33 6 42 77 42 66') - self.assertEquals(partner1.fax, u'+33 1 45 42 12 42') - # Create a partner with country - self.env.ref('base.res_partner_12').country_id =\ - self.env.ref('base.ch').id - partner2 = rpo.create({ - 'name': u'Joël Grand-Guillaume', - 'parent_id': self.env.ref('base.res_partner_12').id, - 'phone': '(0) 21 619 10 10', - 'mobile': '(0) 79 606 42 42', - }) - self.assertEquals(partner2.country_id, self.env.ref('base.ch')) - self.assertEquals(partner2.phone, u'+41 21 619 10 10') - self.assertEquals(partner2.mobile, u'+41 79 606 42 42') - # Write on an existing partner - agrolait = self.env.ref('base.res_partner_2') - self.assertEquals(agrolait.country_id, self.env.ref('base.be')) - agrolait.write({'phone': '(0) 2 391 43 74'}) - self.assertEquals(agrolait.phone, u'+32 2 391 43 74') - # Write on an existing partner with country at the same time - agrolait.write({ - 'fax': '04 72 89 32 43', - 'country_id': fr_country_id, - }) - self.assertEquals(agrolait.fax, u'+33 4 72 89 32 43') - # Write an invalid phone number - partner2.fax = '42' - self.assertEquals(partner2.fax, u'42') - # Test get_name_from_phone_number - pco = self.env['phone.common'] - name = pco.get_name_from_phone_number('0642774266') - self.assertEquals(name, 'Pierre Paillet') - name2 = pco.get_name_from_phone_number('0041216191010') - self.assertEquals(name2, u'Joël Grand-Guillaume (Camptocamp)') - # Test against the POS bug - # https://github.com/OCA/connector-telephony/issues/113 - # When we edit/create a partner from the POS, - # the country_id key in create(vals) is given as a string ! - partnerpos = rpo.create({ - 'name': u'POS customer', - 'phone': '04-72-08-87-42', - 'country_id': str(fr_country_id), - }) - self.assertEquals(partnerpos.phone, u'+33 4 72 08 87 42') - self.assertEquals(partnerpos.country_id.id, fr_country_id) diff --git a/crm_phone/tests/test_crm_phone.py b/crm_phone/tests/test_crm_phone.py index b76fa2d..d5d52c0 100644 --- a/crm_phone/tests/test_crm_phone.py +++ b/crm_phone/tests/test_crm_phone.py @@ -1,5 +1,4 @@ -# -*- coding: utf-8 -*- -# Copyright 2016-2018 Akretion France +# Copyright 2016-2019 Akretion France # @author: Alexis de Lattre # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). @@ -8,6 +7,67 @@ from odoo.tests.common import TransactionCase class TestCRMPhone(TransactionCase): + def test_phone(self): + company = self.env.ref('base.main_company') + fr_country_id = self.env.ref('base.fr').id + company.country_id = fr_country_id + rpo = self.env['res.partner'] + # Create an existing partner without country + partner1 = rpo.create({ + 'name': 'Pierre Paillet', + 'phone': '04-72-08-87-32', + 'mobile': '06.42.77.42.66', + }) + partner1._onchange_phone_validation() + partner1._onchange_mobile_validation() + self.assertEquals(partner1.phone, '+33 4 72 08 87 32') + self.assertEquals(partner1.mobile, '+33 6 42 77 42 66') + # Create a partner with country + self.env.ref('base.res_partner_12').country_id =\ + self.env.ref('base.ch').id + partner2 = rpo.create({ + 'name': 'Joël Grand-Guillaume', + 'parent_id': self.env.ref('base.res_partner_12').id, + 'phone': '(0) 21 619 10 10', + 'mobile': '(0) 79 606 42 42', + }) + partner2._onchange_phone_validation() + partner2._onchange_mobile_validation() + self.assertEquals(partner2.country_id, self.env.ref('base.ch')) + self.assertEquals(partner2.phone, '+41 21 619 10 10') + self.assertEquals(partner2.mobile, '+41 79 606 42 42') + # Write on an existing partner + agrolait = self.env.ref('base.res_partner_2') + self.assertEquals(agrolait.country_id, self.env.ref('base.be')) + agrolait.write({'phone': '(0) 2 391 43 74'}) + agrolait._onchange_phone_validation() + self.assertEquals(agrolait.phone, '+32 2 391 43 74') + # Write on an existing partner with country at the same time + agrolait.write({ + 'phone': '04 72 89 32 43', + 'country_id': fr_country_id, + }) + agrolait._onchange_phone_validation() + self.assertEquals(agrolait.phone, '+33 4 72 89 32 43') + # Test get_name_from_phone_number + pco = self.env['phone.common'] + name = pco.get_name_from_phone_number('0642774266') + self.assertEquals(name, 'Pierre Paillet') + name2 = pco.get_name_from_phone_number('0041216191010') + self.assertEquals(name2, 'Camptocamp, Joël Grand-Guillaume') + # Test against the POS bug + # https://github.com/OCA/connector-telephony/issues/113 + # When we edit/create a partner from the POS, + # the country_id key in create(vals) is given as a string ! + partnerpos = rpo.create({ + 'name': 'POS customer', + 'phone': '04-72-08-87-42', + 'country_id': str(fr_country_id), + }) + partnerpos._onchange_phone_validation() + self.assertEquals(partnerpos.phone, '+33 4 72 08 87 42') + self.assertEquals(partnerpos.country_id.id, fr_country_id) + def test_crm_phone(self): clo = self.env['crm.lead'] lead1 = clo.create({ @@ -18,23 +78,23 @@ class TestCRMPhone(TransactionCase): 'country_id': self.env.ref('base.fr').id, }) lead1._onchange_mobile_validation() - self.assertEquals(lead1.mobile, u'+33 6 42 77 42 77') + self.assertEquals(lead1.mobile, '+33 6 42 77 42 77') lead2 = clo.create({ - 'name': u'Automobile Odoo deployment', - 'partner_name': u'Kia', - 'contact_name': u'Mikaël Content', + 'name': 'Automobile Odoo deployment', + 'partner_name': 'Kia', + 'contact_name': 'Mikaël Content', 'country_id': self.env.ref('base.ch').id, 'phone': '04 31 23 45 67', }) lead2._onchange_phone_validation() - self.assertEquals(lead2.phone, u'+41 43 123 45 67') + self.assertEquals(lead2.phone, '+41 43 123 45 67') lead3 = clo.create({ 'name': 'Angela Strasse', 'country_id': self.env.ref('base.de').id, 'phone': '08912345678', }) lead3._onchange_phone_validation() - self.assertEquals(lead3.phone, u'+49 89 12345678') + self.assertEquals(lead3.phone, '+49 89 12345678') partner4 = self.env['res.partner'].create({ 'name': 'Belgian Guy', 'country_id': self.env.ref('base.be').id, @@ -45,7 +105,7 @@ class TestCRMPhone(TransactionCase): 'mobile': '(0) 2-391-43-75', }) lead4._onchange_mobile_validation() - self.assertEquals(lead4.mobile, u'+32 2 391 43 75') + self.assertEquals(lead4.mobile, '+32 2 391 43 75') pco = self.env['phone.common'] name = pco.get_name_from_phone_number('0642774277') self.assertEquals(name, 'Jacques Toufaux (Ford)')