2016-02-26 10:54:06 +00:00
|
|
|
import pytest
|
|
|
|
|
|
2016-01-11 15:00:51 +00:00
|
|
|
from io import BytesIO
|
2016-01-14 16:00:13 +00:00
|
|
|
from flask import url_for
|
2016-02-26 10:54:06 +00:00
|
|
|
from unittest.mock import ANY
|
2016-03-09 12:10:50 +00:00
|
|
|
from tests import validate_route_permission
|
2016-01-13 17:32:40 +00:00
|
|
|
|
2016-02-22 17:17:18 +00:00
|
|
|
template_types = ['email', 'sms']
|
2016-01-13 17:32:40 +00:00
|
|
|
|
2016-02-22 17:17:18 +00:00
|
|
|
|
|
|
|
|
@pytest.mark.parametrize("template_type", template_types)
|
|
|
|
|
def test_choose_template(
|
|
|
|
|
template_type,
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
|
|
|
|
mock_get_user,
|
|
|
|
|
mock_get_service,
|
|
|
|
|
mock_check_verify_code,
|
|
|
|
|
mock_get_service_templates,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_get_jobs,
|
|
|
|
|
mock_has_permissions
|
2016-02-22 17:17:18 +00:00
|
|
|
):
|
2016-02-02 17:28:30 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
2016-02-22 17:17:18 +00:00
|
|
|
response = client.get(url_for('main.choose_template', template_type=template_type, service_id=12345))
|
2016-02-02 17:28:30 +00:00
|
|
|
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
content = response.get_data(as_text=True)
|
2016-02-22 17:17:18 +00:00
|
|
|
assert '{}_template_one'.format(template_type) in content
|
|
|
|
|
assert '{} template one content'.format(template_type) in content
|
|
|
|
|
assert '{}_template_two'.format(template_type) in content
|
|
|
|
|
assert '{} template two content'.format(template_type) in content
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_upload_empty_csvfile_returns_to_upload_page(
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
|
|
|
|
mock_get_user,
|
|
|
|
|
mock_get_service,
|
|
|
|
|
mock_get_service_templates,
|
|
|
|
|
mock_check_verify_code,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_has_permissions
|
2016-02-22 17:17:18 +00:00
|
|
|
):
|
2016-01-15 15:15:35 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
2016-01-27 12:22:32 +00:00
|
|
|
client.login(api_user_active)
|
2016-01-11 15:00:51 +00:00
|
|
|
upload_data = {'file': (BytesIO(''.encode('utf-8')), 'emtpy.csv')}
|
2016-02-22 17:17:18 +00:00
|
|
|
response = client.post(
|
|
|
|
|
url_for('main.send_messages', service_id=12345, template_id=54321),
|
|
|
|
|
data=upload_data,
|
|
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
2015-12-17 16:21:34 +00:00
|
|
|
|
2016-01-11 15:00:51 +00:00
|
|
|
assert response.status_code == 200
|
|
|
|
|
content = response.get_data(as_text=True)
|
|
|
|
|
assert 'The file emtpy.csv contained no data' in content
|
2015-12-17 16:21:34 +00:00
|
|
|
|
|
|
|
|
|
2016-02-22 17:17:18 +00:00
|
|
|
def test_upload_csvfile_with_invalid_phone_shows_check_page_with_errors(
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
2016-02-26 10:54:06 +00:00
|
|
|
mocker,
|
2016-02-22 17:17:18 +00:00
|
|
|
mock_login,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-26 10:54:06 +00:00
|
|
|
mock_get_service_template,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_s3_upload,
|
|
|
|
|
mock_has_permissions
|
2016-02-22 17:17:18 +00:00
|
|
|
):
|
2016-01-13 22:34:36 +00:00
|
|
|
|
2016-03-01 09:41:08 +00:00
|
|
|
contents = 'phone number,name\n+44 123,test1\n+44 456,test2'
|
2016-01-13 22:34:36 +00:00
|
|
|
file_data = (BytesIO(contents.encode('utf-8')), 'invalid.csv')
|
2016-02-26 10:54:06 +00:00
|
|
|
mocker.patch('app.main.views.send.s3download', return_value=contents)
|
2015-12-17 16:21:34 +00:00
|
|
|
|
2016-01-15 15:15:35 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
2016-01-27 12:22:32 +00:00
|
|
|
client.login(api_user_active)
|
2016-01-13 22:34:36 +00:00
|
|
|
upload_data = {'file': file_data}
|
2016-02-22 17:17:18 +00:00
|
|
|
response = client.post(
|
|
|
|
|
url_for('main.send_messages', service_id=12345, template_id=54321),
|
|
|
|
|
data=upload_data,
|
|
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
2016-01-11 15:00:51 +00:00
|
|
|
assert response.status_code == 200
|
|
|
|
|
content = response.get_data(as_text=True)
|
2016-02-17 15:49:07 +00:00
|
|
|
assert 'Your CSV file contained missing or invalid data' in content
|
2016-01-11 15:00:51 +00:00
|
|
|
assert '+44 123' in content
|
|
|
|
|
assert '+44 456' in content
|
2016-02-22 21:37:32 +00:00
|
|
|
assert 'Upload a CSV file' in content
|
2015-12-17 16:21:34 +00:00
|
|
|
|
|
|
|
|
|
2016-02-26 10:54:06 +00:00
|
|
|
def test_upload_csvfile_removes_empty_lines_and_trailing_commas(
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mocker,
|
|
|
|
|
mock_login,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-26 10:54:06 +00:00
|
|
|
mock_get_service_template,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_s3_upload,
|
|
|
|
|
mock_has_permissions
|
2016-02-26 10:54:06 +00:00
|
|
|
):
|
|
|
|
|
|
2016-03-01 09:41:08 +00:00
|
|
|
contents = 'phone number,name,,,\n++44 7700 900981,test1,,,\n+44 7700 900981,test2,,,\n ,,, \n ,,, \t \t \n'
|
2016-02-26 10:54:06 +00:00
|
|
|
file_data = (BytesIO(contents.encode('utf-8')), 'invalid.csv')
|
|
|
|
|
|
2016-03-01 09:41:08 +00:00
|
|
|
expected_data = {'data': ['phone number,name', '++44 7700 900981,test1', '+44 7700 900981,test2'],
|
2016-02-26 10:54:06 +00:00
|
|
|
'file_name': 'invalid.csv'}
|
|
|
|
|
|
|
|
|
|
mocker.patch('app.main.views.send.s3download',
|
2016-03-01 09:41:08 +00:00
|
|
|
return_value='phone number,name\n++44 7700 900981,test1\n+44 7700 900981,test2')
|
2016-02-26 10:54:06 +00:00
|
|
|
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
|
|
|
|
upload_data = {'file': file_data}
|
|
|
|
|
response = client.post(
|
|
|
|
|
url_for('main.send_messages', service_id=12345, template_id=54321),
|
|
|
|
|
data=upload_data,
|
|
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
mock_s3_upload.assert_called_with(ANY, '12345', expected_data, 'eu-west-1')
|
|
|
|
|
|
|
|
|
|
|
2016-02-18 17:03:32 +00:00
|
|
|
def test_send_test_message_to_self(
|
|
|
|
|
app_,
|
|
|
|
|
mocker,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-26 10:54:06 +00:00
|
|
|
mock_get_service_template,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_s3_upload,
|
|
|
|
|
mock_has_permissions
|
2016-02-18 17:03:32 +00:00
|
|
|
):
|
|
|
|
|
|
2016-03-01 09:41:08 +00:00
|
|
|
expected_data = {'data': ['phone number', '+4412341234'], 'file_name': 'Test run'}
|
|
|
|
|
mocker.patch('app.main.views.send.s3download', return_value='phone number\r\n+4412341234')
|
2016-02-26 10:54:06 +00:00
|
|
|
|
2016-02-18 17:03:32 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
|
|
|
|
response = client.get(
|
2016-02-22 17:17:18 +00:00
|
|
|
url_for('main.send_message_to_self', service_id=12345, template_id=54321),
|
2016-02-18 17:03:32 +00:00
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
2016-02-26 10:54:06 +00:00
|
|
|
mock_s3_upload.assert_called_with(ANY, '12345', expected_data, 'eu-west-1')
|
2016-02-18 17:03:32 +00:00
|
|
|
|
|
|
|
|
|
2016-03-02 17:02:41 +00:00
|
|
|
def test_send_test_message_to_self(
|
|
|
|
|
app_,
|
|
|
|
|
mocker,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
|
|
|
|
mock_get_service,
|
|
|
|
|
mock_get_service_email_template,
|
2016-03-03 09:30:30 +00:00
|
|
|
mock_s3_upload,
|
|
|
|
|
mock_has_permissions
|
2016-03-02 17:02:41 +00:00
|
|
|
):
|
|
|
|
|
|
|
|
|
|
expected_data = {'data': ['email address', 'test@user.gov.uk'], 'file_name': 'Test run'}
|
|
|
|
|
mocker.patch('app.main.views.send.s3download', return_value='email address\r\ntest@user.gov.uk')
|
|
|
|
|
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
|
|
|
|
response = client.get(
|
|
|
|
|
url_for('main.send_message_to_self', service_id=12345, template_id=54321),
|
|
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
mock_s3_upload.assert_called_with(ANY, '12345', expected_data, 'eu-west-1')
|
|
|
|
|
|
|
|
|
|
|
2016-02-18 17:03:32 +00:00
|
|
|
def test_download_example_csv(
|
|
|
|
|
app_,
|
|
|
|
|
mocker,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_has_permissions
|
2016-02-18 17:03:32 +00:00
|
|
|
):
|
|
|
|
|
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
|
|
|
|
response = client.get(
|
|
|
|
|
url_for('main.get_example_csv', service_id=12345, template_id=54321),
|
|
|
|
|
follow_redirects=True
|
|
|
|
|
)
|
|
|
|
|
assert response.status_code == 200
|
2016-03-01 09:41:08 +00:00
|
|
|
assert response.get_data(as_text=True) == 'phone number\r\n+4412341234\r\n'
|
2016-02-18 17:03:32 +00:00
|
|
|
assert 'text/csv' in response.headers['Content-Type']
|
|
|
|
|
|
|
|
|
|
|
2016-02-22 17:17:18 +00:00
|
|
|
def test_upload_csvfile_with_valid_phone_shows_all_numbers(
|
|
|
|
|
app_,
|
|
|
|
|
mocker,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-26 10:54:06 +00:00
|
|
|
mock_get_service_template,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_s3_upload,
|
|
|
|
|
mock_has_permissions
|
2016-02-22 17:17:18 +00:00
|
|
|
):
|
2016-01-13 22:34:36 +00:00
|
|
|
|
2016-03-01 09:41:08 +00:00
|
|
|
contents = 'phone number\n+44 7700 900981\n+44 7700 900982\n+44 7700 900983\n+44 7700 900984\n+44 7700 900985\n+44 7700 900986' # noqa
|
2016-01-13 22:34:36 +00:00
|
|
|
file_data = (BytesIO(contents.encode('utf-8')), 'valid.csv')
|
2016-02-26 10:54:06 +00:00
|
|
|
mocker.patch('app.main.views.send.s3download', return_value=contents)
|
2016-01-11 15:00:51 +00:00
|
|
|
|
2016-01-15 15:15:35 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
2016-01-27 12:22:32 +00:00
|
|
|
client.login(api_user_active)
|
2016-01-13 22:34:36 +00:00
|
|
|
upload_data = {'file': file_data}
|
2016-02-22 17:17:18 +00:00
|
|
|
response = client.post(url_for('main.send_messages', service_id=12345, template_id=54321),
|
2016-01-14 16:00:13 +00:00
|
|
|
data=upload_data,
|
|
|
|
|
follow_redirects=True)
|
2016-02-22 14:52:16 +00:00
|
|
|
with client.session_transaction() as sess:
|
|
|
|
|
assert int(sess['upload_data']['template_id']) == 54321
|
|
|
|
|
assert sess['upload_data']['original_file_name'] == 'valid.csv'
|
|
|
|
|
assert sess['upload_data']['notification_count'] == 6
|
2016-01-11 15:00:51 +00:00
|
|
|
|
2016-02-22 14:52:16 +00:00
|
|
|
content = response.get_data(as_text=True)
|
2016-01-11 15:00:51 +00:00
|
|
|
|
2016-02-22 14:52:16 +00:00
|
|
|
assert response.status_code == 200
|
|
|
|
|
assert '+44 7700 900981' in content
|
|
|
|
|
assert '+44 7700 900982' in content
|
|
|
|
|
assert '+44 7700 900983' in content
|
|
|
|
|
assert '+44 7700 900984' in content
|
|
|
|
|
assert '+44 7700 900985' in content
|
|
|
|
|
assert '+44 7700 900986' in content
|
2016-01-11 15:00:51 +00:00
|
|
|
|
|
|
|
|
|
2016-02-22 17:17:18 +00:00
|
|
|
def test_create_job_should_call_api(
|
|
|
|
|
app_,
|
|
|
|
|
service_one,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
|
|
|
|
job_data,
|
|
|
|
|
mock_create_job,
|
|
|
|
|
mock_get_job,
|
2016-03-02 15:37:35 +00:00
|
|
|
mock_get_notifications,
|
2016-02-26 12:11:55 +00:00
|
|
|
mock_get_service,
|
2016-02-29 17:03:56 +00:00
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_has_permissions
|
2016-02-22 17:17:18 +00:00
|
|
|
):
|
2016-01-29 10:27:23 +00:00
|
|
|
|
2016-01-29 12:19:50 +00:00
|
|
|
service_id = service_one['id']
|
2016-02-01 11:28:36 +00:00
|
|
|
job_id = job_data['id']
|
|
|
|
|
original_file_name = job_data['original_file_name']
|
|
|
|
|
template_id = job_data['template']
|
2016-02-22 14:52:16 +00:00
|
|
|
notification_count = job_data['notification_count']
|
2016-01-29 10:27:23 +00:00
|
|
|
|
2016-01-15 15:15:35 +00:00
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
2016-01-27 12:22:32 +00:00
|
|
|
client.login(api_user_active)
|
2016-02-01 11:28:36 +00:00
|
|
|
with client.session_transaction() as session:
|
2016-02-22 14:52:16 +00:00
|
|
|
session['upload_data'] = {'original_file_name': original_file_name,
|
|
|
|
|
'template_id': template_id,
|
|
|
|
|
'notification_count': notification_count}
|
2016-02-22 17:17:18 +00:00
|
|
|
url = url_for('main.check_messages', service_id=service_one['id'], upload_id=job_id)
|
2016-01-29 10:27:23 +00:00
|
|
|
response = client.post(url, data=job_data, follow_redirects=True)
|
|
|
|
|
|
|
|
|
|
assert response.status_code == 200
|
2016-03-02 21:19:02 +00:00
|
|
|
assert original_file_name in response.get_data(as_text=True)
|
2016-02-22 14:52:16 +00:00
|
|
|
mock_create_job.assert_called_with(job_id, service_id, template_id, original_file_name, notification_count)
|
2016-03-03 15:26:52 +00:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_check_messages_should_revalidate_file_when_uploading_file(
|
|
|
|
|
app_,
|
|
|
|
|
service_one,
|
|
|
|
|
api_user_active,
|
|
|
|
|
mock_login,
|
|
|
|
|
mock_get_service,
|
|
|
|
|
job_data,
|
|
|
|
|
mock_create_job,
|
|
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_s3_upload,
|
2016-03-03 15:39:15 +00:00
|
|
|
mocker,
|
|
|
|
|
mock_has_permissions
|
2016-03-03 15:26:52 +00:00
|
|
|
):
|
|
|
|
|
|
|
|
|
|
service_id = service_one['id']
|
|
|
|
|
contents = 'phone number,name,,,\n++44 7700 900981,test1,,,\n+44 7700 900981,test2,,,\n ,,, \n ,,, \t \t \n'
|
|
|
|
|
file_data = (BytesIO(contents.encode('utf-8')), 'invalid.csv')
|
|
|
|
|
upload_data = {'file': file_data}
|
|
|
|
|
|
|
|
|
|
mocker.patch('app.main.views.send.s3download', return_value=contents)
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
with app_.test_client() as client:
|
|
|
|
|
client.login(api_user_active)
|
|
|
|
|
with client.session_transaction() as session:
|
|
|
|
|
session['upload_data'] = {'original_file_name': 'invalid.csv',
|
|
|
|
|
'template_id': job_data['template'],
|
|
|
|
|
'notification_count': job_data['notification_count']}
|
|
|
|
|
url = url_for('main.check_messages', service_id=service_id, upload_id=job_data['id'])
|
|
|
|
|
response = client.post(url, data=upload_data, follow_redirects=True)
|
|
|
|
|
assert response.status_code == 200
|
|
|
|
|
assert 'Your CSV file contained missing or invalid data' in response.get_data(as_text=True)
|
2016-03-09 12:10:50 +00:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_route_permissions(mocker,
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
|
|
|
|
service_one,
|
|
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_get_jobs,
|
|
|
|
|
mock_get_notifications,
|
|
|
|
|
mock_create_job,
|
|
|
|
|
mock_s3_upload):
|
|
|
|
|
routes = [
|
|
|
|
|
'main.choose_template',
|
|
|
|
|
'main.send_messages',
|
|
|
|
|
'main.get_example_csv']
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
for route in routes:
|
|
|
|
|
validate_route_permission(
|
|
|
|
|
mocker,
|
|
|
|
|
app_,
|
|
|
|
|
"GET",
|
|
|
|
|
200,
|
|
|
|
|
url_for(
|
|
|
|
|
route,
|
|
|
|
|
service_id=service_one['id'],
|
|
|
|
|
template_type='sms',
|
|
|
|
|
template_id=123),
|
|
|
|
|
['send_texts', 'send_emails', 'send_letters'],
|
|
|
|
|
api_user_active,
|
|
|
|
|
service_one)
|
|
|
|
|
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
validate_route_permission(
|
|
|
|
|
mocker,
|
|
|
|
|
app_,
|
|
|
|
|
"GET",
|
|
|
|
|
302,
|
|
|
|
|
url_for(
|
|
|
|
|
'main.send_message_to_self',
|
|
|
|
|
service_id=service_one['id'],
|
|
|
|
|
template_type='sms',
|
|
|
|
|
template_id=123),
|
|
|
|
|
['send_texts', 'send_emails', 'send_letters'],
|
|
|
|
|
api_user_active,
|
|
|
|
|
service_one)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_route_invalid_permissions(mocker,
|
|
|
|
|
app_,
|
|
|
|
|
api_user_active,
|
|
|
|
|
service_one,
|
|
|
|
|
mock_get_service_template,
|
|
|
|
|
mock_get_jobs,
|
|
|
|
|
mock_get_notifications,
|
|
|
|
|
mock_create_job):
|
|
|
|
|
routes = [
|
|
|
|
|
'main.choose_template',
|
|
|
|
|
'main.send_messages',
|
|
|
|
|
'main.get_example_csv',
|
|
|
|
|
'main.send_message_to_self']
|
|
|
|
|
with app_.test_request_context():
|
|
|
|
|
for route in routes:
|
|
|
|
|
validate_route_permission(
|
|
|
|
|
mocker,
|
|
|
|
|
app_,
|
|
|
|
|
"GET",
|
|
|
|
|
403,
|
|
|
|
|
url_for(
|
|
|
|
|
route,
|
|
|
|
|
service_id=service_one['id'],
|
|
|
|
|
template_type='sms',
|
|
|
|
|
template_id=123),
|
|
|
|
|
['blah'],
|
|
|
|
|
api_user_active,
|
|
|
|
|
service_one)
|