Skip to content

Commit 036df6d

Browse files
CopilotCRogos
andauthored
Fix tests: replace base.res_partner_1/2 demo refs with programmatic partners
Agent-Logs-Url: https://github.com/c4a8-odoo/module-oca-sale-workflow/sessions/dfaa91f3-0425-431b-b1d5-95e9d06f6310 Co-authored-by: CRogos <1799080+CRogos@users.noreply.github.com>
1 parent fdc0b4a commit 036df6d

1 file changed

Lines changed: 15 additions & 11 deletions

File tree

sale_product_set/tests/test_product_set.py

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,17 @@ def setUpClass(cls):
108108
}
109109
)
110110

111+
# Create partners (self-contained, no dependency on demo data)
112+
cls.partner_1 = cls.env["res.partner"].create({"name": "Test Partner 1"})
113+
cls.partner_2 = cls.env["res.partner"].create({"name": "Test Partner 2"})
114+
111115
# Create a test product and sale order
112116
test_product = cls.env["product.product"].create(
113117
{"name": "Test Product For SO", "type": "consu"}
114118
)
115119
cls.so = cls.so_model.create(
116120
{
117-
"partner_id": cls.env.ref("base.res_partner_1").id,
121+
"partner_id": cls.partner_1.id,
118122
"order_line": [
119123
(
120124
0,
@@ -212,39 +216,39 @@ def test_delete_set(self):
212216
self.assertFalse(wiz.exists())
213217

214218
def test_add_set_on_empty_so(self):
215-
so = self.so_model.create({"partner_id": self.ref("base.res_partner_1")})
219+
so = self.so_model.create({"partner_id": self.partner_1.id})
216220
wiz = self._get_wiz(order_id=so.id)
217221
wiz.add_set()
218222
self.assertEqual(len(so.order_line), 3)
219223

220224
def test_add_set_non_matching_partner(self):
221-
so = self.so_model.create({"partner_id": self.ref("base.res_partner_1")})
222-
self.product_set.partner_id = self.ref("base.res_partner_2")
225+
so = self.so_model.create({"partner_id": self.partner_1.id})
226+
self.product_set.partner_id = self.partner_2.id
223227
wiz = self._get_wiz(order_id=so.id)
224228
with self.assertRaises(exceptions.ValidationError):
225229
wiz.add_set()
226230

227231
def test_add_set_non_matching_partner_ctx_bypass(self):
228-
so = self.so_model.create({"partner_id": self.ref("base.res_partner_1")})
232+
so = self.so_model.create({"partner_id": self.partner_1.id})
229233
self.assertEqual(len(so.order_line), 0)
230-
self.product_set.partner_id = self.ref("base.res_partner_2")
234+
self.product_set.partner_id = self.partner_2.id
231235
wiz = self._get_wiz(order_id=so.id).with_context(
232236
product_set_add_skip_validation=True
233237
)
234238
wiz.add_set()
235239
self.assertEqual(len(so.order_line), 3)
236240

237241
def test_add_set_non_matching_partner_ctx_override(self):
238-
so = self.so_model.create({"partner_id": self.ref("base.res_partner_1")})
242+
so = self.so_model.create({"partner_id": self.partner_1.id})
239243
self.assertEqual(len(so.order_line), 0)
240244
wiz = self._get_wiz(order_id=so.id).with_context(
241-
allowed_order_partner_ids=[self.ref("base.res_partner_2")]
245+
allowed_order_partner_ids=[self.partner_2.id]
242246
)
243247
wiz.add_set()
244248
self.assertEqual(len(so.order_line), 3)
245249

246250
def test_add_set_no_update_existing_products(self):
247-
so = self.so_model.create({"partner_id": self.ref("base.res_partner_1")})
251+
so = self.so_model.create({"partner_id": self.partner_1.id})
248252
wiz = self._get_wiz(order_id=so.id)
249253
wiz.add_set()
250254
self.assertEqual(len(so.order_line), 3)
@@ -319,8 +323,8 @@ def test_add_set_section(self):
319323
)
320324

321325
def test_product_set_toggle_active(self):
322-
partner_1 = self.env.ref("base.res_partner_1")
323-
partner_2 = self.env.ref("base.res_partner_2")
326+
partner_1 = self.partner_1
327+
partner_2 = self.partner_2
324328

325329
self.product_set_2 = self.product_set.copy()
326330
self.product_set_3 = self.product_set.copy()

0 commit comments

Comments
 (0)