Improve error handling in API and upload tests

Replaced status assertions with explicit error handling in test_api.py and test_upload.py. Now exceptions are raised with API error messages when responses are not OK, improving test reliability and clarity.

Author: JR-1991

tests/api/test_api.py

@@ -174,14 +174,22 @@ def test_token_right_create_dataset_rights(self):
                 )
             )
             resp = api_su.create_dataset(":root", ds.json())
-            pid = resp.json()["data"]["persistentId"]
-            assert resp.json()["status"] == "OK"
+            resp.raise_for_status()
+
+            resp = resp.json()
+            pid = resp["data"]["persistentId"]
+
+            if resp["status"] != "OK":
+                raise Exception(resp["message"])
 
             # with pytest.raises(ApiAuthorizationError):
             #     resp = api_nru.get_dataset(pid)
 
             resp = api_su.delete_dataset(pid)
-            assert resp.json()["status"] == "OK"
+            resp.raise_for_status()
+            resp = resp.json()
+            if resp["status"] != "OK":
+                raise Exception(resp["message"])
 
         def test_token_should_not_be_exposed_on_error(self):
             BASE_URL = os.getenv("BASE_URL")

tests/api/test_upload.py

@@ -184,7 +184,8 @@ def test_file_replacement_wo_metadata(self):
                 is_filepid=False,
             )
 
-            response.raise_for_status()
+            if not response.ok:
+                raise Exception(response.json()["message"])
 
         # Assert
         file_id = response.json()["data"]["files"][0]["dataFile"]["id"]
@@ -222,6 +223,9 @@ def test_file_replacement_w_metadata(self):
             json_str=df.json(),
         )
 
+        if not response.ok:
+            raise Exception(response.json()["message"])
+
         # Retrieve file ID
         file_id = response.json()["data"]["files"][0]["dataFile"]["id"]
 
@@ -248,6 +252,9 @@ def test_file_replacement_w_metadata(self):
                 is_filepid=False,
             )
 
+            if not response.ok:
+                raise Exception(response.json()["message"])
+
         # Assert
         file_id = response.json()["data"]["files"][0]["dataFile"]["id"]
         data_file = api.get_dataset(pid).json()["data"]["latestVersion"]["files"][0]