diff --git a/addons/account/tests/account_test_classes.py b/addons/account/tests/account_test_classes.py index 01a3afd891d..b3ff7d54337 100644 --- a/addons/account/tests/account_test_classes.py +++ b/addons/account/tests/account_test_classes.py @@ -1,44 +1,18 @@ -import logging - +# -*- coding: utf-8 -*- from openerp.tests.common import TransactionCase -from openerp import SUPERUSER_ID -from openerp import api -from openerp.modules.registry import RegistryManager -from openerp.tools import config -DB = config['db_name'] - -_logger = logging.getLogger(__name__) class AccountingTestCase(TransactionCase): """ This class extends the base TransactionCase, in order to test the accounting with localization setups. It is configured to run the tests after - the installation of all modules, and will SKIP TESTS and return a success if - it cannot find an already configured accounting (which means no localization - module has been installed). + the installation of all modules, and will SKIP TESTS ifit cannot find an already + configured accounting (which means no localization module has been installed). """ post_install = True at_install = False - def _check_accounting_configured(self): - # minimal setUp() similar to the one of TransactionCase, although we - # can't use the regular setUp() because we want to assume the accounting - # is already configured when making the setUp of tests. - self.registry = RegistryManager.get(DB) - self.cr = self.cursor() - self.uid = SUPERUSER_ID - self.env = api.Environment(self.cr, self.uid, {}) - # having at least one account created means the accounting is ready to be tested + def setUp(self): + super(AccountingTestCase, self).setUp() domain = [('company_id', '=', self.env.ref('base.main_company').id)] - res = bool(self.env['account.account'].search_count(domain)) - # rollback and close the cursor, and reset the environment - self.tearDown() - return res - - def run(self, result=None): - if not self._check_accounting_configured(): - # no chart of account is available, so skip the tests and consider - # it as a success - _logger.warning("No chart of account, skip %s", self) - return - super(AccountingTestCase, self).run(result=result) + if not self.env['account.account'].search_count(domain): + self.skipTest("No Chart of account found")