fix: raw response in client check (#31)

openfga · Aug 18, 2023 · ddf5f76 · ddf5f76
2 parents 12414f3 + 9ddda5e
commit ddf5f76
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 14 deletions.
diff --git a/client.ts b/client.ts
@@ -494,10 +494,7 @@ export class OpenFgaClient extends BaseAPI {
       },
       contextual_tuples: { tuple_keys: body.contextualTuples || [] },
       authorization_model_id: this.getAuthorizationModelId(options)
-    }, options).then(response => ({
-      ...response,
-      allowed: response.allowed || false,
-    }));
+    }, options);
   }
 
   /**
@@ -523,20 +520,16 @@ export class OpenFgaClient extends BaseAPI {
 
         const responses: ClientBatchCheckSingleResponse[] = [];
         for await (const singleCheckResponse of asyncPool(maxParallelRequests, body, (tuple) => this.check(tuple, { ...options, headers })
-          .then(({ allowed, $response: response }) => {
-            const result = {
-              allowed: allowed || false,
-              _request: tuple,
-            };
-            setNotEnumerableProperty(result, "$response", response);
-            return result as ClientBatchCheckSingleResponse;
+          .then(response => {
+            (response as ClientBatchCheckSingleResponse)._request = tuple;
+            return response as ClientBatchCheckSingleResponse;
           })
           .catch(err => {
             return {
-              allowed: false,
+              allowed: undefined,
               error: err,
               _request: tuple,
-            } as unknown as ClientBatchCheckSingleResponse;
+            };
           })
         )) {
           responses.push(singleCheckResponse);

diff --git a/tests/client.test.ts b/tests/client.test.ts
@@ -434,7 +434,7 @@ describe("OpenFGA Client", () => {
           .toMatchObject(expect.arrayContaining([
             { _request: tuples[0], allowed: true, },
             { _request: tuples[1], allowed: false },
-            { _request: tuples[2], allowed: false, error: expect.any(Error) },
+            { _request: tuples[2], error: expect.any(Error) },
           ]));
       });
     });