From 8645e31863fa76a737ce5cf79a41c95ce7853507 Mon Sep 17 00:00:00 2001 From: Adrian Torres Date: Fri, 3 Jan 2020 09:48:42 +0000 Subject: [PATCH] [FIX] website_sale: do not pollute the environment Instead of polluting an existing environment, use the method that was intended for changing the company `with_company` which returns a brand new environment and leaves the existing one unchanged. --- addons/website_sale/tests/test_website_sale_pricelist.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/addons/website_sale/tests/test_website_sale_pricelist.py b/addons/website_sale/tests/test_website_sale_pricelist.py index 36b742a6782..3782e51244f 100644 --- a/addons/website_sale/tests/test_website_sale_pricelist.py +++ b/addons/website_sale/tests/test_website_sale_pricelist.py @@ -427,10 +427,8 @@ class TestWebsitePriceListMultiCompany(TransactionCaseWithUserDemo): 'company_id': self.company2.id, 'website_id': False, }) - self.demo_user.partner_id.property_product_pricelist = self.c1_pl - # Switch env.user company to create ir.property in company2 - self.env.user.company_id = self.company2 - self.demo_user.partner_id.property_product_pricelist = self.c2_pl + self.demo_user.partner_id.with_company(self.company1.id).property_product_pricelist = self.c1_pl + self.demo_user.partner_id.with_company(self.company2.id).property_product_pricelist = self.c2_pl # Ensure everything was done correctly self.assertEqual(self.demo_user.partner_id.with_company(self.company1.id).property_product_pricelist, self.c1_pl)