darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-30 02:50:03 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug with creating the invited user permission list.

Browse Source
This commit is contained in:
Rebecca Law
2016-03-30 17:39:19 +01:00
parent 9de80d685a
commit c4b65d6af9
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/main/views/manage_users.py
View File

@@ -60,7 +60,9 @@ def invite_user(service_id):
# view_activity is a default role to be added to all users.
# All users will have at minimum view_activity to allow users to see notifications,
# templates, team members but no update privileges
permissions = ','.join(role for role in roles.keys() if request.form.get(role) == 'y').join('view_activity')
selected_permissions = [role for role in roles.keys() if request.form.get(role) == 'y']
selected_permissions.append('view_activity')
permissions = ','.join(selected_permissions)
invited_user = invite_api_client.create_invite(
current_user.id,
service_id,

4
tests/app/main/views/test_manage_users.py
View File

@@ -161,6 +161,10 @@ def test_invite_user(
assert page.h1.string.strip() == 'Manage team'
flash_banner = page.find('div', class_='banner-default-with-tick').string.strip()
assert flash_banner == 'Invite sent to test@example.gov.uk'
app.invite_api_client.create_invite.assert_called_once_with(sample_invite['from_user'],
sample_invite['service'],
email_address,
sample_invite['permissions']+',view_activity')
def test_cancel_invited_user_cancels_user_invitations(app_,