Merge pull request #412 from OpenAPI-Qraft/fix/parameters-and-body-payload

feat(react-client): fix a mutation-related destructuring runtime error when `body` is missing

Author: radist2s

.changeset/weak-knives-pay.md

@@ -0,0 +1,5 @@
+---
+'@openapi-qraft/react': patch
+---
+
+Fix a mutation-related destructuring runtime error when `body` is missing.

packages/react-client/src/callbacks/useMutation.ts

@@ -71,9 +71,7 @@ export const useMutation: <
                 .then(requestFnResponseResolver, requestFnResponseRejecter);
             }
           : function (parametersAndBodyPayload) {
-              const { body, ...parameters } = parametersAndBodyPayload as {
-                body: unknown;
-              };
+              const { body, ...parameters } = parametersAndBodyPayload ?? {};
 
               return qraftOptions
                 .requestFn(schema, {

packages/react-client/src/tests/qraftAPIClient.test.tsx

@@ -1281,8 +1281,11 @@ describe('Qraft uses Mutations', () => {
         mutateWithEmptyBody,
       } = result.current;
 
-      expect(mutateNoArgsWithVoidParameters.data).toBeUndefined();
+      expect(mutateNoArgsWithVoidParameters.isSuccess).toBeTruthy();
+      expect(mutateNoArgsWithVoidParameters.data).toEqual({});
+      expect(mutateNoArgsWithEmptyParameters.isSuccess).toBeTruthy();
       expect(mutateNoArgsWithEmptyParameters.data).toEqual({});
+      expect(mutateWithPredefinedParameters.isSuccess).toBeTruthy();
       expect(mutateWithPredefinedParameters.data).toEqual({
         query: { all: 'true' },
       });
@@ -1913,6 +1916,40 @@ describe('Qraft uses getMutationCache', () => {
       });
     });
 
+    it('supports getMutationCache().find() with status success and useMutation options mutationKey', async () => {
+      const { qraft, queryClient } = createClient();
+      const mutationKey = qraft.files.deleteFiles.getMutationKey();
+
+      const { result: mutationResult } = renderHook(
+        () =>
+          qraft.files.deleteFiles.useMutation(undefined, {
+            gcTime: Infinity,
+            mutationKey,
+          }),
+        {
+          wrapper: (props) => (
+            <Providers {...props} queryClient={queryClient} />
+          ),
+        }
+      );
+
+      act(() => {
+        mutationResult.current.mutate();
+      });
+
+      await waitFor(() => {
+        expect(mutationResult.current.status).toEqual('success');
+      });
+
+      const foundMutation = queryClient.getMutationCache().find({
+        mutationKey,
+        status: 'success',
+      });
+
+      expect(foundMutation).toBeDefined();
+      expect(foundMutation?.state.status).toEqual('success');
+    });
+
     it('supports getMutationCache().find() with partial not exact parameters', async () => {
       const { qraft, queryClient } = createClient();