darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-23 08:51:30 -05:00
Code Issues Packages Projects Releases Wiki Activity

Refactor tests for clarity

Browse Source
This commit is contained in:
Imdad Ahad
2017-05-15 15:02:16 +01:00
parent d993e1a43e
commit aa5f8ba443
2 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
tests/app/dao/test_services_dao.py
View File

@@ -50,7 +50,7 @@ from app.models import (
KEY_TYPE_TEST
)
from tests.app.db import create_user
from tests.app.db import create_user, create_service
from tests.app.conftest import (
sample_notification as create_notification,
sample_notification_history as create_notification_history,
@@ -787,9 +787,11 @@ def test_fetch_monthly_historical_template_stats_for_service_separates_templates
assert str(template_two.id) in result.get('2016-04').keys()
def test_dao_fetch_active_users_for_service_returns_active_only(sample_service):
pending_user = create_user(email='foo@bar.com', state='pending')
dao_add_user_to_service(sample_service, pending_user)
users = dao_fetch_active_users_for_service(sample_service.id)
def test_dao_fetch_active_users_for_service_returns_active_only(notify_db, notify_db_session):
active_user = create_user(email='active@foo.com', state='active')
pending_user = create_user(email='pending@foo.com', state='pending')
service = create_service(user=active_user)
dao_add_user_to_service(service, pending_user)
users = dao_fetch_active_users_for_service(service.id)
assert len(users) == 1

5
tests/app/service/test_sender.py
View File

@@ -77,7 +77,10 @@ def test_send_notification_to_service_users_includes_user_fields_in_personalisat
include_user_fields=['name', 'email_address', 'state']
)
assert persist_mock.mock_calls[0][2]['personalisation'] == {
persist_call = persist_mock.call_args_list[0][1]
assert len(persist_mock.call_args_list) == 1
assert persist_call['personalisation'] == {
'name': sample_user.name,
'email_address': sample_user.email_address,
'state': sample_user.state,