0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch

Lauréline Guérin, 25 mai 2020 10:32

Voir les différences: en ligne côte à côte

Subject: [PATCH 1/5] api: limit accept/suspend/resize endpoints to events
 agenda (#43077)

 chrono/api/views.py |  8 +++---
 tests/test_api.py   | 66 +++++++++++++++++++++++++++++++++------------
 2 files changed, 53 insertions(+), 21 deletions(-)

                 response['api']['accept_url'] = request.build_absolute_uri(
                     reverse('api-accept-booking', kwargs={'booking_pk': primary_booking.id})
+                )
             else:
             elif agenda.kind == 'events':
                 response['api']['suspend_url'] = request.build_absolute_uri(
                     reverse('api-suspend-booking', kwargs={'booking_pk': primary_booking.pk})
+                )
-...
         permission_classes = (permissions.IsAuthenticated,)
         def post(self, request, booking_pk=None, format=None):
             booking = get_object_or_404(Booking, id=booking_pk)
             booking = get_object_or_404(Booking, pk=booking_pk, event__agenda__kind='events')
             if booking.cancellation_datetime:
                 response = {
                     'err': 1,
-...
         permission_classes = (permissions.IsAuthenticated,)
         def post(self, request, booking_pk=None, format=None):
             booking = get_object_or_404(Booking, pk=booking_pk)
             booking = get_object_or_404(Booking, pk=booking_pk, event__agenda__kind='events')
             if booking.cancellation_datetime:
                 response = {
                     'err': 1,
-...
+                )
             payload = serializer.validated_data
             booking = get_object_or_404(Booking, pk=booking_pk)
             booking = get_object_or_404(Booking, pk=booking_pk, event__agenda__kind='events')
             event = booking.event
             if booking.cancellation_datetime:
                 response = {

         assert secondary.cancellation_datetime is not None
     def test_accept_booking(app, some_data, user):
         agenda_id = Agenda.objects.filter(label=u'Foo bar')[0].id
         event = Event.objects.filter(agenda_id=agenda_id).exclude(start_datetime__lt=now())[0]
         event.waiting_list_places = 5
         event.save()
     def test_accept_booking(app, user):
         agenda = Agenda.objects.create(kind='events')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=20, waiting_list_places=5)
         # create a booking on the waiting list
         primary = Booking.objects.create(event=event, in_waiting_list=True)
-...
         assert resp.json['overbooked_places'] == 1
     def test_suspend_booking(app, some_data, user):
         agenda_id = Agenda.objects.filter(label=u'Foo bar')[0].id
         event = Event.objects.filter(agenda_id=agenda_id).exclude(start_datetime__lt=now())[0]
         event.waiting_list_places = 5
         event.save()
     def test_accept_booking_non_event_agenda(app, user):
         app.authorization = ('Basic', ('john.doe', 'password'))
         agenda = Agenda.objects.create(kind='meetings')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=1)
         booking = Booking.objects.create(event=event)
         app.post('/api/booking/%s/accept/' % booking.pk, status=404)
         agenda.kind = 'virtual'
         agenda.save()
         app.post('/api/booking/%s/accept/' % booking.pk, status=404)
     def test_suspend_booking(app, user):
         agenda = Agenda.objects.create(kind='events')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=20, waiting_list_places=5)
         # create a booking not on the waiting list
         primary = Booking.objects.create(event=event, in_waiting_list=False)
-...
         assert primary.in_waiting_list is False
     def test_resize_booking(app, some_data, user):
         agenda_id = Agenda.objects.filter(label=u'Foo bar')[0].id
         event = Event.objects.filter(agenda_id=agenda_id).exclude(start_datetime__lt=now())[0]
         event.places = 5
         event.waiting_list_places = 5
         event.save()
         event2 = Event.objects.filter(agenda_id=agenda_id).exclude(start_datetime__lt=now())[1]
     def test_suspend_booking_non_event_agenda(app, user):
         app.authorization = ('Basic', ('john.doe', 'password'))
         agenda = Agenda.objects.create(kind='meetings')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=1)
         booking = Booking.objects.create(event=event)
         app.post('/api/booking/%s/suspend/' % booking.pk, status=404)
         agenda.kind = 'virtual'
         agenda.save()
         app.post('/api/booking/%s/suspend/' % booking.pk, status=404)
     def test_resize_booking(app, user):
         agenda = Agenda.objects.create(kind='events')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=5, waiting_list_places=5)
         event2 = Event.objects.create(agenda=agenda, start_datetime=now(), places=20, waiting_list_places=0)
         # create a booking not on the waiting list
         primary = Booking.objects.create(event=event, in_waiting_list=False)
-...
         assert resp.json['err'] == 1
     def test_resize_booking_non_event_agenda(app, user):
         app.authorization = ('Basic', ('john.doe', 'password'))
         agenda = Agenda.objects.create(kind='meetings')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=1)
         booking = Booking.objects.create(event=event)
         app.post('/api/booking/%s/resize/' % booking.pk, params={'count': 42}, status=404)
         agenda.kind = 'virtual'
         agenda.save()
         app.post('/api/booking/%s/resize/' % booking.pk, params={'count': 42}, status=404)
     def test_multiple_booking_api(app, some_data, user):
         agenda = Agenda.objects.filter(label=u'Foo bar')[0]
         event = [x for x in Event.objects.filter(agenda=agenda) if x.in_bookable_period()][0]
+    -

Projet

Général

Profil

Produits Entr'ouvert » Chrono

0001-api-limit-accept-suspend-resize-endpoints-to-events-.patch