darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-22 10:31:55 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2404 from alphagov/test-managing-multiple-folder-users-simultaneously

Browse Source 
Test update_template_folder can manage multiple users at once
This commit is contained in:
Pea (Malgorzata Tyczynska)
2019-03-19 14:11:41 +00:00
committed by GitHub
parent 7b8a478b7a 09731c109e
commit a46e7d62f2
1 changed files with 20 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
tests/app/template_folder/test_template_folder_rest.py
View File

@@ -173,21 +173,37 @@ def test_update_template_folder_name(admin_request, sample_service):
def test_update_template_folder_users(admin_request, sample_service):
existing_folder = create_template_folder(sample_service)
user = create_user()
user_1 = create_user(email="notify_1@digital.cabinet-office.gov.uk")
user_2 = create_user(email="notify_2@digital.cabinet-office.gov.uk")
user_3 = create_user(email="notify_3@digital.cabinet-office.gov.uk")
sample_service.users += [user_1, user_2, user_3]
assert len(existing_folder.users) == 0
resp = admin_request.post(
response_1 = admin_request.post(
'template_folder.update_template_folder',
service_id=sample_service.id,
template_folder_id=existing_folder.id,
_data={
'name': 'foo',
'users_with_permission': [str(user.id)]
'users_with_permission': [str(user_1.id)]
}
)
assert resp['data']['users_with_permission'] == [str(user.id)]
assert response_1['data']['users_with_permission'] == [str(user_1.id)]
assert len(existing_folder.users) == 1
response_2 = admin_request.post(
'template_folder.update_template_folder',
service_id=sample_service.id,
template_folder_id=existing_folder.id,
_data={
'name': 'foo',
'users_with_permission': [str(user_2.id), str(user_3.id)]
}
)
assert response_2['data']['users_with_permission'] == [str(user_2.id), str(user_3.id)]
assert len(existing_folder.users) == 2
@pytest.mark.parametrize('data, err', [
({}, 'name is a required property'),