Merge pull request #988 from alphagov/non-null-again

Remove nulls from sms_sender

tests/app/delivery/test_send_to_providers.py

@@ -627,7 +627,6 @@ def test_should_set_international_phone_number_to_sent_status(
     # if 40604 is actually in DB then treat that as if entered manually
     ('40604', '40604', 'bar'),
     # 'testing' is the FROM_NUMBER during unit tests
-    (None, 'testing', 'Sample service: bar'),
     ('testing', 'testing', 'Sample service: bar'),
 ])
 def test_should_handle_sms_sender_and_prefix_message(

tests/app/service/test_rest.py

@@ -1315,61 +1315,57 @@ def test_get_only_api_created_notifications_for_service(
     assert response.status_code == 200
 
 
-def test_set_sms_sender_for_service(notify_api, sample_service):
-    with notify_api.test_request_context():
-        with notify_api.test_client() as client:
-            auth_header = create_authorization_header()
-            resp = client.get(
-                '/service/{}'.format(sample_service.id),
-                headers=[auth_header]
-            )
-            json_resp = json.loads(resp.get_data(as_text=True))
-            assert resp.status_code == 200
-            assert json_resp['data']['name'] == sample_service.name
+def test_set_sms_sender_for_service(client, sample_service):
+    data = {
+        'sms_sender': 'elevenchars',
+    }
 
-            data = {
-                'sms_sender': 'elevenchars',
-            }
+    auth_header = create_authorization_header()
 
-            auth_header = create_authorization_header()
-
-            resp = client.post(
-                '/service/{}'.format(sample_service.id),
-                data=json.dumps(data),
-                headers=[('Content-Type', 'application/json'), auth_header]
-            )
-            result = json.loads(resp.get_data(as_text=True))
-            assert resp.status_code == 200
-            assert result['data']['sms_sender'] == 'elevenchars'
+    resp = client.post(
+        '/service/{}'.format(sample_service.id),
+        data=json.dumps(data),
+        headers=[('Content-Type', 'application/json'), auth_header]
+    )
+    result = json.loads(resp.get_data(as_text=True))
+    assert resp.status_code == 200
+    assert result['data']['sms_sender'] == 'elevenchars'
 
 
-def test_set_sms_sender_for_service_rejects_invalid_characters(notify_api, sample_service):
-    with notify_api.test_request_context():
-        with notify_api.test_client() as client:
-            auth_header = create_authorization_header()
-            resp = client.get(
-                '/service/{}'.format(sample_service.id),
-                headers=[auth_header]
-            )
-            json_resp = json.loads(resp.get_data(as_text=True))
-            assert resp.status_code == 200
-            assert json_resp['data']['name'] == sample_service.name
+def test_set_sms_sender_for_service_rejects_invalid_characters(client, sample_service):
+    data = {
+        'sms_sender': 'invalid####',
+    }
 
-            data = {
-                'sms_sender': 'invalid####',
-            }
+    auth_header = create_authorization_header()
 
-            auth_header = create_authorization_header()
+    resp = client.post(
+        '/service/{}'.format(sample_service.id),
+        data=json.dumps(data),
+        headers=[('Content-Type', 'application/json'), auth_header]
+    )
+    result = json.loads(resp.get_data(as_text=True))
+    assert resp.status_code == 400
+    assert result['result'] == 'error'
+    assert result['message'] == {'sms_sender': ['Only alphanumeric characters allowed']}
 
-            resp = client.post(
-                '/service/{}'.format(sample_service.id),
-                data=json.dumps(data),
-                headers=[('Content-Type', 'application/json'), auth_header]
-            )
-            result = json.loads(resp.get_data(as_text=True))
-            assert resp.status_code == 400
-            assert result['result'] == 'error'
-            assert result['message'] == {'sms_sender': ['Only alphanumeric characters allowed']}
+
+def test_set_sms_sender_for_service_rejects_null(client, sample_service):
+    data = {
+        'sms_sender': None,
+    }
+
+    auth_header = create_authorization_header()
+
+    resp = client.post(
+        '/service/{}'.format(sample_service.id),
+        data=json.dumps(data),
+        headers=[('Content-Type', 'application/json'), auth_header]
+    )
+    result = json.loads(resp.get_data(as_text=True))
+    assert resp.status_code == 400
+    assert result['result'] == 'error'
+    assert result['message'] == {'sms_sender': ['Field may not be null.']}
 
 
 @pytest.mark.parametrize('today_only,stats', [
@@ -1929,22 +1925,6 @@ def test_update_service_does_not_call_send_notification_when_restricted_not_chan
     assert not send_notification_mock.called
 
 
-def test_update_service_works_when_sms_sender_is_null(sample_service, client, mocker):
-    sample_service.sms_sender = None
-    data = {'name': 'new name'}
-
-    resp = client.post(
-        'service/{}'.format(sample_service.id),
-        data=json.dumps(data),
-        headers=[create_authorization_header()],
-        content_type='application/json'
-    )
-
-    assert resp.status_code == 200
-    # make sure it wasn't changed to not-null under the hood
-    assert sample_service.sms_sender is None
-
-
 def test_search_for_notification_by_to_field_filters_by_status(client, notify_db, notify_db_session):
     create_notification = partial(
         create_sample_notification,