used proper waitFor syntax

electric-sql · Dec 10, 2024 · 101a968 · 101a968
1 parent 288fe0c
commit 101a968
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 15 deletions.
diff --git a/packages/react-hooks/test/react-hooks.test.tsx b/packages/react-hooks/test/react-hooks.test.tsx
@@ -363,9 +363,7 @@ describe(`useShape`, () => {
     })
 
     // And wait until it's definitely seen
-    await waitFor(async () => {
-      return parallelWaiterStream.isUpToDate || (await sleep(50))
-    })
+    await waitFor(() => parallelWaiterStream.isUpToDate)
 
     await sleep(50)
 

diff --git a/packages/typescript-client/test/integration.test.ts b/packages/typescript-client/test/integration.test.ts
@@ -307,10 +307,7 @@ describe(`HTTP Sync`, () => {
         ]
       )
 
-      await vi.waitFor(async () => {
-        await sleep(100)
-        return client.isUpToDate
-      })
+      await vi.waitFor(() => client.isUpToDate)
       const updatedData = await client.rows
 
       expect(updatedData).toMatchObject([
@@ -528,10 +525,7 @@ describe(`HTTP Sync`, () => {
     )
 
     // And wait until it's definitely seen
-    await vi.waitFor(async () => {
-      await sleep(50)
-      return issueStream.isUpToDate
-    })
+    await vi.waitFor(() => issueStream.isUpToDate)
 
     let catchupOpsCount = 0
     const newAborter = new AbortController()
@@ -589,6 +583,7 @@ describe(`HTTP Sync`, () => {
     // Wait for server to get all the messages.
     await sleep(40)
 
+    // Request to the beginning of the stream should return an unchanged e-tag for the first chunk
     const res2 = await fetch(
       `${BASE_URL}/v1/shape?table=${issuesTableUrl}&offset=-1`
     )
@@ -799,10 +794,7 @@ describe(`HTTP Sync`, () => {
     )
 
     // And wait until it's definitely seen
-    await vi.waitFor(async () => {
-      await sleep(50)
-      return issueStream.isUpToDate
-    })
+    await vi.waitFor(() => issueStream.isUpToDate)
 
     const responseSizes: number[] = []