From c969ee7bef60dc42b303d979ab99a55dd6b7b2f0 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Sat, 4 Jul 2026 18:44:04 +0530 Subject: [PATCH] test: use separate documents for the invalid and valid slot cases --- .../test_appointment_booking_settings.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/erpnext/crm/doctype/appointment_booking_settings/test_appointment_booking_settings.py b/erpnext/crm/doctype/appointment_booking_settings/test_appointment_booking_settings.py index dd6b2cd5184..f4cab812daa 100644 --- a/erpnext/crm/doctype/appointment_booking_settings/test_appointment_booking_settings.py +++ b/erpnext/crm/doctype/appointment_booking_settings/test_appointment_booking_settings.py @@ -43,17 +43,17 @@ class TestAppointmentBookingSettings(ERPNextTestSuite): ) def test_validate_checks_every_slot(self): - doc = self.make_settings(appointment_duration=30) - doc.append( + bad = self.make_settings(appointment_duration=30) + bad.append( "availability_of_slots", {"day_of_week": "Monday", "from_time": "09:00:00", "to_time": "09:45:00"}, ) - self.assertRaises(frappe.ValidationError, doc.validate) + self.assertRaises(frappe.ValidationError, bad.validate) # a clean 60-minute slot passes end to end - doc.availability_of_slots = [] - doc.append( + good = self.make_settings(appointment_duration=30) + good.append( "availability_of_slots", {"day_of_week": "Monday", "from_time": "09:00:00", "to_time": "10:00:00"}, ) - doc.validate() + good.validate()