From 1ebba3ea2736994d12366db2cc1cb35f2a72b96d Mon Sep 17 00:00:00 2001 From: "Nasreddin Boulif (bon)" Date: Thu, 30 Jun 2022 13:15:21 +0000 Subject: [PATCH] [FIX] website: fix test_01_get_current_website_id Cause: The test fails because there is some demo data that added additional website(s) and when trying to retrieve the current website without domain, it might retrieve the wrong one. Solution: Unlink unused website(s). opw-2899680 closes odoo/odoo#96810 X-original-commit: 45dfbae7301effe0d96163d67218a7854c3bde6f Signed-off-by: Romain Derie (rde) --- .../website/tests/test_get_current_website.py | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/addons/website/tests/test_get_current_website.py b/addons/website/tests/test_get_current_website.py index da36762acee..d40c21ebde8 100644 --- a/addons/website/tests/test_get_current_website.py +++ b/addons/website/tests/test_get_current_website.py @@ -8,13 +8,26 @@ from odoo.tests.common import TransactionCase @tagged('post_install', '-at_install') class TestGetCurrentWebsite(TransactionCase): + def setUp(self): + # Unlink unused website(s) to avoid messing with the expected results + self.website = self.env.ref('website.default_website') + for w in self.env['website'].search([('id', '!=', self.website.id)]): + try: + # Website are impossible to delete most often than not, as if + # there is critical business data linked to it, it will prevent + # the unlink. Could easily happen with a bridge module adding + # some custom data. + w.unlink() + except Exception: + pass + def test_01_get_current_website_id(self): """Make sure `_get_current_website_id works`.""" Website = self.env['website'] # clean initial state - website1 = self.env.ref('website.default_website') + website1 = self.website website1.domain = '' website1.country_group_ids = False @@ -116,7 +129,7 @@ class TestGetCurrentWebsite(TransactionCase): self.assertEqual(Website._get_current_website_id('site-1.com', False), website1.id) def test_02_signup_user_website_id(self): - website = self.env.ref('website.default_website') + website = self.website website.specific_user_account = True user = self.env['res.users'].create({'website_id': website.id, 'login': 'sad@mail.com', 'name': 'Hope Fully'})