fix(tests): update S3Error instantiation in unit tests for proper mocking

Author: ArthurTonial

tests/extensibility/unit/test_client.py

@@ -277,7 +277,8 @@ def _success_payload(workflow_id: str = "wf-001") -> str:
                                             "json": {
                                                 "message_id": "msg-1",
                                                 "context_id": "ctx-1",
-                                                "role": 2,
+                                                "role": "agent",
+                                                "parts": [{"kind": "text", "text": ""}],
                                             }
                                         }
                                     ]
@@ -311,7 +312,8 @@ def _poll_success_payload() -> str:
                                             "json": {
                                                 "message_id": "msg-2",
                                                 "context_id": "ctx-1",
-                                                "role": 2,
+                                                "role": "agent",
+                                                "parts": [{"kind": "text", "text": ""}],
                                             }
                                         }
                                     ]

tests/objectstore/unit/test_s3_client.py

@@ -175,7 +175,7 @@ def test_get_object_success(self, mock_minio_class):
     @patch('sap_cloud_sdk.objectstore._s3.Minio')
     def test_get_object_not_found(self, mock_minio_class):
         mock_minio = Mock()
-        s3_error = S3Error("NoSuchKey", "Key not found", "test.txt", "123", "456", Mock())
+        s3_error = S3Error(Mock(), "NoSuchKey", "Key not found", "test.txt", "123", "456")
         mock_minio.get_object.side_effect = s3_error
         mock_minio_class.return_value = mock_minio
 
@@ -200,7 +200,7 @@ def test_delete_object_success(self, mock_minio_class):
     @patch('sap_cloud_sdk.objectstore._s3.Minio')
     def test_delete_object_not_found_ignored(self, mock_minio_class):
         mock_minio = Mock()
-        s3_error = S3Error("NoSuchKey", "Key not found", "test.txt", "123", "456", Mock())
+        s3_error = S3Error(Mock(), "NoSuchKey", "Key not found", "test.txt", "123", "456")
         mock_minio.remove_object.side_effect = s3_error
         mock_minio_class.return_value = mock_minio
 
@@ -273,7 +273,7 @@ def test_head_object_success(self, mock_minio_class):
     @patch('sap_cloud_sdk.objectstore._s3.Minio')
     def test_head_object_not_found(self, mock_minio_class):
         mock_minio = Mock()
-        s3_error = S3Error("NoSuchKey", "Key not found", "test.txt", "123", "456", Mock())
+        s3_error = S3Error(Mock(), "NoSuchKey", "Key not found", "test.txt", "123", "456")
         mock_minio.stat_object.side_effect = s3_error
         mock_minio_class.return_value = mock_minio
 
@@ -296,7 +296,7 @@ def test_object_exists_true(self, mock_minio_class):
     @patch('sap_cloud_sdk.objectstore._s3.Minio')
     def test_object_exists_false(self, mock_minio_class):
         mock_minio = Mock()
-        s3_error = S3Error("NoSuchKey", "Key not found", "test.txt", "123", "456", Mock())
+        s3_error = S3Error(Mock(), "NoSuchKey", "Key not found", "test.txt", "123", "456")
         mock_minio.stat_object.side_effect = s3_error
         mock_minio_class.return_value = mock_minio