fix(sdk): return the result from pollUntilHasData and pollUntilOk retries

.changeset/poll-api-return-result.md

@@ -0,0 +1,5 @@
+---
+'@relayprotocol/relay-sdk': patch
+---
+
+Fix `pollUntilHasData` and `pollUntilOk` resolving to `undefined` when the data or ok response arrives after the first attempt

packages/sdk/src/utils/pollApi.test.ts

@@ -0,0 +1,32 @@
+import { afterEach, describe, expect, it, vi } from 'vitest'
+import { axios } from './axios'
+import { pollUntilHasData, pollUntilOk } from './pollApi'
+
+const request = { url: 'https://api.relay.link/requests/v2' }
+
+describe('pollApi', () => {
+  afterEach(() => {
+    vi.restoreAllMocks()
+    vi.useRealTimers()
+  })
+
+  it('pollUntilHasData should resolve to the data found on a later attempt', async () => {
+    vi.useFakeTimers()
+    vi.spyOn(axios, 'request')
+      .mockResolvedValueOnce({ data: { requests: [] } })
+      .mockResolvedValueOnce({ data: { requests: [{ id: '0x1' }] } })
+
+    const result = pollUntilHasData(request, (json) => json.requests.length > 0)
+    await vi.advanceTimersByTimeAsync(5000)
+
+    await expect(result).resolves.toEqual({ requests: [{ id: '0x1' }] })
+  })
+
+  it('pollUntilOk should resolve to true when a later attempt is ok', async () => {
+    vi.spyOn(axios, 'request')
+      .mockResolvedValueOnce({ status: 202 })
+      .mockResolvedValueOnce({ status: 200 })
+
+    await expect(pollUntilOk(request, undefined, 15, 0, 0)).resolves.toBe(true)
+  })
+})

packages/sdk/src/utils/pollApi.ts

@@ -34,7 +34,7 @@ export async function pollUntilHasData(
   // The response is still unchanged. Check again in five seconds
   await new Promise((resolve) => setTimeout(resolve, 5000))
   attemptCount++
-  await pollUntilHasData(request, dataParser, maximumAttempts, attemptCount)
+  return pollUntilHasData(request, dataParser, maximumAttempts, attemptCount)
 }
 
 /**
@@ -69,7 +69,7 @@ export async function pollUntilOk(
     // The response is still unchanged
     await new Promise((resolve) => setTimeout(resolve, pollingInterval))
     attemptCount++
-    await pollUntilOk(
+    return pollUntilOk(
       request,
       validate,
       maximumAttempts,