Add check to ensure status param in request is a valid JobStatus memb…

…er. Also add corresponding test

florist/api/routes/server/job.py

@@ -58,8 +58,15 @@ async def list_jobs_with_status(status: JobStatus, request: Request) -> List[Dic
     :param request: (fastapi.Request) the FastAPI request object.
     :return: (List[Dict[str, Any]]) A list where each entry is a dictionary with the attributes
         of a Job instance with the specified status.
+    :raises: (HTTPException) status 400 if status is not a member of the JobStatus Enum.
     """
     status = jsonable_encoder(status)
+
+    assert isinstance(status, str)
+    if status not in JobStatus._member_names_:
+        msg = f"status is not valid. status: {status}."
+        raise HTTPException(status_code=400, detail=msg)
+
     job_db = request.app.database[JOB_DATABASE_NAME]
     result = await job_db.find({"status": status}).to_list(MAX_RECORDS_TO_FETCH)
     assert isinstance(result, list)

florist/tests/integration/api/routes/server/test_job.py

@@ -325,3 +325,10 @@ async def test_list_jobs_with_status(mock_request) -> None:
     }
     assert isinstance(result_finished_successfully[0]["clients_info"][0]["_id"], str)
     assert isinstance(result_finished_successfully[0]["clients_info"][1]["_id"], str)
+
+async def test_list_jobs_with_invalid_status(mock_request) -> None:
+    with raises(HTTPException) as exception_info:
+        await list_jobs_with_status("NON_EXISTENT_STATUS", mock_request)
+
+    assert exception_info.value.status_code == 400
+    assert "status is not valid. status: NON_EXISTENT_STATUS" in exception_info.value.detail