fix(package/gqty): useTransactionQuery should work without suspense

[vicary]

Fixes #2187

Screen.Recording.2025-03-04.at.16.58.33.mov

[github-actions]

🚀 Snapshot Release (canary)

The latest changes of this PR are available as canary on npm (based on the declared changesets):

Package	Version	Info
@gqty/react	3.1.1-canary-20250304090326.1da6588049480a27515b511171d5509edd3b0ec9	npm ↗︎ unpkg ↗︎

[vicary]

@cdomigan Please feel free to test this canary, I'll release the patch once you find it working!

[cdomigan]

Not sure if I'm using it wrong, but I'm getting partial results returned, eg adding the following test to packages/react/test/useTransactionQuery.test.tsx:

  it('should not return partially undefined results', async () => {
    const { useTransactionQuery } = await createReactTestClient();

    const { result } = renderHook(() =>
      useTransactionQuery((query) => {
        const human = query.human();
        return {
          name: human.name
        };
      }, { suspense: false })
    );

    expect(result.current.data).toBeUndefined();
    await waitFor(() => expect(result.current.data).not.toBeUndefined());
    expect(result.current.data?.name).toBeInstanceOf(String);
  });

I'm seeing

packages/react test:   ● useTransactionQuery › should not return partially undefined results
packages/react test:     expect(received).toBeUndefined()
packages/react test:     Received: {"name": undefined}

The docs say that the data returned is only available after a successful fetch., so I had expected data to be undefined until it had successfully resolved?

[vicary]

@cdomigan You may have to wait until isLoading turns true and then false again to mean a successful fetch.