0001-api-patch-booking-extra_data-51340.patch

Lauréline Guérin, 23 février 2021 09:01

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

Subject: [PATCH] api: patch booking extra_data (#51340)

 chrono/api/views.py |  9 ++++++-
 tests/test_api.py   | 58 +++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+), 1 deletion(-)

         class Meta:
             model = Booking
             fields = ['user_was_present', 'user_absence_reason']
             fields = ['user_was_present', 'user_absence_reason', 'extra_data']
         def validate_user_absence_reason(self, value):
             if value is None:
-...
                     status=status.HTTP_400_BAD_REQUEST,
+                )
             if 'extra_data' in serializer.validated_data:
                 extra_data = self.booking.extra_data or {}
                 extra_data.update(serializer.validated_data['extra_data'] or {})
                 serializer.validated_data['extra_data'] = extra_data
             serializer.save()
             if 'user_was_present' in request.data:
                 self.booking.secondary_booking_set.update(user_was_present=self.booking.user_was_present)
             if 'user_absence_reason' in request.data:
                 self.booking.secondary_booking_set.update(user_absence_reason=self.booking.user_absence_reason)
             if 'extra_data' in request.data:
                 self.booking.secondary_booking_set.update(extra_data=self.booking.extra_data)
             response = {'err': 0, 'booking_id': self.booking.pk}
             return Response(response)

         assert other_booking.user_absence_reason == ''  # not changed
     def test_booking_patch_api_extra_data(app, user):
         agenda = Agenda.objects.create(kind='events')
         event = Event.objects.create(agenda=agenda, start_datetime=now(), places=10)
         booking = Booking.objects.create(event=event, user_was_present=True)
         app.authorization = ('Basic', ('john.doe', 'password'))
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {'foo': 'bar'}})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': 'bar'}
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {'foo': 'baz'}})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': 'baz'}
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {'foooo': 'bar'}})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': 'baz', 'foooo': 'bar'}
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {'foooo': 'baz', 'foo': None}})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': None, 'foooo': 'baz'}
         # make secondary bookings
         Booking.objects.create(event=event, primary_booking=booking, user_was_present=False)
         Booking.objects.create(event=event, primary_booking=booking, user_was_present=False)
         # and other booking
         other_booking = Booking.objects.create(event=event)
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {'foooo': 'baz', 'foo': None}})
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': None, 'foooo': 'baz'}
         # all secondary bookings are upadted
         assert list(booking.secondary_booking_set.values_list('extra_data', flat=True)) == [
             {'foo': None, 'foooo': 'baz'},
             {'foo': None, 'foooo': 'baz'},
+        ]
         assert list(booking.secondary_booking_set.values_list('user_was_present', flat=True)) == [
             False,
             False,
         ]  # not changed
         other_booking.refresh_from_db()
         assert other_booking.extra_data is None  # not changed
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': None})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': None, 'foooo': 'baz'}  # not changed
         app.patch_json('/api/booking/%s/' % booking.pk, params={'extra_data': {}})
         booking.refresh_from_db()
         assert booking.user_was_present is True  # not changed
         assert booking.extra_data == {'foo': None, 'foooo': 'baz'}  # not changed
     def test_booking_cancellation_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-patch-booking-extra_data-51340.patch