refactor: avoid bare exception catching

azmeuk · azmeuk · commit dafdfe37f68d · 2025-08-04T18:05:21.000+02:00
diff --git a/scim2_tester/checkers/resource_delete.py b/scim2_tester/checkers/resource_delete.py
@@ -1,5 +1,6 @@
 from typing import Any
 
+from scim2_client import SCIMClientError
 from scim2_models import Resource
 
 from ..utils import CheckContext
@@ -47,7 +48,7 @@ def object_deletion(
                 reason=f"{model.__name__} object with id {test_obj.id} still exists after deletion",
             )
         ]
-    except Exception:
+    except SCIMClientError:
         pass
 
     return [

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`from typing import Any`
`2`	`2`
	`3`	`+from scim2_client import SCIMClientError`
`3`	`4`	`from scim2_models import Resource`
`4`	`5`
`5`	`6`	`from ..utils import CheckContext`
`@@ -47,7 +48,7 @@ def object_deletion(`
`47`	`48`	`reason=f"{model.__name__} object with id {test_obj.id} still exists after deletion",`
`48`	`49`	`)`
`49`	`50`	`]`
`50`		`- except Exception:`
	`51`	`+ except SCIMClientError:`
`51`	`52`	`pass`
`52`	`53`
`53`	`54`	`return [`