darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-01 15:46:07 -05:00
Code Issues Packages Projects Releases Wiki Activity

Switch providers on slow delivery only produces logs

Browse Source
This commit is contained in:
Pea Tyczynska
2018-12-05 14:46:21 +00:00
parent 418060fbdb
commit 6938600ab8
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/celery/scheduled_tasks.py
View File

@@ -37,7 +37,7 @@ from app.dao.notifications_dao import (
) )
from app.dao.provider_details_dao import ( from app.dao.provider_details_dao import (
get_current_provider, 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.service_callback_api_dao import get_service_delivery_status_callback_api_for_service
from app.dao.services_dao import ( 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") @notify_celery.task(name="delete-inbound-sms")

1
tests/app/celery/test_scheduled_tasks.py
View File

@@ -420,6 +420,7 @@ def test_switch_providers_on_slow_delivery_runs_if_config_set(
assert get_notifications_mock.called is True 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( def test_switch_providers_on_slow_delivery_switches_once_then_does_not_switch_if_already_switched(
notify_api, notify_api,
mocker, mocker,