fix: delete methods are more resilient for SW360 responses

Author: t-graf

ChangeLog.md

@@ -5,6 +5,12 @@
 
 # SW360 Base Library for Python
 
+## V1.8.1
+
+* `delete_project` now works properly. Before it could have happened the you get a `JSONDecodeError`.
+  All delete methodds have been checked and made more resilient.
+* dependency updates.
+
 ## V1.8.0
 
 * Update `get_all_releases` to include `isNewClearingWithSourceAvailable` parameter:

sw360/components.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2019-2024 Siemens
+# Copyright (c) 2019-2025 Siemens
 # Copyright (c) 2022 BMW CarIT GmbH
 # All Rights Reserved.
 # Authors: thomas.graf@siemens.com, gernot.hillier@siemens.com
@@ -313,7 +313,8 @@ def delete_component(self, component_id: str) -> Optional[Dict[str, Any]]:
         response = self.api_delete(url)
         if response is not None:
             if response.ok:
-                return response.json()
+                if response.text:
+                    return response.json()
         return None
 
     def get_users_of_component(self, component_id: str) -> Optional[Dict[str, Any]]:

sw360/license.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2019-2022 Siemens
+# Copyright (c) 2019-2025 Siemens
 # Copyright (c) 2022 BMW CarIT GmbH
 # All Rights Reserved.
 # Authors: thomas.graf@siemens.com, gernot.hillier@siemens.com
@@ -68,6 +68,8 @@ def delete_license(self, license_shortname: str) -> Optional[bool]:
         :raises SW360Error: if there is a negative HTTP response
         """
 
+        # 2025-01-23: returns 500 - internal server error
+
         if not license_shortname:
             raise SW360Error(message="No license shortname provided!")
 

sw360/project.py

@@ -437,7 +437,8 @@ def delete_project(self, project_id: str) -> Optional[Dict[str, Any]]:
         response = self.api_delete(url)
         if response is not None:
             if response.ok:
-                return response.json()
+                if response.text:
+                    return response.json()
         return None
 
     def get_users_of_project(self, project_id: str) -> Optional[Dict[str, Any]]:

sw360/releases.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2019-2024 Siemens
+# Copyright (c) 2019-2025 Siemens
 # Copyright (c) 2022 BMW CarIT GmbH
 # All Rights Reserved.
 # Authors: thomas.graf@siemens.com, gernot.hillier@siemens.com
@@ -239,7 +239,8 @@ def delete_release(self, release_id: str) -> Optional[Dict[str, Any]]:
         response = self.api_delete(url)
         if response is not None:
             if response.ok:
-                return response.json()
+                if response.text:
+                    return response.json()
         return None
 
     def get_users_of_release(self, release_id: str) -> Optional[Dict[str, Any]]:

sw360/vendor.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2019-2023 Siemens
+# Copyright (c) 2019-2025 Siemens
 # Copyright (c) 2022 BMW CarIT GmbH
 # All Rights Reserved.
 # Authors: thomas.graf@siemens.com, gernot.hillier@siemens.com
@@ -93,7 +93,7 @@ def delete_vendor(self, vendor_id: str) -> Dict[str, Any]:
         :raises SW360Error: if there is a negative HTTP response
         """
 
-        # 2019-04-03: error 405 - not allowed
+        # 2019-04-03/2025-01-23: error 405 - not allowed
 
         if not vendor_id:
             raise SW360Error(message="No vendor id provided!")
@@ -103,5 +103,7 @@ def delete_vendor(self, vendor_id: str) -> Dict[str, Any]:
         response = self.api_delete(url)
         if response is not None:
             if response.ok:
-                return response.json()
-        raise SW360Error(response, url)
+                if response.text:
+                    return response.json()
+
+        return None

tests/test_sw360_components.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2020-2024 Siemens
+# Copyright (c) 2020-2025 Siemens
 # All Rights Reserved.
 # Author: thomas.graf@siemens.com
 #
@@ -767,7 +767,7 @@ def test_delete_component(self) -> None:
         responses.add(
             responses.DELETE,
             url=self.MYURL + "resource/api/components/123",
-            body="4",
+            body='{\n  "resourceId" : "468abcd6e7f849199d9323c9a16b2776",\n  "status" : 200\n}',
             status=200,
         )
 

tests/test_sw360_projects.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2019-2024 Siemens
+# Copyright (c) 2019-2025 Siemens
 # All Rights Reserved.
 # Author: thomas.graf@siemens.com
 #
@@ -1118,6 +1118,19 @@ def test_delete_project(self) -> None:
 
         lib.delete_project("123")
 
+    @responses.activate
+    def test_delete_project_empty_response(self) -> None:
+        lib = self.get_logged_in_lib()
+
+        responses.add(
+            responses.DELETE,
+            url=self.MYURL + "resource/api/projects/123",
+            body="",
+            status=200,
+        )
+
+        lib.delete_project("123")
+
     @responses.activate
     def test_delete_project_no_id(self) -> None:
         lib = self.get_logged_in_lib()

tests/test_sw360_releases.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2023-2024 Siemens
+# Copyright (c) 2023-2025 Siemens
 # All Rights Reserved.
 # Author: thomas.graf@siemens.com
 #
@@ -530,7 +530,7 @@ def test_delete_release(self) -> None:
         responses.add(
             responses.DELETE,
             url=self.MYURL + "resource/api/releases/123",
-            body="4",
+            body='{\n  "resourceId" : "8ddc6d15e0c4424ea756b8e0b77ad1e9",\n  "status" : 200\n}',
             status=200,
         )