fix: Fix search API to return paginator

neet · neet · commit 2691c274b789 · 2023-12-26T19:49:19.000+09:00
diff --git a/src/mastodon/rest/v1/search-repository.ts b/src/mastodon/rest/v1/search-repository.ts
@@ -17,13 +17,18 @@ export interface SearchParams extends DefaultPaginationParams {
 }
 
 export interface SearchRepository {
+  /**
+   * @deprecated Use `list` instead
+   */
+  fetch(params: SearchParams, meta?: HttpMetaParams): Search;
+
   /**
    * Search, but hashtags is an array of strings instead of an array of Tag.
    * @param params Parameters
    * @return Results
    * @see https://docs.joinmastodon.org/methods/search/
    */
-  fetch(
+  list(
     params: SearchParams,
     meta?: HttpMetaParams,
   ): Paginator<Search, SearchParams>;
diff --git a/src/mastodon/rest/v2/search-repository.ts b/src/mastodon/rest/v2/search-repository.ts
@@ -21,13 +21,18 @@ export interface SearchParams extends DefaultPaginationParams {
 }
 
 export interface SearchRepository {
+  /**
+   * @deprecated Use `list` instead
+   */
+  fetch(params: SearchParams, meta?: HttpMetaParams): Search;
+
   /**
    * Perform a search
    * @param params Parameters
    * @return Results
    * @see https://docs.joinmastodon.org/methods/search/
    */
-  fetch(
+  list(
     params: SearchParams,
     meta?: HttpMetaParams,
   ): Paginator<Search, SearchParams>;
diff --git a/tests/rest/v2/search.spec.ts b/tests/rest/v2/search.spec.ts
@@ -1,6 +1,6 @@
 it("searches", async () => {
   await using session = await sessions.acquire();
-  const results = await session.rest.v2.search.fetch({
+  const results = await session.rest.v2.search.list({
     q: "mastodon",
   });
 
@@ -10,3 +10,19 @@ it("searches", async () => {
     hashtags: expect.any(Array),
   });
 });
+
+it("can be iterated over", async () => {
+  await using session = await sessions.acquire();
+  const results = session.rest.v2.search.list({
+    q: "mastodon",
+  });
+
+  const p1 = await results.next();
+
+  expect(typeof p1.done).toBe("boolean");
+  expect(p1.value).toMatchObject({
+    accounts: expect.any(Array),
+    statuses: expect.any(Array),
+    hashtags: expect.any(Array),
+  });
+});
