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

Added tests for the 2 use cases:

Browse Source 
used templates

unused templates
This commit is contained in:
Martyn Inglis
2016-08-24 12:09:38 +01:00
parent 0b95a30e2f
commit d149a9691a
4 changed files with 100 additions and 58 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/main/views/templates.py
+1 -1
View File
@@ -7,7 +7,7 @@ from dateutil.parser import parse
from notifications_utils.template import Template from notifications_utils.template import Template
from notifications_utils.recipients import first_column_heading from notifications_utils.recipients import first_column_heading
from notifications_python_client.errors import HTTPError from notify_client import HTTPError
from app.main import main from app.main import main
from app.utils import user_has_permissions from app.utils import user_has_permissions
tests/__init__.py
+3 -5
View File
@@ -243,21 +243,19 @@ def single_notification_json(
service_id, service_id,
job=None, job=None,
template=None, template=None,
to='07123456789',
status=None, status=None,
sent_at=None, sent_at=None,
job_row_number=None,
created_at=None, created_at=None,
updated_at=None updated_at=None
): ):
if template is None: if template is None:
template = template_json(service_id, str(generate_uuid())) template = template_json(service_id, str(generate_uuid()))
if sent_at is None: if sent_at is None:
sent_at = str(datetime.utcnow().time()) sent_at = str(datetime.utcnow())
if created_at is None: if created_at is None:
created_at = str(datetime.utcnow().time()) created_at = str(datetime.utcnow())
if updated_at is None: if updated_at is None:
updated_at = str((datetime.utcnow() + timedelta(minutes=1)).time()) updated_at = str(datetime.utcnow() + timedelta(minutes=1))
if status is None: if status is None:
status = 'delivered' status = 'delivered'
job_payload = None job_payload = None
tests/app/main/views/test_templates.py
+95 -51
View File
@@ -1,24 +1,26 @@
from datetime import datetime from datetime import datetime
from unittest.mock import Mock
import pytest import pytest
from bs4 import BeautifulSoup from bs4 import BeautifulSoup
from flask import url_for from flask import url_for
from freezegun import freeze_time from freezegun import freeze_time
from notify_client import HTTPError
from tests import validate_route_permission from tests import validate_route_permission, template_json, single_notification_json
from app.main.views.templates import get_last_use_message, get_human_readable_delta from app.main.views.templates import get_last_use_message, get_human_readable_delta
def test_should_show_page_for_one_template( def test_should_show_page_for_one_template(
app_, app_,
api_user_active, api_user_active,
mock_login, mock_login,
mock_get_service, mock_get_service,
mock_get_service_template, mock_get_service_template,
mock_get_user, mock_get_user,
mock_get_user_by_email, mock_get_user_by_email,
mock_has_permissions, mock_has_permissions,
fake_uuid fake_uuid
): ):
with app_.test_request_context(): with app_.test_request_context():
with app_.test_client() as client: with app_.test_client() as client:
@@ -74,16 +76,16 @@ def test_should_redirect_when_saving_a_template(app_,
def test_should_show_interstitial_when_making_breaking_change( def test_should_show_interstitial_when_making_breaking_change(
app_, app_,
api_user_active, api_user_active,
mock_login, mock_login,
mock_get_service_template, mock_get_service_template,
mock_update_service_template, mock_update_service_template,
mock_get_user, mock_get_user,
mock_get_service, mock_get_service,
mock_get_user_by_email, mock_get_user_by_email,
mock_has_permissions, mock_has_permissions,
fake_uuid fake_uuid
): ):
with app_.test_request_context(): with app_.test_request_context():
with app_.test_client() as client: with app_.test_client() as client:
@@ -143,9 +145,9 @@ def test_should_not_create_too_big_template(app_,
assert resp.status_code == 200 assert resp.status_code == 200
assert ( assert (
"Content has a character count greater" "Content has a character count greater"
" than the limit of 459" " than the limit of 459"
) in resp.get_data(as_text=True) ) in resp.get_data(as_text=True)
def test_should_not_update_too_big_template(app_, def test_should_not_update_too_big_template(app_,
@@ -178,9 +180,9 @@ def test_should_not_update_too_big_template(app_,
assert resp.status_code == 200 assert resp.status_code == 200
assert ( assert (
"Content has a character count greater" "Content has a character count greater"
" than the limit of 459" " than the limit of 459"
) in resp.get_data(as_text=True) ) in resp.get_data(as_text=True)
def test_should_redirect_when_saving_a_template_email(app_, def test_should_redirect_when_saving_a_template_email(app_,
@@ -223,18 +225,59 @@ def test_should_redirect_when_saving_a_template_email(app_,
template_id, name, 'email', content, service_id, subject) template_id, name, 'email', content, service_id, subject)
def test_should_show_delete_template_page(app_, def test_should_show_delete_template_page_with_time_block(app_,
api_user_active, api_user_active,
mock_login, mock_login,
mock_get_service, mock_get_service,
mock_get_service_template, mock_get_service_template,
mock_get_user, mock_get_user,
mock_get_user_by_email, mock_get_user_by_email,
mock_has_permissions, mock_has_permissions,
mock_get_template_statistics_for_template, fake_uuid,
fake_uuid): mocker):
with app_.test_request_context(): with app_.test_request_context():
with app_.test_client() as client: with app_.test_client() as client:
with freeze_time('2012-01-01 12:00:00'):
template = template_json('1234', '1234', "Test template", "sms", "Something very interesting")
notification = single_notification_json('1234', template=template)
mocker.patch('app.template_statistics_client.get_template_statistics_for_template',
return_value=notification)
with freeze_time('2012-01-01 12:10:00'):
client.login(api_user_active)
service_id = fake_uuid
template_id = fake_uuid
response = client.get(url_for(
'.delete_service_template',
service_id=service_id,
template_id=template_id))
content = response.get_data(as_text=True)
assert response.status_code == 200
assert 'Test template was last used 10 minutes ago. Are you sure you want to delete it?' in content
assert 'Are you sure' in content
assert 'Two week reminder' in content
assert 'Your vehicle tax is about to expire' in content
mock_get_service_template.assert_called_with(service_id, template_id)
def test_should_show_delete_template_page_with_never_used_block(app_,
api_user_active,
mock_login,
mock_get_service,
mock_get_service_template,
mock_get_user,
mock_get_user_by_email,
mock_has_permissions,
fake_uuid,
mocker):
with app_.test_request_context():
with app_.test_client() as client:
mocker.patch(
'app.template_statistics_client.get_template_statistics_for_template',
side_effect=HTTPError(response=Mock(status_code=404), message="Default message")
)
client.login(api_user_active) client.login(api_user_active)
service_id = fake_uuid service_id = fake_uuid
template_id = fake_uuid template_id = fake_uuid
@@ -243,12 +286,13 @@ def test_should_show_delete_template_page(app_,
service_id=service_id, service_id=service_id,
template_id=template_id)) template_id=template_id))
content = response.get_data(as_text=True) content = response.get_data(as_text=True)
assert response.status_code == 200 assert response.status_code == 200
assert 'Are you sure' in content assert 'Two week reminder has never been used. Are you sure you want to delete it?' in content
assert 'Two week reminder' in content assert 'Are you sure' in content
assert 'Your vehicle tax is about to expire' in content assert 'Two week reminder' in content
mock_get_service_template.assert_called_with(service_id, template_id) assert 'Your vehicle tax is about to expire' in content
mock_get_service_template.assert_called_with(service_id, template_id)
def test_should_redirect_when_deleting_a_template(app_, def test_should_redirect_when_deleting_a_template(app_,
@@ -294,15 +338,15 @@ def test_should_redirect_when_deleting_a_template(app_,
@freeze_time('2016-01-01T15:00') @freeze_time('2016-01-01T15:00')
def test_should_show_page_for_a_deleted_template( def test_should_show_page_for_a_deleted_template(
app_, app_,
api_user_active, api_user_active,
mock_login, mock_login,
mock_get_service, mock_get_service,
mock_get_deleted_template, mock_get_deleted_template,
mock_get_user, mock_get_user,
mock_get_user_by_email, mock_get_user_by_email,
mock_has_permissions, mock_has_permissions,
fake_uuid fake_uuid
): ):
with app_.test_request_context(), app_.test_client() as client: with app_.test_request_context(), app_.test_client() as client:
client.login(api_user_active) client.login(api_user_active)
tests/conftest.py
+1 -1
View File
@@ -20,7 +20,7 @@ from app.notify_client.models import (
InvitedUser InvitedUser
) )
from notifications_python_client.errors import HTTPError from notify_client.errors import HTTPError
@pytest.fixture(scope='session') @pytest.fixture(scope='session')