From 124d1db26b3f13f3b4669896699a4b9f3e16eb49 Mon Sep 17 00:00:00 2001 From: "Pedro M. Baeza" Date: Mon, 13 Apr 2020 19:02:39 +0200 Subject: [PATCH] [IMP] report_qweb_parameter: Avoid error on integration tests If reusing existing records, there's the risk of conflicting tests, like modifying the VAT for the existing company and that company having a country with VAT check. We avoid it creating a new company instead. --- .../tests/test_report_qweb_parameter.py | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/report_qweb_parameter/tests/test_report_qweb_parameter.py b/report_qweb_parameter/tests/test_report_qweb_parameter.py index df61face..77e2b754 100644 --- a/report_qweb_parameter/tests/test_report_qweb_parameter.py +++ b/report_qweb_parameter/tests/test_report_qweb_parameter.py @@ -13,18 +13,13 @@ class TestReportQWebParameter(common.TransactionCase): report_name = 'report_qweb_parameter.test_report_length' report_obj = self.env['ir.actions.report'] report_object = report_obj._get_report_from_name(report_name) - - docs = self.env['res.company'].search([], limit=1) - vat = docs.vat - website = docs.website - street = docs.street - company_registry = docs.company_registry - docs.update({ + docs = self.env['res.company'].create({ + 'name': 'Test company', 'street': '12345678901', 'vat': '12345678901', - 'website': '1234567890', 'company_registry': '1234567890' }) + docs.website = '1234567890' # for avoding that Odoo adds http:// rep = report_object.render(docs.ids, False) root = ET.fromstring(rep[0]) self.assertEqual(root[0].text, "1234567890") @@ -42,9 +37,3 @@ class TestReportQWebParameter(common.TransactionCase): {'website': '1234567890', 'company_registry': '12345678901'}) with self.assertRaises(QWebException): report_object.render(docs.ids, False) - docs.update({ - 'street': street, - 'vat': vat, - 'website': website, - 'company_registry': company_registry - })