fix: list users contextual tuples should be a list directly

ewanharris · ewanharris · commit 92494b56f66e · 2024-11-05T17:34:33.000Z
It differs from other endpoints in that it isn't an object but just an array directly
diff --git a/openfga_sdk/client/client.py b/openfga_sdk/client/client.py
@@ -744,9 +744,7 @@ async def list_users(
         )
 
         if body.contextual_tuples:
-            req_body.contextual_tuples = ContextualTupleKeys(
-                tuple_keys=convert_tuple_keys(body.contextual_tuples)
-            )
+            req_body.contextual_tuples = convert_tuple_keys(body.contextual_tuples)
 
         api_response = await self._api.list_users(body=req_body, **kwargs)
 
diff --git a/openfga_sdk/sync/client/client.py b/openfga_sdk/sync/client/client.py
@@ -733,9 +733,7 @@ def list_users(self, body: ClientListUsersRequest, options: dict[str, str] = Non
         )
 
         if body.contextual_tuples:
-            req_body.contextual_tuples = ContextualTupleKeys(
-                tuple_keys=convert_tuple_keys(body.contextual_tuples)
-            )
+            req_body.contextual_tuples = convert_tuple_keys(body.contextual_tuples)
 
         api_response = self._api.list_users(body=req_body, **kwargs)
 
diff --git a/test/client/client_test.py b/test/client/client_test.py
@@ -2523,20 +2523,18 @@ async def test_list_users(self, mock_request):
                         {"type": "user"},
                         {"type": "team", "relation": "member"},
                     ],
-                    "contextual_tuples": {
-                        "tuple_keys": [
-                            {
-                                "user": "user:81684243-9356-4421-8fbf-a4f8d36aa31b",
-                                "relation": "editor",
-                                "object": "folder:product",
-                            },
-                            {
-                                "user": "folder:product",
-                                "relation": "parent",
-                                "object": "document:roadmap",
-                            },
-                        ]
-                    },
+                    "contextual_tuples": [
+                        {
+                            "user": "user:81684243-9356-4421-8fbf-a4f8d36aa31b",
+                            "relation": "editor",
+                            "object": "folder:product",
+                        },
+                        {
+                            "user": "folder:product",
+                            "relation": "parent",
+                            "object": "document:roadmap",
+                        },
+                    ],
                     "context": {},
                     "consistency": "MINIMIZE_LATENCY",
                 },
diff --git a/test/sync/client/client_test.py b/test/sync/client/client_test.py
@@ -2526,20 +2526,18 @@ def test_list_users(self, mock_request):
                         {"type": "user"},
                         {"type": "team", "relation": "member"},
                     ],
-                    "contextual_tuples": {
-                        "tuple_keys": [
-                            {
-                                "user": "user:81684243-9356-4421-8fbf-a4f8d36aa31b",
-                                "relation": "editor",
-                                "object": "folder:product",
-                            },
-                            {
-                                "user": "folder:product",
-                                "relation": "parent",
-                                "object": "document:roadmap",
-                            },
-                        ]
-                    },
+                    "contextual_tuples": [
+                        {
+                            "user": "user:81684243-9356-4421-8fbf-a4f8d36aa31b",
+                            "relation": "editor",
+                            "object": "folder:product",
+                        },
+                        {
+                            "user": "folder:product",
+                            "relation": "parent",
+                            "object": "document:roadmap",
+                        },
+                    ],
                     "context": {},
                     "consistency": "MINIMIZE_LATENCY",
                 },

Original file line number	Diff line number	Diff line change
`@@ -744,9 +744,7 @@ async def list_users(`
`744`	`744`	`)`
`745`	`745`
`746`	`746`	`if body.contextual_tuples:`
`747`		`- req_body.contextual_tuples = ContextualTupleKeys(`
`748`		`- tuple_keys=convert_tuple_keys(body.contextual_tuples)`
`749`		`- )`
	`747`	`+ req_body.contextual_tuples = convert_tuple_keys(body.contextual_tuples)`
`750`	`748`
`751`	`749`	`api_response = await self._api.list_users(body=req_body, **kwargs)`
`752`	`750`
Original file line number	Diff line number	Diff line change
`@@ -733,9 +733,7 @@ def list_users(self, body: ClientListUsersRequest, options: dict[str, str] = Non`
`733`	`733`	`)`
`734`	`734`
`735`	`735`	`if body.contextual_tuples:`
`736`		`- req_body.contextual_tuples = ContextualTupleKeys(`
`737`		`- tuple_keys=convert_tuple_keys(body.contextual_tuples)`
`738`		`- )`
	`736`	`+ req_body.contextual_tuples = convert_tuple_keys(body.contextual_tuples)`
`739`	`737`
`740`	`738`	`api_response = self._api.list_users(body=req_body, **kwargs)`
`741`	`739`