Update reply to email test

- Updated reply to email test so that the request to create or update
email is not mocked.
- Excluded reply_to_email_addresses from the Service schema
This commit is contained in:
Katie Smith
2017-09-11 16:28:44 +01:00
parent a507f74823
commit 269e88f7de
2 changed files with 4 additions and 5 deletions

View File

@@ -12,7 +12,7 @@ from app.dao.services_dao import dao_remove_user_from_service
from app.dao.templates_dao import dao_redact_template
from app.dao.users_dao import save_model_user
from app.models import (
User, Organisation, Service, ServicePermission, Notification,
User, Organisation, Service, ServicePermission, Notification, ServiceEmailReplyTo,
DVLA_ORG_LAND_REGISTRY,
KEY_TYPE_NORMAL, KEY_TYPE_TEAM, KEY_TYPE_TEST,
EMAIL_TYPE, SMS_TYPE, LETTER_TYPE, INTERNATIONAL_SMS_TYPE, INBOUND_SMS_TYPE,
@@ -2125,9 +2125,7 @@ def test_is_service_name_unique_returns_400_when_name_does_not_exist(client):
assert json_resp["message"][1]["email_from"] == ["Can't be empty"]
def test_update_service_reply_to_email_address_upserts_email_reply_to(mocker, admin_request, sample_service):
update_mock = mocker.patch('app.service.rest.create_or_update_email_reply_to')
def test_update_service_reply_to_email_address_upserts_email_reply_to(admin_request, sample_service):
admin_request.post(
'service.update_service',
service_id=sample_service.id,
@@ -2137,4 +2135,4 @@ def test_update_service_reply_to_email_address_upserts_email_reply_to(mocker, ad
_expected_status=200
)
assert update_mock.called
assert len(ServiceEmailReplyTo.query.all()) == 1