From 7dffe5ec372a12c68e16be798a064d0f131e6ecf Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Tue, 7 Jul 2020 11:48:19 +0200 Subject: [PATCH] misc: provision user.is_active (#44896) --- hobo/agent/authentic2/provisionning.py | 1 + hobo/provisionning/utils.py | 1 + tests_authentic/test_provisionning.py | 2 ++ 3 files changed, 4 insertions(+) diff --git a/hobo/agent/authentic2/provisionning.py b/hobo/agent/authentic2/provisionning.py index 6f594a8..7fac7d1 100644 --- a/hobo/agent/authentic2/provisionning.py +++ b/hobo/agent/authentic2/provisionning.py @@ -120,6 +120,7 @@ class Provisionning(threading.local): 'first_name': user.first_name, 'last_name': user.last_name, 'email': user.email, + 'is_active': user.is_active, 'roles': [ { 'uuid': role.uuid, diff --git a/hobo/provisionning/utils.py b/hobo/provisionning/utils.py index 980189c..7126488 100644 --- a/hobo/provisionning/utils.py +++ b/hobo/provisionning/utils.py @@ -88,6 +88,7 @@ class NotificationProcessing: user.username = o['uuid'][:30] user.is_superuser = o['is_superuser'] user.is_staff = o['is_superuser'] + user.is_active = o.get('is_active', True) user.save() role_uuids = [role['uuid'] for role in o.get('roles', [])] provision_user_groups(user, role_uuids) diff --git a/tests_authentic/test_provisionning.py b/tests_authentic/test_provisionning.py index dfb618f..c1d040c 100644 --- a/tests_authentic/test_provisionning.py +++ b/tests_authentic/test_provisionning.py @@ -141,6 +141,7 @@ def test_provision_user(transactional_db, tenant, caplog): email='iohn.doe@example.net', first_name=u'John', last_name=u'Doe', + is_active=False, ou=get_default_ou()) role2.members.add(user2) users = {user.uuid: user for user in [user1, user2]} @@ -174,6 +175,7 @@ def test_provision_user(transactional_db, tenant, caplog): assert o['first_name'] == user.first_name assert o['last_name'] == user.last_name assert o['email'] == user.email + assert o['is_active'] is user.is_active assert o['roles'] == [{'name': r.name, 'slug': r.slug, 'uuid': r.uuid} for r in user.roles.all()] assert o['is_superuser'] is False -- 2.26.2