Return error when step.conclusion has any cancelled

haya14busa · Feb 16, 2023 · 5a9adde · 5a9adde
1 parent c5cebfe
commit 5a9adde
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 5 deletions.
diff --git a/.github/workflows/test_post.yml b/.github/workflows/test_post.yml
@@ -19,7 +19,13 @@ jobs:
       - uses: ./
         with:
           matrix_name: ${{ matrix.node_group }}
-      - run: exit 0
+      - run:
+          sleep 10
+          exit 0
+      - name: Dump GitHub context
+        env:
+          GITHUB_CONTEXT: ${{ toJson(github) }}
+        run: echo "$GITHUB_CONTEXT"
 
   post-test-matrix-failure:
     runs-on: ubuntu-latest

diff --git a/action.yml b/action.yml
@@ -17,3 +17,4 @@ runs:
   using: 'node16'
   main: 'dist/index.js'
   post: 'dist/index.js'
+  post-if: 'always()'
diff --git a/dist/index.js b/dist/index.js
@@ -1197,7 +1197,7 @@ function cleanup() {
     });
 }
 function job2status(job, isCleanUp) {
-    var _a;
+    var _a, _b;
     if (!isCleanUp) {
         return 'pending';
     }
@@ -1208,7 +1208,11 @@ function job2status(job, isCleanUp) {
     if (failedStep) {
         return 'failure';
     }
-    return job.conclusion === 'success' ? 'success' : 'error';
+    const cancelledStep = (_b = job.steps) === null || _b === void 0 ? void 0 : _b.find(step => step.conclusion === 'cancelled');
+    if (cancelledStep) {
+        return 'error';
+    }
+    return 'success';
 }
 function postStatus(isCleanUp) {
     var _a;

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -40,7 +40,13 @@ function job2status(
   if (failedStep) {
     return 'failure'
   }
-  return job.conclusion === 'success' ? 'success' : 'error'
+
+  const cancelledStep = job.steps?.find(step => step.conclusion === 'cancelled')
+  if (cancelledStep) {
+    return 'error'
+  }
+
+  return 'success'
 }
 
 async function postStatus(isCleanUp: boolean): Promise<void> {