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

Bug fix for passing list of empty string if no permissions selected for

Browse Source 
invite.

Send empty list instead.
This commit is contained in:
Adam Shimali
2016-03-11 12:50:25 +00:00
parent f81dccc513
commit 069a549d57
2 changed files with 27 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
tests/app/main/views/test_accept_invite.py
View File

@@ -36,6 +36,29 @@ def test_existing_user_accept_invite_calls_api_and_redirects_to_dashboard(app_,
assert response.location == expected_redirect_location
def test_existing_user_with_no_permissions_accept_invite(app_,
mocker,
service_one,
api_user_active,
sample_invite,
mock_check_invite_token,
mock_get_user_by_email,
mock_add_user_to_service):
expected_service = service_one['id']
sample_invite['permissions'] = ''
expected_permissions = []
mocker.patch('app.invite_api_client.accept_invite', return_value=sample_invite)
with app_.test_request_context():
with app_.test_client() as client:
response = client.get(url_for('main.accept_invite', token='thisisnotarealtoken'))
mock_add_user_to_service.assert_called_with(expected_service, api_user_active.id, expected_permissions)
assert response.status_code == 302
def test_existing_user_cant_accept_twice(app_,
mocker,
sample_invite):