0001-api_views-provide-a-default-slug-for-ous-22250.patch

Paul Marillonnet, 14 septembre 2020 17:31

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

Subject: [PATCH] api_views: provide a default slug for ous (#22250)

 src/authentic2/api_views.py | 19 +++++++++++++++++++
 tests/test_api.py           | 22 ++++++++++++++++++++++
 2 files changed, 41 insertions(+)

     from django.contrib.auth.hashers import identify_hasher
     from django.core.exceptions import MultipleObjectsReturned
     from django.utils.translation import ugettext as _
     from django.utils.text import slugify
     from django.utils.encoding import force_text
     from django.utils.dateparse import parse_datetime
     from django.views.decorators.vary import vary_on_headers
-...
     role_memberships = RoleMembershipsAPI.as_view()
     class SlugFromNameDefault:
         requires_context = False
         serializer_instance = None
         def set_context(self, instance):
             self.serializer_instance = instance
         def __call__(self):
             name = self.serializer_instance.context['request'].data.get('name', '')
             return slugify(name)
     class BaseOrganizationalUnitSerializer(serializers.ModelSerializer):
         slug = serializers.SlugField(
                 required=False,
                 allow_blank=False,
                 max_length=127,
                 default=SlugFromNameDefault(),
+            )
         class Meta:
             model = get_ou_model()
             fields = '__all__'

     from django.core import mail
     from django.urls import reverse
     from django.utils.encoding import force_text
     from django.utils.text import slugify
     from django.utils.timezone import now
     from django.utils.http import urlencode
-...
         assert role.ou == get_default_ou()
     def test_api_post_ou_no_slug(app, superuser):
         app.authorization = ('Basic', (superuser.username, superuser.username))
         OU = get_ou_model()
         ou_data = {
             'name': 'Some Organization Unit',
+            }
         resp = app.post_json('/api/ous/', params=ou_data)
         uuid = resp.json['uuid']
         ou = OU.objects.get(uuid=uuid)
         assert ou.id != get_default_ou().id
         assert ou.slug == slugify(ou.name)
         assert ou.slug == slugify(ou_data['name'])
         # no slug no name
         ou_data = {
             'id': 42,
+            }
         resp = app.post_json('/api/ous/', params=ou_data, status=400)
         assert resp.json['errors']['name'] == ['This field is required.']
     def test_api_post_role_unauthorized(app, simple_user, ou1):
         app.authorization = ('Basic', (simple_user.username, simple_user.username))
         Role = get_role_model()
+    -

Produits Entr'ouvert » Authentic 2