test: add integrations, cache, cron, and telemetry tests

- New integrations.test.ts: 13 tests covering CRUD, token generation,
  token lookup, regeneration, and multi-user isolation
- New cache.test.ts: 3 tests for purgeMetadataCache (shell title
  deletion, fully-fetched title preservation, empty case)
- New cron.test.ts: 5 tests for cron run lifecycle (start, complete,
  fail with Error and string)
- New telemetry.test.ts: 7 tests for isTelemetryEnabled toggle and
  performTelemetryReport (disabled skip, enabled send, 24h interval
  throttle, interval expiry, fetch failure resilience)

345 → 386 tests passing across 27 files

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-03-22 12:39:04 -04:00
co-authored by Claude Opus 4.6
parent 554271a888
commit 373a5d3caf
16 changed files with 402 additions and 61 deletions
+92
View File
@@ -0,0 +1,92 @@
import { beforeEach, describe, expect, test, vi } from "vitest";
import { clearAllTables, insertUser } from "@sofa/test/db";
import { setSetting } from "../src/settings";
import { isTelemetryEnabled, performTelemetryReport } from "../src/telemetry";
beforeEach(() => {
clearAllTables();
insertUser("user-1");
});
describe("isTelemetryEnabled", () => {
test("returns false when setting is not set", () => {
expect(isTelemetryEnabled()).toBe(false);
});
test("returns true when setting is 'true'", () => {
setSetting("telemetryEnabled", "true");
expect(isTelemetryEnabled()).toBe(true);
});
test("returns false when setting is 'false'", () => {
setSetting("telemetryEnabled", "false");
expect(isTelemetryEnabled()).toBe(false);
});
});
describe("performTelemetryReport", () => {
test("skips when telemetry is disabled", async () => {
const fetchSpy = vi.spyOn(globalThis, "fetch");
await performTelemetryReport();
expect(fetchSpy).not.toHaveBeenCalled();
});
test("sends report when telemetry is enabled", async () => {
setSetting("telemetryEnabled", "true");
vi.spyOn(globalThis, "fetch").mockImplementation(
(async () => new Response(null, { status: 200 })) as unknown as typeof fetch,
);
await performTelemetryReport();
expect(globalThis.fetch).toHaveBeenCalledOnce();
const call = vi.mocked(globalThis.fetch).mock.calls[0];
const url = String(call[0]);
expect(url).toContain("/v1/telemetry");
const init = call[1] as RequestInit;
expect(init.method).toBe("POST");
const body = JSON.parse(init.body as string);
expect(body).toHaveProperty("instanceId");
expect(body).toHaveProperty("version");
expect(body).toHaveProperty("users");
expect(body).toHaveProperty("titles");
expect(body).toHaveProperty("features");
});
test("respects 24-hour report interval", async () => {
setSetting("telemetryEnabled", "true");
setSetting("telemetryLastReportedAt", new Date().toISOString());
const fetchSpy = vi.spyOn(globalThis, "fetch");
await performTelemetryReport();
expect(fetchSpy).not.toHaveBeenCalled();
});
test("reports again after interval expires", async () => {
setSetting("telemetryEnabled", "true");
const oldDate = new Date(Date.now() - 25 * 60 * 60 * 1000).toISOString();
setSetting("telemetryLastReportedAt", oldDate);
vi.spyOn(globalThis, "fetch").mockImplementation(
(async () => new Response(null, { status: 200 })) as unknown as typeof fetch,
);
await performTelemetryReport();
expect(globalThis.fetch).toHaveBeenCalledOnce();
});
test("does not throw on fetch failure", async () => {
setSetting("telemetryEnabled", "true");
vi.spyOn(globalThis, "fetch").mockImplementation(
(async () => new Response(null, { status: 500 })) as unknown as typeof fetch,
);
await expect(performTelemetryReport()).resolves.toBeUndefined();
});
});