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

Make test use new fixture

Browse Source 
The `client_request` fixture means the tests take fewer lines of code
and also checks some extra things (eg page titles matching <h1>s).

Changing it in this commit, independently of functional changes to this
test.
This commit is contained in:
Chris Hill-Scott
2017-12-07 14:32:05 +00:00
parent 5bac021493
commit 71368301f7
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
tests/app/main/views/test_send.py
View File

@@ -441,7 +441,7 @@ def test_upload_csv_invalid_extension(
def test_upload_valid_csv_shows_file_contents(
logged_in_client,
client_request,
mocker,
mock_get_service_template_with_placeholders,
mock_s3_upload,
@@ -455,14 +455,13 @@ def test_upload_valid_csv_shows_file_contents(
07700900986, Jo, foo, foo, foo
""")
response = logged_in_client.post(
url_for('main.send_messages', service_id=SERVICE_ONE_ID, template_id=fake_uuid),
data={'file': (BytesIO(''.encode('utf-8')), 'valid.csv')},
follow_redirects=True,
page = client_request.post(
'main.send_messages', service_id=SERVICE_ONE_ID, template_id=fake_uuid,
_data={'file': (BytesIO(''.encode('utf-8')), 'valid.csv')},
_follow_redirects=True,
_expected_status=200,
)
assert response.status_code == 200
page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser')
assert page.h1.text.strip() == 'Preview of Two week reminder'
for index, cell in enumerate([
'<td class="table-field-index"> <span class=""> 2 </span> </td>',