From 6938600ab89c5c058a77e4411fd067daebd64685 Mon Sep 17 00:00:00 2001 From: Pea Tyczynska Date: Wed, 5 Dec 2018 14:46:21 +0000 Subject: [PATCH] Switch providers on slow delivery only produces logs --- app/celery/scheduled_tasks.py | 4 ++-- tests/app/celery/test_scheduled_tasks.py | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/app/celery/scheduled_tasks.py b/app/celery/scheduled_tasks.py index 148749994..1c8d561c7 100644 --- a/app/celery/scheduled_tasks.py +++ b/app/celery/scheduled_tasks.py @@ -37,7 +37,7 @@ from app.dao.notifications_dao import ( ) from app.dao.provider_details_dao import ( get_current_provider, - dao_toggle_sms_provider + # dao_toggle_sms_provider ) from app.dao.service_callback_api_dao import get_service_delivery_status_callback_api_for_service from app.dao.services_dao import ( @@ -274,7 +274,7 @@ def switch_current_sms_provider_on_slow_delivery(): ) ) - dao_toggle_sms_provider(current_provider.identifier) + # dao_toggle_sms_provider(current_provider.identifier) @notify_celery.task(name="delete-inbound-sms") diff --git a/tests/app/celery/test_scheduled_tasks.py b/tests/app/celery/test_scheduled_tasks.py index 979c2df5f..4b7e9f2ac 100644 --- a/tests/app/celery/test_scheduled_tasks.py +++ b/tests/app/celery/test_scheduled_tasks.py @@ -420,6 +420,7 @@ def test_switch_providers_on_slow_delivery_runs_if_config_set( assert get_notifications_mock.called is True +@pytest.mark.skip(reason="Not switching it on yet") def test_switch_providers_on_slow_delivery_switches_once_then_does_not_switch_if_already_switched( notify_api, mocker,