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

Merge pull request #2755 from alphagov/fix-serialization-error

Browse Source 
Fix serialisation error
This commit is contained in:
Rebecca Law
2020-03-17 10:44:47 +00:00
committed by GitHub
parent 90783c0f23 51f43563d3
commit ac07ea3e3f
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/celery/scheduled_tasks.py
View File

@@ -205,7 +205,7 @@ def replay_created_notifications():
current_app.logger.info(msg) current_app.logger.info(msg)
for letter in letters: for letter in letters:
create_letters_pdf.apply_async([letter.id], queue=QueueNames.LETTERS) create_letters_pdf.apply_async([str(letter.id)], queue=QueueNames.LETTERS)
@notify_celery.task(name='check-precompiled-letter-state') @notify_celery.task(name='check-precompiled-letter-state')

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

@@ -329,8 +329,8 @@ def test_replay_created_notifications_create_letters_pdf_tasks_for_letters_not_r
replay_created_notifications() replay_created_notifications()
calls = [call([notification_1.id], queue=QueueNames.LETTERS), calls = [call([str(notification_1.id)], queue=QueueNames.LETTERS),
call([notification_2.id], queue=QueueNames.LETTERS), call([str(notification_2.id)], queue=QueueNames.LETTERS),
] ]
mock_task.assert_has_calls(calls, any_order=True) mock_task.assert_has_calls(calls, any_order=True)