test: add tests for getAllPullRequestsWithRetry

Introduce mock context and data to validate pull request retrieval.

src/utils/get-pull-requests-fallback.ts

@@ -19,7 +19,7 @@ export async function getAllPullRequestsFallback(
   const organization = context.payload.repository.owner.login;
 
   try {
-    const repositories = await octokit.paginate(octokit.repos.listForOrg, {
+    const repositories = await octokit.paginate(octokit.rest.repos.listForOrg, {
       org: organization,
       per_page: 100,
       type: "all",
@@ -29,7 +29,7 @@ export async function getAllPullRequestsFallback(
 
     const tasks = repositories.map(async (repo) => {
       try {
-        const prs = await octokit.paginate(octokit.pulls.list, {
+        const prs = await octokit.paginate(octokit.rest.pulls.list, {
           owner: organization,
           repo: repo.name,
           state,

tests/fallbacks.test.ts

@@ -0,0 +1,49 @@
+import { RestEndpointMethodTypes } from "@octokit/rest";
+import { Logs } from "@ubiquity-os/ubiquity-os-logger";
+import { Context } from "../src/types/context";
+import { getAllPullRequestsWithRetry } from "../src/utils/issue";
+
+const username = "private-user";
+
+const mockPullRequestData = [
+  { id: 1, number: 123, state: "open", user: { login: username } },
+  { id: 2, number: 124, state: "open", user: { login: "public-user" } },
+] as unknown as RestEndpointMethodTypes["pulls"]["list"]["response"]["data"];
+
+const mockOctokit = {
+  paginate: jest.fn().mockResolvedValue(mockPullRequestData),
+  rest: {
+    pulls: {
+      list: jest.fn().mockResolvedValue(mockPullRequestData),
+    },
+    repos: {
+      listForOrg: jest.fn().mockResolvedValue(mockPullRequestData),
+    },
+  },
+};
+
+// Mock context
+const context: Context = {
+  eventName: "pull_request",
+  payload: {
+    repository: {
+      name: "test-repo",
+      owner: {
+        login: "test-owner",
+      },
+    },
+  },
+  octokit: mockOctokit as unknown as Context["octokit"],
+  logger: new Logs("debug"),
+  adapters: {},
+} as unknown as Context;
+
+describe("getAllPullRequestsWithRetry", () => {
+  it("should return pull requests even if user information is private", async () => {
+    const pullRequests = await getAllPullRequestsWithRetry(context, "all", username);
+    expect(pullRequests).toHaveLength(2);
+    expect(pullRequests[0].user?.login).toBe(username);
+    expect(pullRequests[1].user?.login).toBe(username);
+    console.log(pullRequests);
+  });
+});