@jsenv/github-check-run 0.0.11 → 0.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/github-check-run",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "description": "Interact with GitHub check rest API",
   "license": "MIT",
   "author": {

package/src/start_github_check_run.js

@@ -74,6 +74,8 @@ export const startGithubCheckRun = async ({
     return { progress: () => {}, fail: () => {}, pass: () => {} };
   }
 
+  let checkConclusion;
+
   const update = async ({
     status,
     conclusion,
@@ -87,6 +89,9 @@ export const startGithubCheckRun = async ({
     if (typeof summary !== "string") {
       throw new TypeError(`summary must be a string, got ${summary}`);
     }
+    if (conclusion) {
+      checkConclusion = conclusion;
+    }
 
     let annotationsSent = 0;
     const annotationsBatch = annotations.slice(annotationsSent, 50);
@@ -152,6 +157,12 @@ ${JSON.stringify(body, null, "  ")}`);
 
   return {
     progress: async ({ title, summary, annotations = [] }) => {
+      if (checkConclusion === "failure") {
+        throw new Error(`cannot progress() after fail()`);
+      }
+      if (checkConclusion === "success") {
+        throw new Error(`cannot progress() after pass()`);
+      }
       const nowMs = Date.now();
       const isFirstCall = !lastProgressCall;
       lastProgressCall = nowMs;
@@ -196,9 +207,17 @@ ${JSON.stringify(body, null, "  ")}`);
       });
     },
     fail: ({ title, summary, annotations } = {}) => {
+      if (checkConclusion === "failure") {
+        throw new Error(`already failed`);
+      }
+      if (checkConclusion === "success") {
+        throw new Error(`cannot fail() after pass()`);
+      }
+
       if (pendingAbortController) {
         pendingAbortController.abort();
       }
+      // TODO: wait to any update before PATCH
       return update({
         status: "completed",
         conclusion: "failure",
@@ -208,9 +227,17 @@ ${JSON.stringify(body, null, "  ")}`);
       });
     },
     pass: ({ title, summary, annotations } = {}) => {
+      if (checkConclusion === "success") {
+        throw new Error(`already passed`);
+      }
+      if (checkConclusion === "failure") {
+        throw new Error(`cannot pass() after fail()`);
+      }
+
       if (pendingAbortController) {
         pendingAbortController.abort();
       }
+      // TODO: wait to any pending update before PATCH
       return update({
         status: "completed",
         conclusion: "success",