@todesktop/cli 1.7.0-1 → 1.7.0-3

package/dist/cli.js

@@ -4178,7 +4178,7 @@ function BuildPicker({
       state: state.state,
       updateState
     });
-  }, [buildId, configPath, state.state]);
+  }, []);
   onInput((input, key) => {
     if (key.escape && ["show-builds", "loading"].includes(state.state)) {
       exit();
@@ -4411,7 +4411,8 @@ function OsProgress({
   const colors = {
     progress: "white",
     error: "red",
-    done: "green"
+    done: "green",
+    skipped: "yellow"
   };
   const text = progress.message + (progress.state === "progress" ? "..." : "");
   return /* @__PURE__ */ (0, import_jsx_runtime40.jsx)(
@@ -4436,6 +4437,8 @@ function SmokeTestView({
     case "build-error": {
       return /* @__PURE__ */ (0, import_jsx_runtime41.jsx)(BuildError, { build: state.build, message: (_a = state.error) == null ? void 0 : _a.message });
     }
+    case "canceled":
+      return /* @__PURE__ */ (0, import_jsx_runtime41.jsx)(import_ink36.Text, { children: "Smoke test canceled." });
     case "complete":
       return /* @__PURE__ */ (0, import_jsx_runtime41.jsx)(import_ink36.Text, { bold: true, color: "greenBright", children: "Smoke test passed." });
     case "error":
@@ -4493,6 +4496,52 @@ function isBuildTestable(build) {
     return false;
   }
 }
+function isTestRunning(build) {
+  const { isFinished } = makeProgress(build);
+  return !isFinished;
+}
+function makeProgress(build) {
+  var _a, _b, _c, _d, _e, _f;
+  const def = {
+    progress: 0,
+    state: "progress",
+    shouldSkip: false
+  };
+  const linux = {
+    ...((_a = build.smokeTest) == null ? void 0 : _a.linux) || def,
+    shouldSkip: ((_b = build.linux) == null ? void 0 : _b.shouldSkip) || false
+  };
+  const mac = {
+    ...((_c = build.smokeTest) == null ? void 0 : _c.mac) || def,
+    shouldSkip: ((_d = build.mac) == null ? void 0 : _d.shouldSkip) || false
+  };
+  const windows = {
+    ...((_e = build.smokeTest) == null ? void 0 : _e.windows) || def,
+    shouldSkip: ((_f = build.windows) == null ? void 0 : _f.shouldSkip) || false
+  };
+  const platforms = [linux, mac, windows].filter(
+    (p) => !p.shouldSkip && p.state !== "skipped"
+  );
+  const isFinished = build.smokeTest.isCanceled || platforms.every((p) => p.state !== "progress");
+  let totalState = "progress";
+  if (build.smokeTest.isCanceled) {
+    totalState = "canceled";
+  } else if (platforms.every((p) => p.state === "done")) {
+    totalState = "done";
+  } else if (isFinished && platforms.some((p) => p.state === "error")) {
+    totalState = "error";
+  }
+  return {
+    isFinished,
+    linux,
+    mac,
+    windows,
+    total: {
+      error: platforms.filter((p) => p.state === "error").map((p) => p.message).join("\n\n"),
+      state: totalState
+    }
+  };
+}
 function validateBuild(build) {
   if (build.status !== "succeeded") {
     throw new SmokeError(
@@ -4604,46 +4653,6 @@ async function waitUntilFinished({
     );
   });
 }
-function makeProgress(build) {
-  var _a, _b, _c, _d, _e, _f;
-  const def = {
-    progress: 0,
-    state: "progress",
-    shouldSkip: false
-  };
-  const linux = {
-    ...((_a = build.smokeTest) == null ? void 0 : _a.linux) || def,
-    shouldSkip: ((_b = build.linux) == null ? void 0 : _b.shouldSkip) || false
-  };
-  const mac = {
-    ...((_c = build.smokeTest) == null ? void 0 : _c.mac) || def,
-    shouldSkip: ((_d = build.mac) == null ? void 0 : _d.shouldSkip) || false
-  };
-  const windows = {
-    ...((_e = build.smokeTest) == null ? void 0 : _e.windows) || def,
-    shouldSkip: ((_f = build.windows) == null ? void 0 : _f.shouldSkip) || false
-  };
-  const platforms = [linux, mac, windows].filter(
-    (p) => !p.shouldSkip
-  );
-  const isFinished = platforms.every((p) => p.state !== "progress");
-  let totalState = "progress";
-  if (platforms.every((p) => p.state === "done")) {
-    totalState = "done";
-  } else if (isFinished && platforms.some((p) => p.state === "error")) {
-    totalState = "error";
-  }
-  return {
-    isFinished,
-    linux,
-    mac,
-    windows,
-    total: {
-      error: platforms.filter((p) => p.state === "error").map((p) => p.message).join("\n\n"),
-      state: totalState
-    }
-  };
-}
 
 // src/commands/smoke-test/SmokeTestCommand.tsx
 var import_jsx_runtime42 = require("react/jsx-runtime");
@@ -4683,7 +4692,7 @@ function SmokeTestContainer({
   const abortSignal = abortController.signal;
   (0, import_react22.useEffect)(() => {
     smokeTestWorkflow({ abortSignal, buildId, configPath, updateState });
-  }, [buildId, configPath]);
+  }, []);
   function updateState(changes) {
     setState((previousState) => ({ ...previousState, ...changes }));
     if (["build-error", "complete", "progress-error"].includes(changes.state)) {
@@ -4718,7 +4727,7 @@ async function smokeTestWorkflow({
     const build = await fetchBuild({ appId, buildId, userId });
     updateState({ appId, build, userId });
     validateBuild(build);
-    if (!abortSignal.aborted) {
+    if (!abortSignal.aborted && !isTestRunning(build)) {
       await queueSmokeTest({ appId, buildId, nodeVersion, userId });
     }
     updateState({ state: "progress" });
@@ -4728,9 +4737,13 @@ async function smokeTestWorkflow({
       userId,
       onProgress: (progress) => updateState({ progress })
     });
-    updateState({
-      state: total.state == "error" ? "progress-error" : "complete"
-    });
+    const stateMap = {
+      done: "complete",
+      progress: "complete",
+      error: "progress-error",
+      canceled: "canceled"
+    };
+    updateState({ state: stateMap[total.state] });
   } catch (error) {
     updateState({ error, state: error.type || "error" });
   }
@@ -4740,10 +4753,7 @@ function buildCanBeCanceled(build) {
   if ((_a = build == null ? void 0 : build.smokeTest) == null ? void 0 : _a.isCanceled) {
     return false;
   }
-  if (!((_b = build == null ? void 0 : build.smokeTest) == null ? void 0 : _b.buildServerExecutionId)) {
-    return false;
-  }
-  return true;
+  return Boolean((_b = build == null ? void 0 : build.smokeTest) == null ? void 0 : _b.buildServerExecutionId);
 }
 function isCancelable(state) {
   const cancelableStates = ["loading", "progress"];
@@ -4810,7 +4820,7 @@ var package_default = {
     access: "public"
   },
   name: "@todesktop/cli",
-  version: "1.7.0-0",
+  version: "1.7.0-2",
   license: "MIT",
   author: "Dave Jeffery <dave@todesktop.com> (http://www.todesktop.com/)",
   homepage: "https://todesktop.com/cli",
@@ -4890,7 +4900,7 @@ var package_default = {
     "xdg-basedir": "^4.0.0"
   },
   devDependencies: {
-    "@todesktop/shared": "^7.178.0",
+    "@todesktop/shared": "^7.179.0",
     "@types/bunyan": "^1.8.6",
     "@types/react": "^18.0.26",
     "@typescript-eslint/eslint-plugin": "^5.46.1",