Change provider balance on slow delivery every 5 minutes

Instead of every 10 minutes - this way we react faster, and
fewer text messages get delayed.

This becomes more important as we now send lots of login codes
for various organisations.

app/celery/scheduled_tasks.py

@@ -131,8 +131,8 @@ def delete_invitations():
 def switch_current_sms_provider_on_slow_delivery():
     """
     Reduce provider's priority if at least 15% of notifications took more than 5 minutes to be delivered
-    in the last ten minutes. If both providers are slow, don't do anything. If we changed the providers in the
-    last ten minutes, then don't update them again either.
+    in the last 15 minutes. If both providers are slow, don't do anything. If we changed the providers in the
+    last 5 minutes, then don't update them again either.
     """
     slow_delivery_notifications = is_delivery_slow_for_providers(
         created_within_minutes=15,
@@ -146,7 +146,7 @@ def switch_current_sms_provider_on_slow_delivery():
         for provider_name, is_slow in slow_delivery_notifications.items():
             if is_slow:
                 current_app.logger.warning("Slow delivery notifications detected for provider %s", provider_name)
-                dao_reduce_sms_provider_priority(provider_name, time_threshold=timedelta(minutes=10))
+                dao_reduce_sms_provider_priority(provider_name, time_threshold=timedelta(minutes=5))
 
 
 def _check_slow_text_message_delivery_reports_and_raise_error_if_needed(reports: list[SlowProviderDeliveryReport]):

tests/app/celery/test_scheduled_tasks.py

@@ -139,7 +139,7 @@ def test_switch_current_sms_provider_on_slow_delivery_switches_when_one_provider
     switch_current_sms_provider_on_slow_delivery()
 
     mock_is_slow.assert_called_once_with(created_within_minutes=15, delivered_within_minutes=5, threshold=0.15)
-    mock_reduce.assert_called_once_with("firetext", time_threshold=timedelta(minutes=10))
+    mock_reduce.assert_called_once_with("firetext", time_threshold=timedelta(minutes=5))
 
 
 @freeze_time("2017-05-01 14:00:00")