fix: unit tests

Author: azmeuk

scim2_client/client.py

@@ -205,10 +205,6 @@ def get_resource_model(self, name: str) -> Optional[type[Resource]]:
     def _check_resource_model(
         self, resource_model: type[Resource], payload=None
     ) -> None:
-        # We need to check the actual schema names, comparing the class
-        # types does not work because if the resource_models are
-        # discovered. The classes might differ:
-        #   <class 'scim2_models.rfc7643.user.User'> vs <class 'scim2_models.rfc7643.schema.User'>
         schema_to_check = resource_model.model_fields["schemas"].default[0]
         for element in self.resource_models:
             schema = element.model_fields["schemas"].default[0]

tests/test_delete.py

@@ -1,12 +1,16 @@
 import pytest
 from scim2_models import Error
-from scim2_models import Group
+from scim2_models import Resource
 from scim2_models import User
 
 from scim2_client import RequestNetworkError
 from scim2_client import SCIMRequestError
 
 
+class UnregisteredResource(Resource):
+    schemas: list[str] = ["urn:test:schemas:UnregisteredResource"]
+
+
 def test_delete_user(httpserver, sync_client):
     """Nominal case for a User deletion."""
     httpserver.expect_request(
@@ -45,7 +49,7 @@ def test_errors(httpserver, code, sync_client):
 def test_invalid_resource_model(httpserver, sync_client):
     """Test that resource_models passed to the method must be part of SCIMClient.resource_models."""
     with pytest.raises(SCIMRequestError, match=r"Unknown resource type"):
-        sync_client.delete(Group(display_name="foobar"), id="foobar")
+        sync_client.delete(UnregisteredResource, id="foobar")
 
 
 def test_dont_check_response_payload(httpserver, sync_client):