[REF] base_address_extended: clean and improve tests
Purpose is to make tests easier to understand and more data oriented. We now understand more clearly what is an input and what is expected. Tests about company / partner address fields are added to ensure coherency. Task ID 2158302 PR odoo/odoo#42678
This commit is contained in:
Thibault Delavallée
@@ -1,6 +1,7 @@
|
||||
# -*- coding: utf-8 -*-
|
||||
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
||||
|
||||
from odoo import models
|
||||
from odoo.tests.common import SavepointCase
|
||||
|
||||
|
||||
@@ -15,44 +16,153 @@ class TestStreetFields(SavepointCase):
|
||||
cls.env.ref('base.ch').write({'street_format': 'header %(street_name)s, %(street_number)s - %(street_number2)s trailer'})
|
||||
cls.env.ref('base.mx').write({'street_format': '%(street_name)s %(street_number)s/%(street_number2)s'})
|
||||
|
||||
def create_and_assert(self, partner_name, country_id, street, street_name, street_number, street_number2):
|
||||
partner = self.Partner.create({'name': partner_name + '-1', 'street': street, 'country_id': country_id})
|
||||
self.assertEqual(partner.street_name or '', street_name, 'wrong street name for %s: %s' % (partner_name, partner.street_name))
|
||||
self.assertEqual(partner.street_number or '', street_number, 'wrong house number for %s: %s' % (partner_name, partner.street_number))
|
||||
self.assertEqual(partner.street_number2 or '', street_number2, 'wrong door number for %s: %s' % (partner_name, partner.street_number2))
|
||||
partner = self.Partner.create({
|
||||
'name': partner_name + '-2',
|
||||
'street_name': street_name,
|
||||
'street_number': street_number,
|
||||
'street_number2': street_number2,
|
||||
'country_id': country_id,
|
||||
def assertStreetVals(self, record, street_data):
|
||||
for key, val in street_data.items():
|
||||
if key not in ['street', 'street_name', 'street_number', 'street_number2', 'name', 'city', 'country_id']:
|
||||
continue
|
||||
if isinstance(record[key], models.BaseModel):
|
||||
self.assertEqual(record[key].id, val, 'Wrongly formatted street field %s: expected %s, received %s' % (key, val, record[key]))
|
||||
else:
|
||||
self.assertEqual(record[key], val, 'Wrongly formatted street field %s: expected %s, received %s' % (key, val, record[key]))
|
||||
|
||||
def test_company_create(self):
|
||||
""" Will test the compute and inverse methods of street fields when creating partner records. """
|
||||
us_id = self.env.ref('base.us').id
|
||||
mx_id = self.env.ref('base.mx').id
|
||||
ch_id = self.env.ref('base.ch').id
|
||||
input_values = [
|
||||
{'country_id': us_id, 'street': '40/2b Chaussee de Namur'},
|
||||
{'country_id': us_id, 'street': '40 Chaussee de Namur'},
|
||||
{'country_id': us_id, 'street': 'Chaussee de Namur'},
|
||||
{'country_id': mx_id, 'street': 'Av. Miguel Hidalgo y Costilla 601'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur, 40 - 2b trailer'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur, 40 trailer'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur trailer'},
|
||||
]
|
||||
expected = [
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': False},
|
||||
{'street_name': 'de Namur', 'street_number': 'Chaussee', 'street_number2': False},
|
||||
{'street_name': 'Av.', 'street_number': 'Miguel Hidalgo y Costilla 601', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': False, 'street_number2': False}
|
||||
]
|
||||
|
||||
# test street -> street values (compute)
|
||||
for idx, (company_values, expected_vals) in enumerate(zip(input_values, expected)):
|
||||
company_values['name'] = 'Test-%2d' % idx
|
||||
company = self.env['res.company'].create(company_values)
|
||||
self.assertStreetVals(company, expected_vals)
|
||||
self.assertStreetVals(company.partner_id, expected_vals)
|
||||
|
||||
# test street_values -> street (inverse)
|
||||
for idx, (company_values, expected_vals) in enumerate(zip(input_values, expected)):
|
||||
company_values['name'] = 'TestNew-%2d' % idx
|
||||
expected_street = company_values.pop('street')
|
||||
company_values.update(expected_vals)
|
||||
company = self.env['res.company'].create(company_values)
|
||||
self.assertEqual(company.street, expected_street)
|
||||
self.assertStreetVals(company, company_values)
|
||||
self.assertEqual(company.partner_id.street, expected_street)
|
||||
self.assertStreetVals(company.partner_id, company_values)
|
||||
|
||||
def test_company_write(self):
|
||||
""" Will test the compute and inverse methods of street fields when updating partner records. """
|
||||
be_id = self.env.ref('base.be').id
|
||||
company = self.env['res.company'].create({
|
||||
'name': 'Test',
|
||||
'country_id': be_id,
|
||||
'street': 'Chaussee de Namur, 40/2b'
|
||||
})
|
||||
self.assertEqual(partner.street or '', street, 'wrong street for %s: %s' % (partner_name, partner.street))
|
||||
return partner
|
||||
self.assertStreetVals(company, {'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'})
|
||||
|
||||
def write_and_assert(self, partner, vals, street, street_name, street_number, street_number2):
|
||||
partner.write(vals)
|
||||
self.assertEqual(partner.street_name or '', street_name, 'wrong street name: %s' % partner.street_name)
|
||||
self.assertEqual(partner.street_number or '', street_number, 'wrong house number: %s' % partner.street_number)
|
||||
self.assertEqual(partner.street_number2 or '', street_number2, 'wrong door number: %s' % partner.street_number2)
|
||||
self.assertEqual(partner.street or '', street, 'wrong street: %s' % partner.street)
|
||||
input_values = [
|
||||
{'street': 'Chaussee de Namur, 43'},
|
||||
{'street': 'Chaussee de Namur'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40'},
|
||||
{'street_number2': '4'},
|
||||
{'country_id': self.env.ref('base.us').id},
|
||||
]
|
||||
expected = [
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '43', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': False, 'street_number2': False},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': False, 'street': 'Chee de Namur, 40'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': '4', 'street': 'Chee de Namur, 40/4'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': '4', 'street': '40/4 Chee de Namur'},
|
||||
]
|
||||
|
||||
def test_00_res_partner_name_create(self):
|
||||
self.create_and_assert('Test00', self.env.ref('base.us').id, '40/2b Chaussee de Namur', 'Chaussee de Namur', '40', '2b')
|
||||
self.create_and_assert('Test01', self.env.ref('base.us').id, '40 Chaussee de Namur', 'Chaussee de Namur', '40', '')
|
||||
self.create_and_assert('Test02', self.env.ref('base.us').id, 'Chaussee de Namur', 'de Namur', 'Chaussee', '')
|
||||
self.create_and_assert('Test03', self.env.ref('base.mx').id, 'Av. Miguel Hidalgo y Costilla 601', 'Av.', 'Miguel Hidalgo y Costilla 601', '')
|
||||
# test both compute and inverse (could probably be pimp)
|
||||
for write_values, expected_vals in zip(input_values, expected):
|
||||
company.write(write_values)
|
||||
self.assertStreetVals(company, expected_vals)
|
||||
self.assertStreetVals(company.partner_id, expected_vals)
|
||||
|
||||
def test_01_header_trailer(self):
|
||||
self.create_and_assert('Test10', self.env.ref('base.ch').id, 'header Chaussee de Namur, 40 - 2b trailer', 'Chaussee de Namur', '40', '2b')
|
||||
self.create_and_assert('Test11', self.env.ref('base.ch').id, 'header Chaussee de Namur, 40 trailer', 'Chaussee de Namur', '40', '')
|
||||
self.create_and_assert('Test12', self.env.ref('base.ch').id, 'header Chaussee de Namur trailer', 'Chaussee de Namur', '', '')
|
||||
def test_partner_create(self):
|
||||
""" Will test the compute and inverse methods of street fields when creating partner records. """
|
||||
us_id = self.env.ref('base.us').id
|
||||
mx_id = self.env.ref('base.mx').id
|
||||
ch_id = self.env.ref('base.ch').id
|
||||
input_values = [
|
||||
{'country_id': us_id, 'street': '40/2b Chaussee de Namur'},
|
||||
{'country_id': us_id, 'street': '40 Chaussee de Namur'},
|
||||
{'country_id': us_id, 'street': 'Chaussee de Namur'},
|
||||
{'country_id': mx_id, 'street': 'Av. Miguel Hidalgo y Costilla 601'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur, 40 - 2b trailer'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur, 40 trailer'},
|
||||
{'country_id': ch_id, 'street': 'header Chaussee de Namur trailer'},
|
||||
]
|
||||
expected = [
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': False},
|
||||
{'street_name': 'de Namur', 'street_number': 'Chaussee', 'street_number2': False},
|
||||
{'street_name': 'Av.', 'street_number': 'Miguel Hidalgo y Costilla 601', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': False, 'street_number2': False}
|
||||
]
|
||||
|
||||
def test_02_res_partner_write(self):
|
||||
p1 = self.create_and_assert('Test20', self.env.ref('base.be').id, 'Chaussee de Namur, 40/2b', 'Chaussee de Namur', '40', '2b')
|
||||
self.write_and_assert(p1, {'street': 'Chaussee de Namur, 43'}, 'Chaussee de Namur, 43', 'Chaussee de Namur', '43', '')
|
||||
self.write_and_assert(p1, {'street': 'Chaussee de Namur'}, 'Chaussee de Namur', 'Chaussee de Namur', '', '')
|
||||
self.write_and_assert(p1, {'street_name': 'Chee de Namur', 'street_number': '40'}, 'Chee de Namur, 40', 'Chee de Namur', '40', '')
|
||||
self.write_and_assert(p1, {'street_number2': '4'}, 'Chee de Namur, 40/4', 'Chee de Namur', '40', '4')
|
||||
#we do recompute the street fields when we change the country
|
||||
self.write_and_assert(p1, {'country_id': self.env.ref('base.us').id}, '40/4 Chee de Namur', 'Chee de Namur', '40', '4')
|
||||
# test street -> street values (compute)
|
||||
for partner_values, expected_vals in zip(input_values, expected):
|
||||
partner_values['name'] = 'Test'
|
||||
partner = self.env['res.partner'].create(partner_values)
|
||||
self.assertStreetVals(partner, expected_vals)
|
||||
|
||||
# test street_values -> street (inverse)
|
||||
for partner_values, expected_vals in zip(input_values, expected):
|
||||
partner_values['name'] = 'Test'
|
||||
expected_street = partner_values.pop('street')
|
||||
partner_values.update(expected_vals)
|
||||
partner = self.env['res.partner'].create(partner_values)
|
||||
self.assertEqual(partner.street, expected_street)
|
||||
self.assertStreetVals(partner, partner_values)
|
||||
|
||||
def test_partner_write(self):
|
||||
""" Will test the compute and inverse methods of street fields when updating partner records. """
|
||||
be_id = self.env.ref('base.be').id
|
||||
partner = self.env['res.partner'].create({
|
||||
'name': 'Test',
|
||||
'country_id': be_id,
|
||||
'street': 'Chaussee de Namur, 40/2b'
|
||||
})
|
||||
self.assertStreetVals(partner, {'street_name': 'Chaussee de Namur', 'street_number': '40', 'street_number2': '2b'})
|
||||
|
||||
input_values = [
|
||||
{'street': 'Chaussee de Namur, 43'},
|
||||
{'street': 'Chaussee de Namur'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40'},
|
||||
{'street_number2': '4'},
|
||||
{'country_id': self.env.ref('base.us').id},
|
||||
]
|
||||
expected = [
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': '43', 'street_number2': False},
|
||||
{'street_name': 'Chaussee de Namur', 'street_number': False, 'street_number2': False},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': False, 'street': 'Chee de Namur, 40'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': '4', 'street': 'Chee de Namur, 40/4'},
|
||||
{'street_name': 'Chee de Namur', 'street_number': '40', 'street_number2': '4', 'street': '40/4 Chee de Namur'},
|
||||
]
|
||||
|
||||
# test both compute and inverse (could probably be pimp)
|
||||
for write_values, expected_vals in zip(input_values, expected):
|
||||
partner.write(write_values)
|
||||
self.assertStreetVals(partner, expected_vals)
|
||||
|
||||
Reference in New Issue
Block a user