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

Fix test assertion

Browse Source 
It wasn't varying the email address seen in the header based on the two
parametrized cases. Not the end of the world, but we should make it
correct
This commit is contained in:
David McDonald
2021-03-04 16:54:41 +00:00
committed by Katie Smith
parent 76fae6f5f8
commit fed40326da
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -1004,7 +1004,7 @@ def test_invite_user(
mock_get_template_folders,
mock_get_organisations,
):
sample_invite['email_address'] = 'test@example.gov.uk'
sample_invite['email_address'] = email_address
assert is_gov_user(email_address) == gov_user
mocker.patch('app.models.user.InvitedUsers.client_method', return_value=[sample_invite])
@@ -1027,7 +1027,7 @@ def test_invite_user(
)
assert page.h1.string.strip() == 'Team members'
flash_banner = page.find('div', class_='banner-default-with-tick').string.strip()
assert flash_banner == 'Invite sent to test@example.gov.uk'
assert flash_banner == f"Invite sent to {email_address}"
expected_permissions = {'manage_api_keys', 'manage_service', 'manage_templates', 'send_messages', 'view_activity'}