diff --git a/app/notifications/process_client_response.py b/app/notifications/process_client_response.py index 4699e0265..f07366422 100644 --- a/app/notifications/process_client_response.py +++ b/app/notifications/process_client_response.py @@ -58,10 +58,10 @@ def process_sms_client_response(status, reference, client_name): # record stats notification = notifications_dao.update_notification_status_by_id(reference, notification_status) if not notification: - current_app.logger.info("{} callback failed: notification {} either not found or already updated " - "from sending. Status {}".format(client_name, - reference, - notification_status_message)) + current_app.logger.warning("{} callback failed: notification {} either not found or already updated " + "from sending. Status {}".format(client_name, + reference, + notification_status_message)) return success, errors if not notification_success: diff --git a/tests/app/notifications/rest/test_callbacks.py b/tests/app/notifications/rest/test_callbacks.py index a1947f57b..f1df63908 100644 --- a/tests/app/notifications/rest/test_callbacks.py +++ b/tests/app/notifications/rest/test_callbacks.py @@ -389,15 +389,6 @@ def test_process_mmg_response_returns_400_for_malformed_data(client): assert "{} callback failed: {} missing".format('MMG', 'CID') in json_data['message'] -def test_ses_callback_should_not_need_auth(client): - response = client.post( - path='/notifications/email/ses', - data=ses_notification_callback(), - headers=[('Content-Type', 'text/plain; charset=UTF-8')] - ) - assert response.status_code == 404 - - def test_mmg_callback_returns_200_when_notification_id_not_found_or_already_updated(client): data = '{"reference": "10100164", "CID": "send-sms-code", "MSISDN": "447775349060", "status": "3", \ "deliverytime": "2016-04-05 16:01:07"}'