Files
miti99bot-js/tests/scripts/backfill-mongo-to-kv.test.js
tiennm99 3f03521e84 feat(scripts): phase 07 — reverse-backfill scripts + delete guard
Pre-execution prerequisites for the Phase 07 cutover. Stage 2 of the
cutover keeps DUAL_WRITE=0 for ~6 days; if anything regresses during
that window the operator MUST be able to roll back to KV/D1 with the
last N days of Mongo-only writes recovered. Pre-building these scripts
(per code-reviewer #4) eliminates "draft a backfill under outage
pressure" — the anti-pattern of writing untested code at 4am.

Reverse-backfill
- scripts/backfill-mongo-to-kv.js: full-scan Mongo collection per module,
  PUT each doc back to CF KV via REST. expiresAt → expirationTtl (clamped
  to 60s minimum per CF KV); already-expired docs are skipped (won't
  resurrect dead state). 50 ops/sec throttle. --dry-run + --module flags.
- scripts/backfill-mongo-to-d1.js: full-scan trading_trades, build INSERT
  SQL preserving legacy_id where present (round-trips D1 autoincrement IDs
  preserved by phase-05 forward backfill). Sequential int generation for
  any docs without legacy_id. Pipes through wrangler d1 execute.
- scripts/lib/migration-helpers.js: cfKvPut helper added.

Delete guard (debugger #12)
- scripts/wrangler-delete-guard.sh: interactive CONFIRM wrapper around
  wrangler kv namespace delete + wrangler d1 delete. Exits 3 when stdin
  is not a tty so it cannot run in CI. Documented: never run in CI.

package.json: backfill:mongo:kv[:dry] + backfill:mongo:d1[:dry] scripts
wired.

Tests: 697 → 733 (+36).
- 7 cfKvPut tests (REST URL, querystring, body, expiration_ttl param).
- 10 reverse-KV TTL math tests (expired sentinel, future seconds, no-TTL,
  CF 60s minimum clamp).
- 9 reverse-D1 SQL construction tests (escaping, legacy_id preservation,
  sequential generation).

Lint clean. No Worker code touched. Stage 1 cutover, 7-day soak,
snapshots, and Stage 3 cleanup (delete CFKVStore + simplify factories +
edit package.json deploy chain) remain operator-driven and will be
committed separately after binding deletion.
2026-04-26 09:29:14 +07:00

119 lines
4.4 KiB
JavaScript

/**
* @file backfill-mongo-to-kv.test.js — unit tests for reverse-backfill TTL math.
*
* Tests the `computeTtl` helper exported from backfill-mongo-to-kv.js.
* No real Atlas connection, no real CF REST calls.
*/
import { afterEach, describe, expect, it, vi } from "vitest";
import { computeTtl } from "../../scripts/backfill-mongo-to-kv.js";
// ─── computeTtl ───────────────────────────────────────────────────────────────
describe("computeTtl", () => {
const NOW = 1_700_000_000_000; // fixed epoch ms for deterministic tests
it("returns null when expiresAt is null (persistent key)", () => {
expect(computeTtl(null, NOW)).toBeNull();
});
it("returns null when expiresAt is undefined (persistent key)", () => {
expect(computeTtl(undefined, NOW)).toBeNull();
});
it("returns 'expired' when expiresAt is in the past", () => {
const past = new Date(NOW - 1000); // 1 second ago
expect(computeTtl(past, NOW)).toBe("expired");
});
it("returns 'expired' when expiresAt equals now exactly", () => {
const now = new Date(NOW);
expect(computeTtl(now, NOW)).toBe("expired");
});
it("returns correct ttl (seconds) for a future expiresAt", () => {
const future = new Date(NOW + 120_000); // 120 seconds from now
const result = computeTtl(future, NOW);
expect(result).not.toBeNull();
expect(result).not.toBe("expired");
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(120);
});
it("floors partial seconds (no rounding up)", () => {
// 90.9 seconds remaining → floor → 90
const future = new Date(NOW + 90_900);
const result = computeTtl(future, NOW);
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(90);
});
it("clamps ttl to MIN_TTL_SECS (60) when remaining < 60s", () => {
// 30 seconds remaining — below CF KV minimum of 60
const future = new Date(NOW + 30_000);
const result = computeTtl(future, NOW);
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(60);
});
it("clamps ttl to MIN_TTL_SECS (60) when remaining is 1s", () => {
const future = new Date(NOW + 1_000);
const result = computeTtl(future, NOW);
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(60);
});
it("does NOT clamp when remaining is exactly 60s", () => {
const future = new Date(NOW + 60_000);
const result = computeTtl(future, NOW);
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(60);
});
it("does NOT clamp for large TTL values", () => {
// 7 days = 604800s
const future = new Date(NOW + 7 * 24 * 3600 * 1000);
const result = computeTtl(future, NOW);
expect(/** @type {{ttl: number}} */ (result).ttl).toBe(604800);
});
});
// ─── cfKvPut integration (fetch mock) ────────────────────────────────────────
// Verify that computeTtl=null produces a PUT with no expiration_ttl query param,
// and computeTtl={ttl:N} appends the param.
describe("cfKvPut TTL query param (fetch mock)", () => {
afterEach(() => vi.restoreAllMocks());
async function mockPut(expirationTtl) {
const fetchMock = vi.fn().mockResolvedValue({
ok: true,
status: 200,
text: () => Promise.resolve(""),
});
vi.stubGlobal("fetch", fetchMock);
const { cfKvPut } = await import("../../scripts/lib/migration-helpers.js");
const opts = expirationTtl != null ? { expirationTtl } : {};
await cfKvPut("acct", "ns", "tok", "mod:key", "value", opts);
return fetchMock.mock.calls[0][0]; // the URL string
}
it("omits expiration_ttl param when no TTL provided", async () => {
const url = await mockPut(undefined);
expect(url).not.toContain("expiration_ttl");
});
it("appends expiration_ttl param when TTL provided", async () => {
const url = await mockPut(300);
expect(url).toContain("expiration_ttl=300");
});
it("URL-encodes the key in the PUT request URL", async () => {
const fetchMock = vi.fn().mockResolvedValue({
ok: true,
text: () => Promise.resolve(""),
});
vi.stubGlobal("fetch", fetchMock);
const { cfKvPut } = await import("../../scripts/lib/migration-helpers.js");
await cfKvPut("acct", "ns", "tok", "mod:key with spaces", "v", {});
const url = fetchMock.mock.calls[0][0];
expect(url).toContain("mod%3Akey%20with%20spaces");
});
});