From e8ce02167fd670bb515f8216d699a3e9abca21d4 Mon Sep 17 00:00:00 2001 From: Tom Kralidis Date: Mon, 15 Jul 2024 21:17:51 -0400 Subject: [PATCH] OAProc: fix response type on delete_job (#1721) (#1726) --- pygeoapi/api/processes.py | 7 +++---- tests/api/test_processes.py | 3 +++ 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/pygeoapi/api/processes.py b/pygeoapi/api/processes.py index 8ba7599..3eb9ea6 100644 --- a/pygeoapi/api/processes.py +++ b/pygeoapi/api/processes.py @@ -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 diff --git a/tests/api/test_processes.py b/tests/api/test_processes.py index a339728..bfd5ca9 100644 --- a/tests/api/test_processes.py +++ b/tests/api/test_processes.py @@ -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