handling for fail (#55)

Signed-off-by: Piotr Karpala <[email protected]>

Author: karpikpl

__tests__/main.test.js

@@ -201,7 +201,7 @@ describe('action', () => {
     expect(createCommentMock).toHaveBeenCalled()
   })
 
-  it('sets a failed status when github fails', async () => {
+  it('doesnt set failed status when github fails', async () => {
     // Set the action's inputs as return values from core.getInput()
     mockInput()
     createChecksMock.mockRejectedValue(new Error('Failed to create check'))
@@ -211,7 +211,7 @@ describe('action', () => {
     expect(runMock).toHaveReturned()
 
     // Verify that all of the core library functions were called correctly
-    expect(setFailedMock).toHaveBeenNthCalledWith(1, 'Failed to create comment')
+    expect(setFailedMock).toHaveBeenCalledTimes(0)
   })
 
   it('fails if no pylint file provided', async () => {

badges/coverage.svg

@@ -98,13 +98,17 @@ async function run() {
         core.debug(
           `Trying to add a comment instead for repo:${repo_name} owner:${repo_owner} issue:${issueNumber}`
         )
-        const commentResp = await octokit.rest.issues.createComment({
-          owner: repo_owner,
-          repo: repo_name,
-          issue_number: issueNumber,
-          body: comment
-        })
-        core.debug(`response from comment create: ${commentResp.status}`)
+        try {
+          const commentResp = await octokit.rest.issues.createComment({
+            owner: repo_owner,
+            repo: repo_name,
+            issue_number: issueNumber,
+            body: comment
+          })
+          core.debug(`response from comment create: ${commentResp.status}`)
+        } catch (commentError) {
+          core.error(`Error creating comment: ${commentError}`)
+        }
       }
     }