Add response status checks to upload tests

JR-1991 · JR-1991 · commit aab9a9355b00 · 2025-09-24T16:10:29.000+02:00
Inserted response.raise_for_status() calls after API requests in test_upload.py to ensure HTTP errors are caught during testing. Also refactored some assert statements for improved readability.
diff --git a/tests/api/test_upload.py b/tests/api/test_upload.py
@@ -161,6 +161,8 @@ def test_file_replacement_wo_metadata(self):
             json_str=df.json(),
         )
 
+        response.raise_for_status()
+
         # Retrieve file ID
         file_id = response.json()["data"]["files"][0]["dataFile"]["id"]
 
@@ -182,6 +184,8 @@ def test_file_replacement_wo_metadata(self):
                 is_filepid=False,
             )
 
+            response.raise_for_status()
+
         # Assert
         file_id = response.json()["data"]["files"][0]["dataFile"]["id"]
         content = data_api.get_datafile(file_id, is_pid=False).text

Original file line number	Diff line number	Diff line change
`@@ -161,6 +161,8 @@ def test_file_replacement_wo_metadata(self):`
`161`	`161`	`json_str=df.json(),`
`162`	`162`	`)`
`163`	`163`
	`164`	`+ response.raise_for_status()`
	`165`	`+`
`164`	`166`	`# Retrieve file ID`
`165`	`167`	`file_id = response.json()["data"]["files"][0]["dataFile"]["id"]`
`166`	`168`
`@@ -182,6 +184,8 @@ def test_file_replacement_wo_metadata(self):`
`182`	`184`	`is_filepid=False,`
`183`	`185`	`)`
`184`	`186`
	`187`	`+ response.raise_for_status()`
	`188`	`+`
`185`	`189`	`# Assert`
`186`	`190`	`file_id = response.json()["data"]["files"][0]["dataFile"]["id"]`
`187`	`191`	`content = data_api.get_datafile(file_id, is_pid=False).text`