Merge pull request #3358 from alphagov/remove-yesterdays-planned-tests-on-govuk-alerts

Republish gov.uk/alerts every night to clear down planned tests
This commit is contained in:
Chris Hill-Scott
2021-11-15 15:34:21 +00:00
committed by GitHub
3 changed files with 27 additions and 0 deletions

View File

@@ -334,3 +334,11 @@ def auto_expire_broadcast_messages():
name=TaskNames.PUBLISH_GOVUK_ALERTS,
queue=QueueNames.GOVUK_ALERTS
)
@notify_celery.task(name='remove-yesterdays-planned-tests-on-govuk-alerts')
def remove_yesterdays_planned_tests_on_govuk_alerts():
notify_celery.send_task(
name=TaskNames.PUBLISH_GOVUK_ALERTS,
queue=QueueNames.GOVUK_ALERTS
)

View File

@@ -332,6 +332,11 @@ class Config(object):
'schedule': timedelta(minutes=5),
'options': {'queue': QueueNames.PERIODIC}
},
'remove-yesterdays-planned-tests-on-govuk-alerts': {
'task': 'remove-yesterdays-planned-tests-on-govuk-alerts',
'schedule': crontab(hour=00, minute=00),
'options': {'queue': QueueNames.PERIODIC}
},
}
}

View File

@@ -19,6 +19,7 @@ from app.celery.scheduled_tasks import (
check_job_status,
delete_invitations,
delete_verify_codes,
remove_yesterdays_planned_tests_on_govuk_alerts,
replay_created_notifications,
run_scheduled_jobs,
switch_current_sms_provider_on_slow_delivery,
@@ -700,3 +701,16 @@ def test_auto_expire_broadcast_messages(
)
else:
assert not mock_celery.called
def test_remove_yesterdays_planned_tests_on_govuk_alerts(
mocker
):
mock_celery = mocker.patch('app.celery.scheduled_tasks.notify_celery.send_task')
remove_yesterdays_planned_tests_on_govuk_alerts()
mock_celery.assert_called_once_with(
name=TaskNames.PUBLISH_GOVUK_ALERTS,
queue=QueueNames.GOVUK_ALERTS
)