Projet

Général

Profil

0001-hobo_notify-handle-new-user-attribute-is_superuser-f.patch

Benjamin Dauvergne, 04 janvier 2016 16:48

Télécharger (2,73 ko)

Voir les différences:

Subject: [PATCH] hobo_notify: handle new user attribute is_superuser (fixes
 #9488)

 tests/test_hobo_notify.py | 4 ++++
 wcs/ctl/hobo_notify.py    | 4 +++-
 2 files changed, 7 insertions(+), 1 deletion(-)
tests/test_hobo_notify.py
372 372
                    u'last_name': u'Doe',
373 373
                    u'email': u'john.doe@example.net',
374 374
                    u'zipcode': u'13400',
375
                    u'is_superuser': False,
375 376
                    u'roles': [
376 377
                        {
377 378
                            u'uuid': u'12345',
......
398 399
    assert user.form_data['_last_name'] == 'Doe'
399 400
    assert user.form_data['_zipcode'] == '13400'
400 401
    assert user.name_identifiers == ['a'*32]
402
    assert user.is_admin is False
401 403
    assert set(user.roles) == set(['12345', old_role.id])
402 404

  
403 405
    notification = {
......
413 415
                    u'last_name': u'Doe',
414 416
                    u'email': u'john.doe@example.net',
415 417
                    u'zipcode': u'13600',
418
                    u'is_superuser': True,
416 419
                    u'roles': [
417 420
                        {
418 421
                            u'uuid': u'xyz',
......
434 437
    assert user.form_data['_last_name'] == 'Doe'
435 438
    assert user.form_data['_zipcode'] == '13600'
436 439
    assert user.name_identifiers == ['a'*32]
440
    assert user.is_admin is True
437 441
    assert set(user.roles) == set([old_role.id])
wcs/ctl/hobo_notify.py
150 150
               and 'email' in o \
151 151
               and 'first_name' in o \
152 152
               and 'last_name' in o \
153
               and 'roles' in o
153
               and 'roles' in o \
154
               and 'is_superuser' in o
154 155

  
155 156
    @classmethod
156 157
    def provision_user(cls, publisher, issuer, action, data, full=False):
......
180 181
                    user.name_identifiers = [uuid]
181 182
                    role_uuids = [role['uuid'] for role in o['roles']]
182 183
                    # reset roles
184
                    user.is_admin = o['is_superuser']
183 185
                    user.roles = []
184 186
                    for role_uuid in role_uuids:
185 187
                        try:
186
-