fix: handle NaN in build time estimate (#5679)

presleyp · web-flow · commit 1df72ee093d1 · 2023-01-11T15:56:21.000-05:00
diff --git a/site/src/components/WorkspaceBuildProgress/WorkspaceBuildProgress.stories.tsx b/site/src/components/WorkspaceBuildProgress/WorkspaceBuildProgress.stories.tsx
@@ -67,3 +67,9 @@ StartingHighVariaton.args = {
   ...Starting.args,
   transitionStats: { P50: 10000, P95: 20000 },
 }
+
+export const StartingZeroEstimate = Template.bind({})
+StartingZeroEstimate.args = {
+  ...Starting.args,
+  transitionStats: { P50: 0, P95: 0 },
+}
diff --git a/site/src/components/WorkspaceBuildProgress/WorkspaceBuildProgress.tsx b/site/src/components/WorkspaceBuildProgress/WorkspaceBuildProgress.tsx
@@ -34,11 +34,13 @@ const estimateFinish = (
   p95: number,
 ): [number | undefined, string] => {
   const sinceStart = dayjs().diff(startedAt)
-  const secondsLeft = (est: number) =>
-    Math.max(
+  const secondsLeft = (est: number) => {
+    const max = Math.max(
       Math.ceil(dayjs.duration((1 - sinceStart / est) * est).asSeconds()),
       0,
     )
+    return isNaN(max) ? 0 : max
+  }
 
   const lowGuess = secondsLeft(p50)
   const highGuess = secondsLeft(p95)
