Address PR feedback: improve exception handling and error messages

Co-authored-by: nolan1999 <54246789+nolan1999@users.noreply.github.com>

Author: Copilot

tests/unit/crud/test_job_tracking.py

@@ -21,4 +21,4 @@ def test_update_job_raises_job_deleted_exception(mock_put):
     with pytest.raises(JobDeletedException) as exc_info:
         update_job(job_id, JobStates.running)
 
-    assert "it was probably deleted by the user" in str(exc_info.value)
+    assert f"Job {job_id} not found; it was probably deleted by the user." in str(exc_info.value)

workerfacing_api/core/queue.py

@@ -510,7 +510,7 @@ def _update_job_status(
             # job probably deleted by user
             session.delete(job)
             session.commit()
-            raise JobDeletedException(f"Could not update job, probably deleted by user: {e}")
+            raise e from e
 
     def update_job_status(
         self,

workerfacing_api/crud/job_tracking.py

@@ -20,5 +20,5 @@ def update_job(
         headers={"x-api-key": settings.internal_api_key_secret},
     )
     if resp.status_code == 404:
-        raise JobDeletedException("it was probably deleted by the user")
+        raise JobDeletedException(f"Job {job_id} not found; it was probably deleted by the user.")
     resp.raise_for_status()