test(web-evals): cover formatters; memoize statusLabels

hannesrudolph · hannesrudolph · commit cffcf555fe4b · 2025-12-14T13:13:20.000-07:00
diff --git a/apps/web-evals/src/app/runs/[id]/run.tsx b/apps/web-evals/src/app/runs/[id]/run.tsx
@@ -554,20 +554,23 @@ export function Run({ run }: { run: Run }) {
 		return groups
 	}, [tasks, groupByStatus, getTaskStatusCategory])
 
-	const statusLabels: Record<TaskStatusCategory, { label: string; className: string; count: number }> = {
-		failed: { label: "Failed", className: "text-red-500", count: groupedTasks?.failed.length ?? 0 },
-		in_progress: {
-			label: "In Progress",
-			className: "text-yellow-500",
-			count: groupedTasks?.in_progress.length ?? 0,
-		},
-		passed: { label: "Passed", className: "text-green-500", count: groupedTasks?.passed.length ?? 0 },
-		not_started: {
-			label: "Not Started",
-			className: "text-muted-foreground",
-			count: groupedTasks?.not_started.length ?? 0,
-		},
-	}
+	const statusLabels = useMemo(
+		(): Record<TaskStatusCategory, { label: string; className: string; count: number }> => ({
+			failed: { label: "Failed", className: "text-red-500", count: groupedTasks?.failed.length ?? 0 },
+			in_progress: {
+				label: "In Progress",
+				className: "text-yellow-500",
+				count: groupedTasks?.in_progress.length ?? 0,
+			},
+			passed: { label: "Passed", className: "text-green-500", count: groupedTasks?.passed.length ?? 0 },
+			not_started: {
+				label: "Not Started",
+				className: "text-muted-foreground",
+				count: groupedTasks?.not_started.length ?? 0,
+			},
+		}),
+		[groupedTasks],
+	)
 
 	const statusOrder: TaskStatusCategory[] = ["failed", "in_progress", "passed", "not_started"]
 
diff --git a/apps/web-evals/src/lib/__tests__/formatters.spec.ts b/apps/web-evals/src/lib/__tests__/formatters.spec.ts
@@ -0,0 +1,30 @@
+import { formatDuration, formatTokens } from "../formatters"
+
+describe("formatDuration()", () => {
+	it("formats as H:MM:SS", () => {
+		expect(formatDuration(0)).toBe("0:00:00")
+		expect(formatDuration(1_000)).toBe("0:00:01")
+		expect(formatDuration(61_000)).toBe("0:01:01")
+		expect(formatDuration(3_661_000)).toBe("1:01:01")
+	})
+})
+
+describe("formatTokens()", () => {
+	it("formats small numbers without suffix", () => {
+		expect(formatTokens(0)).toBe("0")
+		expect(formatTokens(999)).toBe("999")
+	})
+
+	it("formats thousands without decimals and clamps to 1.0M at boundary", () => {
+		expect(formatTokens(1_000)).toBe("1k")
+		expect(formatTokens(72_500)).toBe("73k")
+		expect(formatTokens(999_499)).toBe("999k")
+		expect(formatTokens(999_500)).toBe("1.0M")
+	})
+
+	it("formats millions with one decimal and clamps to 1.0B at boundary", () => {
+		expect(formatTokens(1_000_000)).toBe("1.0M")
+		expect(formatTokens(3_240_000)).toBe("3.2M")
+		expect(formatTokens(999_950_000)).toBe("1.0B")
+	})
+})
