0001-hobo_notify-handle-new-user-attribute-is_superuser-f.patch
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 |
- |