test: add tests for countUploadableQueries

Extract UPLOADABLE_STATES and countUploadableQueries as exported
functions so they can be tested. Tests verify:
- Only improvements_available, no_improvement_found, error are counted
- not_supported, timeout, waiting, optimizing are excluded
- countUploadableQueries agrees with buildQueries for every state

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: veksen

src/runner.test.ts

@@ -0,0 +1,64 @@
+import { test, expect, describe } from "vitest";
+import { countUploadableQueries, UPLOADABLE_STATES } from "./runner.ts";
+import { buildQueries } from "./reporters/site-api.ts";
+import type { OptimizedQuery } from "./sql/recent-query.ts";
+
+function fakeQuery(state: string): OptimizedQuery {
+  return {
+    optimization: { state },
+  } as OptimizedQuery;
+}
+
+describe("countUploadableQueries", () => {
+  test("counts improvements_available, no_improvement_found, and error", () => {
+    const results = [
+      fakeQuery("improvements_available"),
+      fakeQuery("no_improvement_found"),
+      fakeQuery("error"),
+    ];
+    expect(countUploadableQueries(results)).toBe(3);
+  });
+
+  test("excludes not_supported, timeout, waiting, optimizing", () => {
+    const results = [
+      fakeQuery("not_supported"),
+      fakeQuery("timeout"),
+      fakeQuery("waiting"),
+      fakeQuery("optimizing"),
+    ];
+    expect(countUploadableQueries(results)).toBe(0);
+  });
+
+  test("counts only uploadable in a mixed set", () => {
+    const results = [
+      fakeQuery("improvements_available"),
+      fakeQuery("not_supported"),
+      fakeQuery("no_improvement_found"),
+      fakeQuery("timeout"),
+      fakeQuery("error"),
+      fakeQuery("waiting"),
+    ];
+    expect(countUploadableQueries(results)).toBe(3);
+  });
+});
+
+describe("UPLOADABLE_STATES matches buildQueries filter", () => {
+  test("buildQueries keeps exactly the same states as UPLOADABLE_STATES", () => {
+    const allStates = [
+      "improvements_available",
+      "no_improvement_found",
+      "error",
+      "not_supported",
+      "timeout",
+      "waiting",
+      "optimizing",
+    ];
+
+    for (const state of allStates) {
+      const results = [fakeQuery(state)];
+      const uploaded = buildQueries(results).length;
+      const counted = countUploadableQueries(results);
+      expect(counted, `state "${state}"`).toBe(uploaded);
+    }
+  });
+});

src/runner.ts

@@ -217,8 +217,7 @@ export class Runner {
       }
     }
 
-    const uploadableStates = new Set(["improvements_available", "no_improvement_found", "error"]);
-    const total = allResults.filter((q) => uploadableStates.has(q.optimization.state)).length;
+    const total = countUploadableQueries(allResults);
 
     const statistics = deriveIndexStatistics(filteredRecommendations);
     const timeElapsed = Date.now() - startDate.getTime();
@@ -247,4 +246,10 @@ export class Runner {
   }
 }
 
+export const UPLOADABLE_STATES = new Set(["improvements_available", "no_improvement_found", "error"]);
+
+export function countUploadableQueries(results: OptimizedQuery[]): number {
+  return results.filter((q) => UPLOADABLE_STATES.has(q.optimization.state)).length;
+}
+
 export type QueryProcessResult = OptimizedQuery;