From e6f891d80ea6196cb26f7aad2408450d8ca5da56 Mon Sep 17 00:00:00 2001 From: wan Date: Thu, 6 Jan 2022 10:11:50 +0000 Subject: [PATCH] [FIX] base: generic test for reports for l10n too MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A lot of localization are inheriting reports and changing them. This should be tested in a generic test like /base.test_reports to avoid possible tracebacks. closes odoo/odoo#82312 Related: odoo/enterprise#23299 Signed-off-by: William André (wan) --- odoo/addons/base/tests/test_reports.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/odoo/addons/base/tests/test_reports.py b/odoo/addons/base/tests/test_reports.py index 3ba9a669df6..b783b4de671 100644 --- a/odoo/addons/base/tests/test_reports.py +++ b/odoo/addons/base/tests/test_reports.py @@ -9,7 +9,7 @@ import odoo.tests _logger = logging.getLogger(__name__) -@odoo.tests.tagged('post_install', '-at_install') +@odoo.tests.tagged('post_install', '-at_install', 'post_install_l10n') class TestReports(odoo.tests.TransactionCase): def test_reports(self): domain = [('report_type', 'like', 'qweb')] @@ -24,10 +24,12 @@ class TestReports(odoo.tests.TransactionCase): report_records = report_model.search([], limit=10) if not report_records: _logger.info("no record found skipping report %s", report.report_name) - if not report.multi: - report_records = report_records[:1] # Test report generation - report._render_qweb_html(report_records.ids) + if not report.multi: + for record in report_records: + report._render_qweb_html(record.ids) + else: + report._render_qweb_html(report_records.ids) else: continue