OAProc: fix response type on delete_job (#1721) (#1726)

This commit is contained in:
Tom Kralidis
2024-07-15 21:17:51 -04:00
committed by GitHub
parent 0169e43329
commit e8ce02167f
2 changed files with 6 additions and 4 deletions
+3 -4
View File
@@ -512,9 +512,7 @@ def get_job_result(api: API, request: APIRequest,
return headers, HTTPStatus.OK, content
def delete_job(
api: API, request: APIRequest, job_id
) -> Tuple[dict, int, str]:
def delete_job(api: API, request: APIRequest, job_id) -> Tuple[dict, int, str]:
"""
Delete a process job
@@ -522,6 +520,7 @@ def delete_job(
:returns: tuple of headers, status code, content
"""
response_headers = request.get_response_headers(
SYSTEM_LOCALE, **api.api_headers)
try:
@@ -555,7 +554,7 @@ def delete_job(
)
LOGGER.info(response)
# TODO: this response does not have any headers
return {}, http_status, response
return {}, http_status, to_json(response, api.pretty_print)
def get_oas_30(cfg: dict, locale: str) -> tuple[list[dict[str, str]], dict[str, dict]]: # noqa
+3
View File
@@ -386,7 +386,10 @@ def test_delete_job(api_):
job_id = _execute_a_job(api_)
rsp_headers, code, response = delete_job(api_, mock_api_request(), job_id)
data = json.loads(response)
assert code == HTTPStatus.OK
assert data['message'] == 'Job dismissed'
rsp_headers, code, response = delete_job(api_, mock_api_request(), job_id)
assert code == HTTPStatus.NOT_FOUND