fix: simplify reservation admin labels

This commit is contained in:
bisco
2026-04-30 00:47:36 +02:00
parent 47449ce8dd
commit 1629544b76
4 changed files with 81 additions and 2 deletions

View File

@@ -44,7 +44,9 @@ class Reservation(TimeStampedModel):
]
def __str__(self):
return f"{self.name} ({self.party_size}) for {self.performance}"
if self.party_size > 1:
return f"{self.name} ({self.party_size})"
return self.name
@property
def is_confirmed(self):

View File

@@ -160,3 +160,20 @@ class ReservationAdminTests(TestCase):
check_in = CheckIn.objects.get(reservation=reservation)
self.assertEqual(check_in.checked_in_by, self.admin_user)
self.assertEqual(check_in.source, CheckIn.Source.MANUAL)
def test_reservation_string_is_concise_without_performance_details(self):
single_guest_reservation = Reservation.objects.create(
performance=self.performance,
name="Maria Rossi",
email="maria@example.com",
party_size=1,
)
group_reservation = Reservation.objects.create(
performance=self.performance,
name="Luca Bianchi",
email="luca@example.com",
party_size=3,
)
self.assertEqual(str(single_guest_reservation), "Maria Rossi")
self.assertEqual(str(group_reservation), "Luca Bianchi (3)")