Browse Source

Fix bug #154.

Conflicts:
	partner_firstname/__openerp__.py
	partner_firstname/tests/__init__.py

Fix flake8 error

After all, this was just a workaround. Remove it.

The real fix was in #171. I leave the tests.

Fix test that was having sentences after return.

PEP8 fix.
14.0
Jairo Llopis 9 years ago
committed by Luis Torres
parent
commit
402a40d201
  1. 5
      partner_firstname/__openerp__.py
  2. 9
      partner_firstname/tests/__init__.py
  3. 3
      partner_firstname/tests/base.py
  4. 5
      partner_firstname/tests/test_create.py
  5. 38
      partner_firstname/tests/test_delete.py

5
partner_firstname/__openerp__.py

@ -21,12 +21,13 @@
{
'name': 'Partner first name and last name',
'summary': "Split first name and last name for non company partners",
'version': '8.0.2.0.1',
'version': '8.0.2.1.0',
'author': "Camptocamp, Grupo ESOC, Odoo Community Association (OCA)",
"license": "AGPL-3",
'maintainer': 'Camptocamp, Acsone',
'category': 'Extra Tools',
'website': 'http://www.camptocamp.com, http://www.acsone.eu',
'website':
'http://www.camptocamp.com, http://www.acsone.eu, http://grupoesoc.es',
'depends': ['base'],
'data': [
'views/res_partner.xml',

9
partner_firstname/tests/__init__.py

@ -29,4 +29,11 @@
#
##############################################################################
from . import test_create, test_defaults, test_empty, test_name, test_onchange
from . import (
test_create,
test_defaults,
test_delete,
test_empty,
test_name,
test_onchange
)

3
partner_firstname/tests/base.py

@ -71,7 +71,8 @@ class BaseCase(TransactionCase, MailInstalled):
def test_copy(self):
"""Copy the partner and compare the result."""
self.expect(self.lastname, u"%s (copy)" % self.firstname)
self.changed = self.original.with_context(copy=True, lang="en_US").copy()
self.changed = (self.original.with_context(copy=True, lang="en_US")
.copy())
def test_one_name(self):
"""Test what happens when only one name is given."""

5
partner_firstname/tests/test_create.py

@ -62,8 +62,9 @@ class CompanyCase(PersonCase):
context = {"default_is_company": True}
def setUp(self):
return super(CompanyCase, self).setUp()
self.values.update(lastname=self.values["name"], firstname=False)
super(CompanyCase, self).setUp()
self.good_values.update(lastname=self.values["name"], firstname=False)
self.values = self.good_values.copy()
class UserCase(PersonCase, MailInstalled):

38
partner_firstname/tests/test_delete.py

@ -0,0 +1,38 @@
# -*- coding: utf-8 -*-
# © 2015 Grupo ESOC
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
from openerp.tests.common import TransactionCase
from .base import MailInstalled
class CompanyCase(TransactionCase):
model = "res.partner"
context = {"default_is_company": True}
def test_computing_after_unlink(self):
"""Test what happens if recomputed after unlinking.
This test might seem useless, but really this happens when module
``partner_relations`` is installed.
See https://github.com/OCA/partner-contact/issues/154.
"""
data = {"name": u"Söme name"}
record = self.env[self.model].with_context(**self.context).create(data)
record.unlink()
record.recompute()
class PersonCase(CompanyCase):
context = {"default_is_company": False}
class UserCase(CompanyCase, MailInstalled):
model = "res.users"
context = {"default_login": "user@example.com"}
def test_computing_after_unlink(self):
# Cannot create users if ``mail`` is installed
if not self.mail_installed():
super(UserCase, self).test_computing_after_unlink()
Loading…
Cancel
Save