fix: parallel check requests and error handling (#150)

* fix: parallel check requests and error handling * build: release v2.1.2
pozil · Jan 3, 2025 · c015a6a · c015a6a
1 parent f09bc3b
commit c015a6a
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/node_modules/.package-lock.json b/node_modules/.package-lock.json
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "auto-assign-issue",
-    "version": "2.1.1",
+    "version": "2.1.2",
     "private": true,
     "description": "GitHub action that auto-assigns issues to users",
     "main": "src/index.js",

diff --git a/src/__tests__/action.test.js b/src/__tests__/action.test.js
@@ -226,7 +226,7 @@ describe('action', () => {
         });
 
         it('fails if user cannot be assigned and failsIfUsersCannotBeAssigned flag is true', async () => {
-            const requestMock = jest.fn(() => Promise.resolve({ status: 404 }));
+            const requestMock = jest.fn(() => Promise.reject({ status: 404 }));
             const octokitMock = getOctokitMock({ requestMock });
 
             await expect(

diff --git a/src/utils.js b/src/utils.js
@@ -184,11 +184,11 @@ const checkIfUsersCanBeAssigned = async (
         );
         requests.push(request);
     }
-    const responses = await Promise.all(requests);
+    const responses = await Promise.allSettled(requests);
     const result = { isSuccess: true, assigneeErrors: [] };
     for (let i = 0; i < assignees.length; i++) {
         const { status } = responses[i];
-        if (status != 204) {
+        if (status === 'rejected') {
             result.isSuccess = false;
             result.assigneeErrors.push(assignees[i]);
         }