test(solid-query/useMutation): split 'should pass meta to mutation' into separate success and error tests

packages/solid-query/src/__tests__/useMutation.test.tsx

@@ -1244,29 +1244,61 @@ describe('useMutation', () => {
     consoleMock.mockRestore()
   })
 
-  it('should pass meta to mutation', async () => {
-    const errorMock = vi.fn()
+  it('should pass meta to mutation on success', async () => {
     const successMock = vi.fn()
 
     const queryClientMutationMeta = new QueryClient({
       mutationCache: new MutationCache({
         onSuccess: (_, __, ___, mutation) => {
           successMock(mutation.meta?.metaSuccessMessage)
         },
-        onError: (_, __, ___, mutation) => {
-          errorMock(mutation.meta?.metaErrorMessage)
-        },
       }),
     })
 
     const metaSuccessMessage = 'mutation succeeded'
-    const metaErrorMessage = 'mutation failed'
 
     function Page() {
       const mutationSucceed = useMutation(() => ({
         mutationFn: () => Promise.resolve(''),
         meta: { metaSuccessMessage },
       }))
+
+      return (
+        <div>
+          <button onClick={() => mutationSucceed.mutate()}>succeed</button>
+          {mutationSucceed.isSuccess && <div>successTest</div>}
+        </div>
+      )
+    }
+
+    const rendered = render(() => (
+      <QueryClientProvider client={queryClientMutationMeta}>
+        <Page />
+      </QueryClientProvider>
+    ))
+
+    fireEvent.click(rendered.getByText('succeed'))
+    await vi.advanceTimersByTimeAsync(0)
+    expect(rendered.queryByText('successTest')).not.toBeNull()
+
+    expect(successMock).toHaveBeenCalledTimes(1)
+    expect(successMock).toHaveBeenCalledWith(metaSuccessMessage)
+  })
+
+  it('should pass meta to mutation on error', async () => {
+    const errorMock = vi.fn()
+
+    const queryClientMutationMeta = new QueryClient({
+      mutationCache: new MutationCache({
+        onError: (_, __, ___, mutation) => {
+          errorMock(mutation.meta?.metaErrorMessage)
+        },
+      }),
+    })
+
+    const metaErrorMessage = 'mutation failed'
+
+    function Page() {
       const mutationError = useMutation(() => ({
         mutationFn: () => {
           throw new Error('')
@@ -1276,9 +1308,7 @@ describe('useMutation', () => {
 
       return (
         <div>
-          <button onClick={() => mutationSucceed.mutate()}>succeed</button>
           <button onClick={() => mutationError.mutate()}>error</button>
-          {mutationSucceed.isSuccess && <div>successTest</div>}
           {mutationError.isError && <div>errorTest</div>}
         </div>
       )
@@ -1290,15 +1320,10 @@ describe('useMutation', () => {
       </QueryClientProvider>
     ))
 
-    fireEvent.click(rendered.getByText('succeed'))
     fireEvent.click(rendered.getByText('error'))
     await vi.advanceTimersByTimeAsync(0)
-    expect(rendered.queryByText('successTest')).not.toBeNull()
     expect(rendered.queryByText('errorTest')).not.toBeNull()
 
-    expect(successMock).toHaveBeenCalledTimes(1)
-    expect(successMock).toHaveBeenCalledWith(metaSuccessMessage)
-
     expect(errorMock).toHaveBeenCalledTimes(1)
     expect(errorMock).toHaveBeenCalledWith(metaErrorMessage)
   })