Merge pull request #1 from zentered/fix/abort-poll

fix(polling): abort when build fails

.github/workflows/test.yml

@@ -20,8 +20,10 @@ jobs:
         uses: github/[email protected]
         env:
           LINTER_RULES_PATH: /
+          FILTER_REGEX_EXCLUDE: (.*dist\/.*)|(.husky\/.*)
           JAVASCRIPT_ES_CONFIG_FILE: .eslintrc.json
           VALIDATE_ALL_CODEBASE: true
           VALIDATE_JAVASCRIPT_STANDARD: false
           DEFAULT_BRANCH: main
+          IGNORE_GENERATED_FILES: true
       - run: npm test

cloudflare-statuscheck.mjs

@@ -28,6 +28,12 @@ export default async function waitForDeployment(
   if (!build) {
     core.error(data)
     core.setFailed('no build with this ID found.')
+    throw new Error('No build id. Abort.')
+  }
+
+  if (build.latest_stage.status === 'failure') {
+    core.setFailed(`${build.latest_stage.name}: ${build.latest_stage.status}`)
+    throw new Error('Build failed. Abort.')
   }
 
   return (

test/cloudflare-statuscheck.test.js

@@ -4,7 +4,8 @@ import axios from 'axios'
 
 jest.mock('@actions/core', () => {
   return {
-    info: jest.fn()
+    info: jest.fn(),
+    setFailed: jest.fn()
   }
 })
 jest.mock('axios')
@@ -58,3 +59,30 @@ test('waitForDeployment() should wait until a deployment is successful', async (
   expect(firstCheck).toEqual(false)
   expect(secondCheck).toEqual(true)
 })
+
+test('waitForDeployment() should abort when a build has failed', async () => {
+  axios.get.mockResolvedValueOnce({
+    data: {
+      result: [
+        {
+          id: '123abc',
+          environment: 'preview',
+          latest_stage: {
+            name: 'build',
+            status: 'failure'
+          }
+        }
+      ]
+    }
+  })
+
+  await expect(
+    checkDeploymentStatus(
+      '123xyz',
+      'zentered',
+      '[email protected]',
+      'cf-project',
+      '123abc'
+    )
+  ).rejects.toThrow('Build failed. Abort.')
+})