fix getAll bugs

src/factory.spec.ts

@@ -7,23 +7,43 @@ const factory = new Factory();
 
 describe("Factory", () => {
   beforeEach(() => {
-    vi.mocked(client.get)
-      .mockResolvedValue({
-        data: {
-          results: [{}],
-          next: null,
-        },
-      })
-      .mockResolvedValueOnce({
-        data: {
-          results: [{}],
-          next: "the_next_url",
-        },
-      });
+    vi.mocked(client.get).mockResolvedValue({
+      data: {
+        results: [{}],
+        next: null,
+      },
+    });
   });
 
   it("gets all", async () => {
+    vi.mocked(client.get).mockResolvedValueOnce({
+      data: {
+        results: [{}],
+        next: "the_next_url",
+      },
+    });
     await factory.getMany(1, Row);
     expect(client.get).toBeCalledTimes(2);
   });
+
+  it("treats page as one indexed", async () => {
+    await factory.getMany(1, Row);
+    expect(client.get).not.toBeCalledWith(
+      expect.anything(),
+      expect.objectContaining({ params: expect.objectContaining({ page: 0 }) }),
+    );
+  });
+
+  it("includes page with all requests", async () => {
+    await factory.getMany(1, Row);
+    expect(client.get).toBeCalledWith(
+      expect.anything(),
+      expect.objectContaining({ params: expect.objectContaining({ page: 1 }) }),
+    );
+  });
+
+  it("returns results", async () => {
+    const results = await factory.getMany(1, Row);
+    expect(results).toHaveLength(1);
+  });
 });

src/factory.ts

@@ -34,16 +34,16 @@ export class Factory {
     options: Record<string, unknown> & { page?: number } = {},
     accumulator: R[] = [],
   ): Promise<R[]> {
-    const { results, next } = await this.sdk.listRows<R>(tableId, options);
+    const { page = 1 } = options;
+    const { results, next } = await this.sdk.listRows<R>(tableId, {
+      ...options,
+      page,
+    });
+    accumulator.push(...results);
     if (!next) {
       return accumulator;
     }
-    accumulator.push(...results);
-    return this.getAll(
-      tableId,
-      { ...options, page: (options.page ?? 0) + 1 },
-      accumulator,
-    );
+    return this.getAll(tableId, { ...options, page: page + 1 }, accumulator);
   }
 
   private createRows<T extends Row, R extends RowType, F extends Factory>(