All checks were successful
CI / verification (pull_request) Successful in 31s
The CI runs the tests in node:22-alpine, where 9 of the 12 cases failed: no ssh on PATH, and busybox has neither `find -printf` nor `date -d "... days"`. The newest dated directory is now found by a loop over the glob, and the rotation cutoff by epoch arithmetic, which GNU date and busybox both read. The tests add a fake ssh, and a fake flock where the host has none. The rotation_failed case failed the pass through a read-only directory, which root ignores: it now uses a fake rm. Review suggestions: the rsync argv must put --include=*/ before --exclude=* (without it nothing crosses and the pass still says ok), and a case runs the real rsync through the fake ssh when rsync is present. A pass whose newest directory is not today's logs a warning. Refs maximus/la-compagnie-maximus#220 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
273 lines
11 KiB
JavaScript
273 lines
11 KiB
JavaScript
const fs = require("node:fs");
|
|
const os = require("node:os");
|
|
const path = require("node:path");
|
|
const { spawnSync } = require("node:child_process");
|
|
|
|
// Coverage for the PostgreSQL backup pull agent (la-compagnie-maximus#220).
|
|
//
|
|
// The script runs under a systemd user timer and talks to the VPS over
|
|
// rsync/ssh, then reports to POST /backups/pull. Both are replaced here by fake
|
|
// binaries on PATH, which record their argv so the tests can check what would
|
|
// have crossed the wire — in particular that the ingest token never appears in
|
|
// a command line nor in the script's own output (journald keeps it forever).
|
|
|
|
const SCRIPT = path.join(__dirname, "..", "agent", "pg-backup-pull", "run-pg-backup-pull.sh");
|
|
const TOKEN = "ingest-token-0123456789abcdef";
|
|
const TODAY = "2026-10-11";
|
|
|
|
let tmp;
|
|
let bin;
|
|
|
|
function writeExecutable(file, content) {
|
|
fs.writeFileSync(file, content, { mode: 0o755 });
|
|
}
|
|
|
|
beforeEach(() => {
|
|
tmp = fs.mkdtempSync(path.join(os.tmpdir(), "pg-backup-pull-"));
|
|
bin = path.join(tmp, "bin");
|
|
fs.mkdirSync(bin);
|
|
fs.writeFileSync(path.join(tmp, "key"), "fake key", { mode: 0o600 });
|
|
fs.writeFileSync(path.join(tmp, "known_hosts"), "[100.86.30.21]:2222 ssh-ed25519 AAAA");
|
|
|
|
// Fake rsync: records argv, then lands two .age files in today's directory.
|
|
writeExecutable(
|
|
path.join(bin, "rsync"),
|
|
`#!/bin/sh
|
|
printf '%s\\n' "$@" > "$FAKE_DIR/rsync.argv"
|
|
[ -n "\${FAKE_RSYNC_FAIL:-}" ] && exit 23
|
|
for last; do :; done
|
|
mkdir -p "$last/${TODAY}"
|
|
echo x > "$last/${TODAY}/maximus_api.dump.age"
|
|
echo x > "$last/${TODAY}/globals.sql.age"
|
|
`,
|
|
);
|
|
// Fake ssh: what rrsync does on the VPS, minus the restriction. Skips the
|
|
// options, drops the host, and runs the remote rsync command inside
|
|
// $FAKE_OUTBOX. Only reached by the real-rsync case; the fake rsync above
|
|
// never calls it, but the script requires ssh on PATH (none in node:22-alpine).
|
|
writeExecutable(
|
|
path.join(bin, "ssh"),
|
|
`#!/bin/sh
|
|
while [ $# -gt 0 ]; do
|
|
case "$1" in
|
|
-p|-i|-o|-l) shift 2 ;;
|
|
-*) shift ;;
|
|
*) shift; break ;;
|
|
esac
|
|
done
|
|
cd "$FAKE_OUTBOX" && exec "$@"
|
|
`,
|
|
);
|
|
// No flock in the CI's busybox: a pass-through stand-in, only where the host
|
|
// has none (no case here races two runs).
|
|
if (!["/usr/bin/flock", "/bin/flock"].some((f) => fs.existsSync(f))) {
|
|
writeExecutable(path.join(bin, "flock"), "#!/bin/sh\nexit 0\n");
|
|
}
|
|
// Fake curl: records argv, the header file it was handed and the body.
|
|
writeExecutable(
|
|
path.join(bin, "curl"),
|
|
`#!/bin/sh
|
|
printf '%s\\n' "$@" > "$FAKE_DIR/curl.argv"
|
|
prev=""
|
|
for a; do
|
|
case "$prev" in -H) case "$a" in @*) cp "\${a#@}" "$FAKE_DIR/curl.header" ;; esac ;; esac
|
|
prev="$a"
|
|
done
|
|
cat > "$FAKE_DIR/curl.body"
|
|
printf '%s' "\${FAKE_CURL_STATUS:-204}"
|
|
`,
|
|
);
|
|
});
|
|
|
|
afterEach(() => {
|
|
fs.rmSync(tmp, { recursive: true, force: true });
|
|
});
|
|
|
|
function run({ envFile, extraEnv = {} } = {}) {
|
|
const dest = path.join(tmp, "dest");
|
|
const result = spawnSync("sh", [SCRIPT], {
|
|
encoding: "utf8",
|
|
env: {
|
|
PATH: `${bin}:/usr/bin:/bin`,
|
|
HOME: tmp,
|
|
FAKE_DIR: tmp,
|
|
PG_BACKUP_PULL_ENV_FILE: envFile ?? path.join(tmp, "absent.env"),
|
|
PG_BACKUP_PULL_DEST: dest,
|
|
PG_BACKUP_PULL_STATE_DIR: path.join(tmp, "state"),
|
|
PG_BACKUP_PULL_SSH_KEY: path.join(tmp, "key"),
|
|
PG_BACKUP_PULL_KNOWN_HOSTS: path.join(tmp, "known_hosts"),
|
|
PG_BACKUP_PULL_TODAY: TODAY,
|
|
...extraEnv,
|
|
},
|
|
});
|
|
return { ...result, output: result.stdout + result.stderr, dest };
|
|
}
|
|
|
|
function writeEnv() {
|
|
const file = path.join(tmp, "agent.env");
|
|
fs.writeFileSync(
|
|
file,
|
|
`BACKUPS_API_URL=https://health.example.test\nBACKUPS_INGEST_TOKEN=${TOKEN}\n`,
|
|
{ mode: 0o600 },
|
|
);
|
|
return file;
|
|
}
|
|
|
|
const read = (name) => fs.readFileSync(path.join(tmp, name), "utf8");
|
|
|
|
describe("pull", () => {
|
|
test("pulls over port 2222 with a pinned host key, .age files only, never --delete", () => {
|
|
const { status } = run();
|
|
expect(status).toBe(0);
|
|
const argv = read("rsync.argv").split("\n");
|
|
expect(argv).toContain("--include=*.age");
|
|
expect(argv).toContain("--exclude=*");
|
|
// rsync keeps the first matching rule: without `--include=*/` first, the
|
|
// dated directories hit `--exclude=*` and nothing crosses, silently.
|
|
expect(argv.indexOf("--include=*/")).toBeGreaterThan(-1);
|
|
expect(argv.indexOf("--include=*/")).toBeLessThan(argv.indexOf("--exclude=*"));
|
|
expect(argv.indexOf("--include=*.age")).toBeLessThan(argv.indexOf("--exclude=*"));
|
|
expect(argv).not.toContain("--delete");
|
|
expect(argv).toContain("pgbackup@100.86.30.21:");
|
|
const ssh = argv.find((a) => a.startsWith("ssh "));
|
|
expect(ssh).toContain("-p 2222");
|
|
expect(ssh).toContain("StrictHostKeyChecking=yes");
|
|
expect(ssh).toContain("BatchMode=yes");
|
|
expect(ssh).toContain("IdentitiesOnly=yes");
|
|
});
|
|
|
|
test("without a token, still pulls and exits 0, saying the report was skipped", () => {
|
|
const { status, output, dest } = run();
|
|
expect(status).toBe(0);
|
|
expect(output).toContain("report skipped");
|
|
expect(fs.existsSync(path.join(dest, TODAY, "maximus_api.dump.age"))).toBe(true);
|
|
expect(fs.existsSync(path.join(tmp, "curl.argv"))).toBe(false);
|
|
});
|
|
|
|
test("keeps the destination private", () => {
|
|
const { dest } = run();
|
|
expect(fs.statSync(dest).mode & 0o777).toBe(0o700);
|
|
expect(fs.statSync(path.join(dest, TODAY, "globals.sql.age")).mode & 0o077).toBe(0);
|
|
});
|
|
|
|
test("a failed pull exits 2 and reports pull_failed", () => {
|
|
const { status } = run({ envFile: writeEnv(), extraEnv: { FAKE_RSYNC_FAIL: "1" } });
|
|
expect(status).toBe(2);
|
|
expect(JSON.parse(read("curl.body"))).toMatchObject({ ok: false, files: 0, error: "pull_failed" });
|
|
});
|
|
|
|
test("logs a warning when the newest directory is not today's", () => {
|
|
fs.writeFileSync(
|
|
path.join(bin, "rsync"),
|
|
`#!/bin/sh\nfor last; do :; done\nmkdir -p "$last/2026-10-09"\necho x > "$last/2026-10-09/a.age"\n`,
|
|
);
|
|
const { status, output } = run();
|
|
expect(status).toBe(0);
|
|
expect(output).toContain("latest=2026-10-09 files=1");
|
|
expect(output).toContain(`warning: no directory for ${TODAY}, latest is 2026-10-09`);
|
|
});
|
|
|
|
test("no warning when today's directory is there", () => {
|
|
const { output } = run();
|
|
expect(output).toContain(`latest=${TODAY} files=2`);
|
|
expect(output).not.toContain("warning");
|
|
});
|
|
|
|
// The real rsync with the script's own filter, against a local outbox through
|
|
// the fake ssh: only .age files land, inside their dated directory. Skipped
|
|
// where rsync is absent (the CI image); the argv order test above runs there.
|
|
test.skipIf(!["/usr/bin/rsync", "/bin/rsync"].some((f) => fs.existsSync(f)))(
|
|
"with the real rsync, only .age files cross, inside their dated directory",
|
|
() => {
|
|
fs.rmSync(path.join(bin, "rsync"));
|
|
const outbox = path.join(tmp, "outbox");
|
|
fs.mkdirSync(path.join(outbox, TODAY), { recursive: true });
|
|
fs.writeFileSync(path.join(outbox, TODAY, "maximus_api.dump.age"), "cipher");
|
|
fs.writeFileSync(path.join(outbox, TODAY, "maximus_api.dump"), "clear");
|
|
fs.writeFileSync(path.join(outbox, "stray.sql"), "clear");
|
|
const { status, output, dest } = run({ extraEnv: { FAKE_OUTBOX: outbox } });
|
|
expect(status).toBe(0);
|
|
expect(fs.readdirSync(dest)).toEqual([TODAY]);
|
|
expect(fs.readdirSync(path.join(dest, TODAY))).toEqual(["maximus_api.dump.age"]);
|
|
expect(output).toContain(`latest=${TODAY} files=1`);
|
|
},
|
|
);
|
|
|
|
test("missing ssh key is an install problem: exit 1, nothing pulled", () => {
|
|
fs.rmSync(path.join(tmp, "key"));
|
|
const { status } = run();
|
|
expect(status).toBe(1);
|
|
expect(fs.existsSync(path.join(tmp, "rsync.argv"))).toBe(false);
|
|
});
|
|
});
|
|
|
|
describe("report", () => {
|
|
test("posts pulledAt, files and ok to /backups/pull", () => {
|
|
const { status } = run({ envFile: writeEnv() });
|
|
expect(status).toBe(0);
|
|
const body = JSON.parse(read("curl.body"));
|
|
expect(Object.keys(body).sort()).toEqual(["files", "ok", "pulledAt"]);
|
|
expect(body).toMatchObject({ ok: true, files: 2 });
|
|
expect(body.pulledAt).toMatch(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}Z$/);
|
|
expect(read("curl.argv").split("\n")).toContain("https://health.example.test/backups/pull");
|
|
});
|
|
|
|
test("the token travels in a header file, never in argv nor in the output", () => {
|
|
const { output } = run({ envFile: writeEnv() });
|
|
expect(read("curl.header")).toBe(`Authorization: Bearer ${TOKEN}\n`);
|
|
expect(read("curl.argv")).not.toContain(TOKEN.slice(0, 12));
|
|
expect(output).not.toContain(TOKEN.slice(0, 12));
|
|
expect(fs.readdirSync(path.join(tmp, "state"))).not.toContain("auth.header");
|
|
});
|
|
|
|
test("a non-2xx answer exits 3", () => {
|
|
const { status, output } = run({ envFile: writeEnv(), extraEnv: { FAKE_CURL_STATUS: "401" } });
|
|
expect(status).toBe(3);
|
|
expect(output).toContain("report failed status=401");
|
|
});
|
|
});
|
|
|
|
describe("rotation", () => {
|
|
test("keeps 30 dated directories, today included, and ignores other names", () => {
|
|
const dest = path.join(tmp, "dest");
|
|
for (const d of ["2026-09-11", "2026-09-12", "notes"]) fs.mkdirSync(path.join(dest, d), { recursive: true });
|
|
const { status, output } = run();
|
|
expect(status).toBe(0);
|
|
// The newest of several dated directories is the one counted.
|
|
expect(output).toContain(`latest=${TODAY} files=2`);
|
|
expect(fs.existsSync(path.join(dest, "2026-09-11"))).toBe(false);
|
|
expect(fs.existsSync(path.join(dest, "2026-09-12"))).toBe(true);
|
|
expect(fs.existsSync(path.join(dest, "notes"))).toBe(true);
|
|
});
|
|
|
|
// The failure comes from a fake rm, not from file modes: the CI runs as
|
|
// root, which ignores a read-only directory.
|
|
test("a directory that cannot be removed fails loudly and reports rotation_failed", () => {
|
|
const dest = path.join(tmp, "dest");
|
|
fs.mkdirSync(path.join(dest, "2026-09-01"), { recursive: true });
|
|
writeExecutable(
|
|
path.join(bin, "rm"),
|
|
`#!/bin/sh\ncase "$*" in *2026-09-01*) exit 1 ;; esac\nexec /bin/rm "$@"\n`,
|
|
);
|
|
const { status, output } = run({ envFile: writeEnv() });
|
|
expect(status).toBe(1);
|
|
expect(output).toContain("rotation: could not remove 2026-09-01");
|
|
expect(JSON.parse(read("curl.body"))).toMatchObject({ ok: false, error: "rotation_failed" });
|
|
expect(fs.existsSync(path.join(dest, "2026-09-01"))).toBe(true);
|
|
});
|
|
});
|
|
|
|
describe("the script itself", () => {
|
|
test("never enables shell tracing — `set -x` would echo the token into journald", () => {
|
|
const traced = fs
|
|
.readFileSync(SCRIPT, "utf8")
|
|
.split("\n")
|
|
.filter((line) => !line.trim().startsWith("#"))
|
|
.filter((line) => /\bset\b[^#\n]*-[a-z]*x/.test(line));
|
|
expect(traced).toEqual([]);
|
|
});
|
|
|
|
test("is executable", () => {
|
|
expect(fs.statSync(SCRIPT).mode & 0o111).toBeGreaterThan(0);
|
|
});
|
|
});
|