lint

Author: swaroopAkkineniWorkos

tests/test_authorization_resource_crud.py

@@ -42,17 +42,23 @@ def test_create_resource_required_fields_only(
 
         resource = syncify(
             self.authorization.create_resource(
-                resource_type="document",
+                resource_type_slug="document",
                 organization_id="org_01EHT88Z8J8795GZNQ4ZP1J81T",
+                external_id="ext_123",
+                name="Test Resource",
+                parent={"parent_resource_id": "res_01PARENT"},
             )
         )
 
         assert resource.id == "res_01ABC"
         assert request_kwargs["method"] == "post"
         assert request_kwargs["url"].endswith("/authorization/resources")
         assert request_kwargs["json"] == {
-            "resource_type": "document",
+            "resource_type_slug": "document",
             "organization_id": "org_01EHT88Z8J8795GZNQ4ZP1J81T",
+            "external_id": "ext_123",
+            "name": "Test Resource",
+            "parent_resource_id": "res_01PARENT",
         }
 
     def test_create_resource_with_all_optional_fields(
@@ -64,20 +70,22 @@ def test_create_resource_with_all_optional_fields(
 
         syncify(
             self.authorization.create_resource(
-                resource_type="document",
+                resource_type_slug="document",
                 organization_id="org_01EHT88Z8J8795GZNQ4ZP1J81T",
                 external_id="ext_123",
-                meta={"key": "value"},
-                parent={"resource_id": "res_01PARENT"},
+                name="Test Resource",
+                parent={"parent_resource_id": "res_01PARENT"},
+                description="A test document",
             )
         )
 
         assert request_kwargs["json"] == {
-            "resource_type": "document",
+            "resource_type_slug": "document",
             "organization_id": "org_01EHT88Z8J8795GZNQ4ZP1J81T",
             "external_id": "ext_123",
-            "meta": {"key": "value"},
-            "parent": {"resource_id": "res_01PARENT"},
+            "name": "Test Resource",
+            "parent_resource_id": "res_01PARENT",
+            "description": "A test document",
         }
 
     def test_create_resource_with_parent_by_id(
@@ -89,13 +97,15 @@ def test_create_resource_with_parent_by_id(
 
         syncify(
             self.authorization.create_resource(
-                resource_type="document",
+                resource_type_slug="document",
                 organization_id="org_01EHT88Z8J8795GZNQ4ZP1J81T",
-                parent={"resource_id": "res_01PARENT"},
+                external_id="ext_123",
+                name="Test Resource",
+                parent={"parent_resource_id": "res_01PARENT"},
             )
         )
 
-        assert request_kwargs["json"]["parent"] == {"resource_id": "res_01PARENT"}
+        assert request_kwargs["json"]["parent_resource_id"] == "res_01PARENT"
 
     def test_create_resource_with_parent_by_external_id(
         self, mock_resource, capture_and_mock_http_client_request
@@ -106,21 +116,19 @@ def test_create_resource_with_parent_by_external_id(
 
         syncify(
             self.authorization.create_resource(
-                resource_type="document",
+                resource_type_slug="document",
                 organization_id="org_01EHT88Z8J8795GZNQ4ZP1J81T",
+                external_id="ext_123",
+                name="Test Resource",
                 parent={
-                    "organization_id": "org_01EHT88Z8J8795GZNQ4ZP1J81T",
-                    "resource_type": "folder",
-                    "external_id": "ext_parent_456",
+                    "parent_resource_external_id": "ext_parent_456",
+                    "parent_resource_type_slug": "folder",
                 },
             )
         )
 
-        assert request_kwargs["json"]["parent"] == {
-            "organization_id": "org_01EHT88Z8J8795GZNQ4ZP1J81T",
-            "resource_type": "folder",
-            "external_id": "ext_parent_456",
-        }
+        assert request_kwargs["json"]["parent_resource_external_id"] == "ext_parent_456"
+        assert request_kwargs["json"]["parent_resource_type_slug"] == "folder"
 
     # --- update_resource ---
 
@@ -134,14 +142,18 @@ def test_update_resource_with_meta(
         resource = syncify(
             self.authorization.update_resource(
                 "res_01ABC",
-                meta={"updated_key": "updated_value"},
+                name="Updated Name",
+                description="Updated description",
             )
         )
 
         assert resource.id == "res_01ABC"
         assert request_kwargs["method"] == "patch"
         assert request_kwargs["url"].endswith("/authorization/resources/res_01ABC")
-        assert request_kwargs["json"] == {"meta": {"updated_key": "updated_value"}}
+        assert request_kwargs["json"] == {
+            "name": "Updated Name",
+            "description": "Updated description",
+        }
 
     def test_update_resource_without_meta(
         self, mock_resource, capture_and_mock_http_client_request