diff --git a/app/letters/send_letter_jobs.py b/app/letters/send_letter_jobs.py index 806689567..7030b9bc5 100644 --- a/app/letters/send_letter_jobs.py +++ b/app/letters/send_letter_jobs.py @@ -17,7 +17,7 @@ def send_letter_jobs(): job_ids = validate(request.get_json(), letter_job_ids) notify_celery.send_task(name="send-files-to-dvla", args=(job_ids['job_ids'],), queue="process-ftp") - return "Task created to send files to DVLA" + return jsonify(data={"response": "Task created to send files to DVLA"}), 201 @letter_job.route('/letter-jobs', methods=['GET']) diff --git a/tests/app/letters/test_send_letter_jobs.py b/tests/app/letters/test_send_letter_jobs.py index f006a981d..7f1d88c85 100644 --- a/tests/app/letters/test_send_letter_jobs.py +++ b/tests/app/letters/test_send_letter_jobs.py @@ -16,8 +16,8 @@ def test_send_letter_jobs(client, mocker): data=json.dumps(job_ids), headers=[('Content-Type', 'application/json'), auth_header]) - assert response.status_code == 200 - assert response.get_data(as_text=True) == "Task created to send files to DVLA" + assert response.status_code == 201 + assert json.loads(response.get_data())['data'] == {'response': "Task created to send files to DVLA"} mock_celery.assert_called_once_with(name="send-files-to-dvla", args=(job_ids['job_ids'],),