Don’t allow test messages to be scheduled

For two reasons:
- it’s extra stuff in tour that users dont yet need to know about
- test messages are hidden from the dashboard, so you’d have no
  visibility of when they were sending once you’d scheduled them

app/main/views/send.py

@@ -250,8 +250,10 @@ def check_messages(service_id, template_type, upload_id):
             back_link = url_for(
                 '.choose_template', service_id=service_id, template_type=template.template_type, **extra_args
             )
+        choose_time_form = None
     else:
         back_link = url_for('.send_messages', service_id=service_id, template_id=template.id)
+        choose_time_form = ChooseTimeForm()
 
     with suppress(StopIteration):
         template.values = next(recipients.rows)
@@ -276,7 +278,7 @@ def check_messages(service_id, template_type, upload_id):
         upload_id=upload_id,
         form=CsvUploadForm(),
         remaining_messages=remaining_messages,
-        choose_time_form=ChooseTimeForm(),
+        choose_time_form=choose_time_form,
         back_link=back_link,
         help=get_help_argument()
     )

app/templates/views/check.html

@@ -145,10 +145,12 @@
     <form method="post" enctype="multipart/form-data" action="{{url_for('main.start_job', service_id=current_service.id, upload_id=upload_id)}}" class='page-footer'>
       <input type="hidden" name="csrf_token" value="{{ csrf_token() }}" />
       <input type="hidden" name="help" value="{{ '3' if help else 0 }}" />
-      {{ radio_select(
-        choose_time_form.scheduled_for,
-        wrapping_class='bottom-gutter-2-3'
-      ) }}
+      {% if choose_time_form %}
+        {{ radio_select(
+          choose_time_form.scheduled_for,
+          wrapping_class='bottom-gutter-2-3'
+        ) }}
+      {% endif %}
       <input type="submit" class="button" value="Send {{ count_of_recipients }} {{ message_count_label(count_of_recipients, template.template_type, suffix='') }}" />
       <a href="{{ back_link }}" class="page-footer-back-link">Back</a>
     </form>

tests/app/main/views/test_send.py

@@ -322,6 +322,41 @@ def test_upload_csvfile_with_valid_phone_shows_all_numbers(
             mock_get_detailed_service_for_today.assert_called_once_with(fake_uuid)
 
 
+def test_test_message_can_only_be_sent_now(
+    app_,
+    mocker,
+    api_user_active,
+    mock_login,
+    mock_get_service,
+    mock_get_service_template,
+    mock_s3_download,
+    mock_has_permissions,
+    mock_get_users_by_service,
+    mock_get_detailed_service_for_today,
+    fake_uuid
+):
+
+    with app_.test_request_context(), app_.test_client() as client:
+        client.login(api_user_active)
+        with client.session_transaction() as session:
+            session['upload_data'] = {
+                'original_file_name': 'Test message',
+                'template_id': fake_uuid,
+                'notification_count': 1,
+                'valid': True
+            }
+        response = client.get(url_for(
+            'main.check_messages',
+            service_id=fake_uuid,
+            upload_id=fake_uuid,
+            template_type='sms',
+            from_test=True
+        ))
+
+        content = response.get_data(as_text=True)
+        assert 'name="scheduled_for"' not in content
+
+
 @pytest.mark.parametrize(
     'when', [
         '', '2016-08-25T13:04:21.767198'