[REM] *: incorrect taggings which break tests when applied

Not entirely sure about TestAllocationRights. For TestEsEdiCommon issue is quite obviously that it's inherited by tests which are external, so when the `post_install_l10n` tag gets applied those tests get run during "normal" l10n and they break. closes odoo/odoo#98814 Related: odoo/enterprise#30825 Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
2022-09-05 08:33:13 +02:00
parent d256bbac3e
commit 5afcd06168
18 changed files with 0 additions and 21 deletions
@@ -9,7 +9,6 @@ import base64
 from lxml import etree


-@tagged('post_install', '-at_install')
 class AccountTestInvoicingCommon(TransactionCase):

    @classmethod
@@ -537,7 +536,6 @@ class AccountTestInvoicingCommon(TransactionCase):
        return etree.fromstring(xml_tree_str)


-@tagged('post_install', '-at_install')
 class AccountTestInvoicingHttpCommon(AccountTestInvoicingCommon, HttpCase):
    pass

@@ -3,7 +3,6 @@ from odoo.addons.account.tests.common import AccountTestInvoicingCommon
 from odoo.tests import tagged


-@tagged('post_install', '-at_install')
 class TestTaxCommon(AccountTestInvoicingCommon):

    @classmethod
@@ -13,7 +13,6 @@ from odoo.tests.common import tagged
 from odoo.tools import mute_logger


-@tagged('lead_assign')
 class TestLeadAssignCommon(TestLeadConvertCommon):

    @classmethod
@@ -10,7 +10,6 @@ from odoo.tests.common import tagged, users
 from odoo.tools import mute_logger


-@tagged('lead_manage')
 class TestLeadMergeCommon(TestLeadConvertMassCommon):
    """ During a mixed merge (involving leads and opps), data should be handled a certain way following their type
    (m2o, m2m, text, ...). """
@@ -7,7 +7,6 @@ from odoo.exceptions import AccessError, UserError
 import time


-@tests.tagged('access_rights', 'post_install', '-at_install')
 class TestAllocationRights(TestHrHolidaysCommon):

    @classmethod
@@ -10,7 +10,6 @@ from odoo.tools.misc import file_open
 from odoo.tests import tagged


-@tagged('post_install_l10n', 'post_install', '-at_install')
 class TestUBLCommon(AccountEdiTestCommon):

    @classmethod
@@ -6,7 +6,6 @@ from odoo.tests import tagged
 from odoo.addons.account_edi.tests.common import AccountEdiTestCommon


-@tagged('post_install_l10n', 'post_install', '-at_install')
 class TestEGEdiCommon(AccountEdiTestCommon):

    @classmethod
@@ -3,12 +3,10 @@ import base64
 from pytz import timezone
 from datetime import datetime

-from odoo.tests import tagged
 from odoo.tools import misc
 from odoo.addons.account_edi.tests.common import AccountEdiTestCommon


-@tagged('post_install_l10n', 'post_install', '-at_install')
 class TestEsEdiCommon(AccountEdiTestCommon):

    @classmethod
@@ -13,7 +13,6 @@ from odoo.exceptions import UserError

 _logger = logging.getLogger(__name__)

-@tagged('post_install_l10n', 'post_install', '-at_install')
 class TestItEdiCommon(AccountEdiTestCommon):

    @classmethod
@@ -11,7 +11,6 @@ import logging

 _logger = logging.getLogger(__name__)

-@tagged('post_install', '-at_install')
 class TestPointOfSaleCommon(ValuationReconciliationTestCommon):

    @classmethod
@@ -132,7 +131,6 @@ class TestPointOfSaleCommon(ValuationReconciliationTestCommon):
        (invoice_rep_lines | refund_rep_lines).write({'account_id': cls.company_data['default_account_tax_sale'].id})


-@tagged('post_install', '-at_install')
 class TestPoSCommon(ValuationReconciliationTestCommon):
    """ Set common values for different special test cases.

@@ -3,7 +3,6 @@
 from odoo.tests import tagged, common


-@tagged('post_install', '-at_install')
 class TestMatrixCommon(common.HttpCase):

    def setUp(self):
@@ -4,7 +4,6 @@ from odoo.addons.stock_account.tests.test_anglo_saxon_valuation_reconciliation_c
 from odoo.tests import Form, tagged


-@tagged('post_install', '-at_install')
 class TestValuationReconciliationCommon(ValuationReconciliationTestCommon):

    @classmethod
@@ -6,7 +6,6 @@ from odoo.addons.stock_account.tests.test_stockvaluation import _create_accounti
 from odoo.tests.common import tagged, Form


-@tagged("post_install", "-at_install")
 class TestAccountMoveStockCommon(AccountTestInvoicingCommon):
    @classmethod
    def setUpClass(cls, chart_template_ref=None):
@@ -5,7 +5,6 @@ from odoo.tests import tagged
 from odoo import fields


-@tagged('-at_install', 'post_install')
 class ValuationReconciliationTestCommon(AccountTestInvoicingCommon):
    """ Base class for tests checking interim accounts reconciliation works
    in anglosaxon accounting. It sets up everything we need in the tests, and is
@@ -11,7 +11,6 @@ from odoo.tests import tagged, users
 from odoo.tools import mute_logger


-@tagged('mail_template', 'multi_lang')
 class TestMailTemplateCommon(TestMailCommon, TestRecipients):

    @classmethod
@@ -21,7 +21,6 @@ from odoo.tools import mute_logger, formataddr
 from odoo.tests.common import users


-@tagged('mail_post')
 class TestMessagePostCommon(TestMailCommon, TestRecipients):

    @classmethod
@@ -5,7 +5,6 @@ from odoo.tests import common
 from odoo.tests.common import tagged


-@tagged('post_install', '-at_install', 'utm_consistency')
 class TestUTMCommon(common.TransactionCase):

    @classmethod
@@ -11,7 +11,6 @@ from odoo.tools import config, file_open
 from .test_common import TestHttpBase


-@tagged('post_install', '-at_install')
 class TestHttpStaticCommon(TestHttpBase):
    @classmethod
    def setUpClass(cls):