From eff4f644265c38d8832f97ec6dcb815b2f31d7f7 Mon Sep 17 00:00:00 2001 From: Daniel Roth Date: Tue, 14 Jul 2026 13:43:05 +0000 Subject: [PATCH] =?UTF-8?q?Raise=20loudly=20when=20RdSAP=20Summary=20fetch?= =?UTF-8?q?=20returns=20an=20HTTP=20error=20=F0=9F=9F=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-Authored-By: Claude Opus 4.8 (1M context) --- backend/pashub_fetcher/tests/test_pashub_client.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/backend/pashub_fetcher/tests/test_pashub_client.py b/backend/pashub_fetcher/tests/test_pashub_client.py index 1e6bbadb3..48731eea6 100644 --- a/backend/pashub_fetcher/tests/test_pashub_client.py +++ b/backend/pashub_fetcher/tests/test_pashub_client.py @@ -333,3 +333,17 @@ def test_get_rdsap_summary_raises_unauthorized_on_401() -> None: client.session.get = MagicMock(return_value=make_response(status_code=401)) with pytest.raises(UnauthorizedError): client.get_rdsap_summary_by_job_id("job-1") + + +def test_get_rdsap_summary_raises_on_http_error() -> None: + # Arrange + client = make_client() + + # Act / Assert — a 404/error must raise loudly, not resolve to empty values + client.session.get = MagicMock( + return_value=make_response( + status_code=404, raise_on_status=requests.HTTPError("not found") + ) + ) + with pytest.raises(requests.HTTPError): + client.get_rdsap_summary_by_job_id("job-1")