Test limits

packages/backend-core/src/db/lucene.ts

@@ -495,13 +495,15 @@ export class QueryBuilder<T> {
 
     this.excludeDocs()
     let skipRemaining = skip
+    let iterationFetched = 0
     do {
       const toSkip = Math.min(QueryBuilder.maxLimit, skipRemaining)
       this.setLimit(toSkip)
       const { bookmark, rows } = await this.#execute()
       this.setBookmark(bookmark)
+      iterationFetched = rows.length
       skipRemaining -= rows.length
-    } while (skipRemaining > 0)
+    } while (skipRemaining > 0 && iterationFetched > 0)
 
     this.#includeDocs = prevIncludeDocs
     this.#limit = prevLimit

packages/backend-core/src/db/tests/lucene.spec.ts

@@ -210,6 +210,17 @@ describe("lucene", () => {
           docs.slice(skip, skip + resp.rows.length).map(expect.objectContaining)
         )
       })
+
+      it("should not return results if skipping all docs", async () => {
+        const builder = new QueryBuilder(skipDbName, INDEX_NAME)
+        // Skipping 2 max limits plus a little bit more
+        const skip = docs.length + 1
+        builder.setSkip(skip)
+
+        const resp = await builder.run()
+
+        expect(resp.rows.length).toBe(0)
+      })
     })
   })