Repositories can be a list

vrk-kpa · Apr 11, 2024 · bdb2d16 · bdb2d16
1 parent e69df6c
commit bdb2d16
Show file tree

Hide file tree

Showing 4 changed files with 84 additions and 43 deletions.
diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts
@@ -129,7 +129,7 @@ describe('action', () => {
     getInputMock.mockImplementation(name => {
       switch (name) {
         case 'repository':
-          return 'vrk-kpa/fetch-open-prs-action'
+          return 'vrk-kpa/fetch-open-prs'
         case 'ignored_users':
           return '["first_ignore", "second_ignore"]'
         default:
@@ -140,10 +140,9 @@ describe('action', () => {
     await main.run()
     expect(runMock).toHaveReturned()
 
-    expect(debugMock).toHaveBeenNthCalledWith(1, 'owner: vrk-kpa')
-    expect(debugMock).toHaveBeenNthCalledWith(2, 'repo: fetch-open-prs-action')
-
-    expect(debugMock).toHaveBeenNthCalledWith(3, 'Ignored users length: 2')
+    expect(debugMock).toHaveBeenNthCalledWith(1, 'Ignored users length: 2')
+    expect(debugMock).toHaveBeenNthCalledWith(2, 'owner: vrk-kpa')
+    expect(debugMock).toHaveBeenNthCalledWith(3, 'repo: fetch-open-prs')
 
     expect(setOutputMock).toHaveBeenNthCalledWith(
       1,
@@ -188,4 +187,27 @@ describe('action', () => {
       expect.stringContaining('* [some title](http://example.com) by some_user')
     )
   })
+
+  it('Repository could be a list', async () => {
+    getInputMock.mockImplementation(name => {
+      switch (name) {
+        case 'repository':
+          return '["vrk-kpa/fetch-open-prs", "vrk-kpa/some-other-repo"]'
+        default:
+          return ''
+      }
+    })
+
+    await main.run()
+    expect(runMock).toHaveReturned()
+
+    expect(debugMock).toHaveBeenNthCalledWith(1, 'owner: vrk-kpa')
+    expect(debugMock).toHaveBeenNthCalledWith(2, 'repo: fetch-open-prs')
+    expect(debugMock).toHaveBeenNthCalledWith(3, 'owner: vrk-kpa')
+    expect(debugMock).toHaveBeenNthCalledWith(4, 'repo: some-other-repo')
+
+    expect(setOutputMock.mock.calls[0][1]).toHaveLength(6)
+
+    expect(errorMock).not.toHaveBeenCalled()
+  })
 })
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -10,15 +10,13 @@ export async function run(): Promise<void> {
     const token = core.getInput('token')
     const octokit = github.getOctokit(token)
     const repository = core.getInput('repository')
-    const [owner, repo] = repository.split('/')
 
-    core.debug(`owner: ${owner}`)
-    core.debug(`repo: ${repo}`)
-
-    const prList = await octokit.rest.pulls.list({
-      owner,
-      repo
-    })
+    let repositories = []
+    try {
+      repositories = JSON.parse(repository)
+    } catch (err) {
+      repositories = [repository]
+    }
 
     const ignored_users = core.getInput('ignored_users')
 
@@ -35,18 +33,30 @@ export async function run(): Promise<void> {
       created_at: string
     }[] = []
 
-    for (const pr of prList.data) {
-      const parsedPr = {
-        url: pr['html_url'],
-        title: pr['title'],
-        user: '',
-        created_at: pr['created_at']
-      }
+    for (let r of repositories) {
+      const [owner, repo] = r.split('/')
+
+      core.debug(`owner: ${owner}`)
+      core.debug(`repo: ${repo}`)
+
+      const prList = await octokit.rest.pulls.list({
+        owner,
+        repo
+      })
+
+      for (const pr of prList.data) {
+        const parsedPr = {
+          url: pr['html_url'],
+          title: pr['title'],
+          user: '',
+          created_at: pr['created_at']
+        }
 
-      if (pr['user'] && !ignored_users_list.includes(pr['user']['login'])) {
-        parsedPr['user'] = pr['user']['login']
-        if (!pr['draft']) {
-          parsedPrList.push(parsedPr)
+        if (pr['user'] && !ignored_users_list.includes(pr['user']['login'])) {
+          parsedPr['user'] = pr['user']['login']
+          if (!pr['draft']) {
+            parsedPrList.push(parsedPr)
+          }
         }
       }
     }