cache-apt-pkgs-action/test/cache-restore.test.ts
awalsh128 eec6442bbe test: cover cache restore and save failure paths
Codecov flagged lines in #246 as uncovered patch lines; they were
untested before the reformat touched them. Add behavior tests for:

- Cache: cache miss, missing or empty manifests, unreadable cache dir,
  metapackage-only restore, install scripts running after their archive
  with dpkg actions, existing archives, unavailable cache service, and
  save failures.
- ManifestEntry.fromJSON round trip.
- buildFileList dropping missing files, updateAptLists skipping update
  for populated lists and searching only to max depth, and fromInput
  ignoring a version without a package name.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-11-09 11:35:54 -08:00

312 lines
12 KiB
TypeScript

import fs from "node:fs";
import path from "node:path";
import * as ghcache from "@actions/cache";
import { createPackageName, type CommandOptions, type PackageName } from "ts-apt";
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
import winston from "winston";
import {
Cache,
CacheKey,
CACHE_DEFAULT_DIRNAME,
MANIFEST_ALL_FILENAME,
MANIFEST_MAIN_FILENAME,
} from "../src/cache.js";
import { Manifest, ManifestEntry } from "../src/manifest.js";
import { ActionPackageNames, findInstallScript } from "../src/packages.js";
// Cache resolves its directory from os.homedir(), so point it at a temp dir.
const { HOME_DIR } = await vi.hoisted(async () => {
const nodeFs = await import("node:fs");
const nodeOs = await import("node:os");
const nodePath = await import("node:path");
return {
HOME_DIR: nodeFs.mkdtempSync(nodePath.join(nodeOs.tmpdir(), "cache-restore-test-")),
};
});
vi.mock("node:os", async (importOriginal) => {
const actual = await importOriginal<typeof import("node:os")>();
return { ...actual, homedir: () => HOME_DIR };
});
// Restore looks for install scripts under the real "/", so stub the lookup.
vi.mock("../src/packages.js", async (importOriginal) => {
const actual = await importOriginal<typeof import("../src/packages.js")>();
return { ...actual, findInstallScript: vi.fn() };
});
vi.mock("@actions/cache", () => ({
restoreCache: vi.fn(),
saveCache: vi.fn(),
isFeatureAvailable: vi.fn(),
}));
const CACHE_DIR = path.join(HOME_DIR, CACHE_DEFAULT_DIRNAME);
const CACHE_ID = 42;
const REQUESTED_PACKAGE = createPackageName("app", "1.0");
// Install order from apt, deliberately not alphabetical.
const INSTALL_ORDER = [
createPackageName("zlib1g", "1.2"),
createPackageName("libfoo", "2.0"),
REQUESTED_PACKAGE,
];
// Packages without files (e.g. metapackages) get no archive.
const METAPACKAGE = createPackageName("meta", "1.0");
const KEY = new CacheKey("v1", "1", "x86_64", ActionPackageNames.fromInput(REQUESTED_PACKAGE.serialize()));
const archivePath = (packageName: PackageName) => path.join(CACHE_DIR, packageName.serialize() + ".tar");
const filesOf = (packageName: PackageName) => [`usr/lib/${packageName.name}.so`];
const entryOf = (packageName: PackageName) => new ManifestEntry(packageName, filesOf(packageName));
const MANIFEST = new Manifest(
new Date(0),
[...INSTALL_ORDER.map(entryOf), new ManifestEntry(METAPACKAGE)],
KEY,
);
async function writeCache(archivedPackages = INSTALL_ORDER): Promise<void> {
fs.mkdirSync(CACHE_DIR, { recursive: true });
await new Manifest(MANIFEST.created, [entryOf(REQUESTED_PACKAGE)], KEY).writeToFile(
path.join(CACHE_DIR, MANIFEST_MAIN_FILENAME),
);
await MANIFEST.writeToFile(path.join(CACHE_DIR, MANIFEST_ALL_FILENAME));
archivedPackages.forEach((pkg) => fs.writeFileSync(archivePath(pkg), ""));
}
/** Archive and install script operations observed through the command runner. */
interface TarCalls {
created: [archive: string, files: string[]][];
extracted: string[];
/** Number of commands run that extracted archives. */
extractCommands?: number;
/** Extractions and script runs, in the order they ran. */
events?: string[];
}
const argValue = (args: string[], flag: string) =>
args.find((arg) => arg.startsWith(`${flag}=`))?.slice(flag.length + 1);
/** Records sudo tar and script calls, including the file list piped to tar create. */
function recordTar(calls: TarCalls) {
return vi.fn(async (_command: string, args: string[] = [], options: CommandOptions = {}) => {
const archive = argValue(args, "--file") ?? "";
if (args.includes("--create")) {
calls.created.push([archive, String(options.input ?? "").split("\0")]);
}
// Extraction runs as `sh -c <script> sh <archive>...`.
if (args[0] === "sh" && args[1] === "-c") {
calls.extracted.push(...args.slice(4));
calls.extractCommands = (calls.extractCommands ?? 0) + 1;
calls.events?.push(...args.slice(4).map((archive) => `extract ${path.basename(archive)}`));
}
// Install scripts run as `sh -x <script> <action>`.
if (args[0] === "sh" && args[1] === "-x") {
calls.events?.push(`script ${path.basename(args[2] ?? "")} ${args[3]}`);
}
return { cmdLine: "", stdout: "", stderr: "", exitCode: 0 };
});
}
function createCache(calls: TarCalls = { created: [], extracted: [] }): Cache {
const logger = winston.createLogger({ silent: true });
return new Cache({ run: recordTar(calls) } as never, logger);
}
describe("Cache", () => {
beforeEach(() => {
vi.mocked(ghcache.restoreCache).mockResolvedValue(KEY.hash);
vi.mocked(ghcache.saveCache).mockResolvedValue(CACHE_ID);
vi.mocked(ghcache.isFeatureAvailable).mockReturnValue(true);
vi.mocked(findInstallScript).mockResolvedValue(undefined);
});
afterEach(() => {
fs.rmSync(CACHE_DIR, { recursive: true, force: true });
vi.clearAllMocks();
});
it("loadAndRestore cache hit extracts archives in install order", async () => {
await writeCache();
const calls: TarCalls = { created: [], extracted: [] };
const manifest = await createCache(calls).loadAndRestore(KEY, false);
expect(manifest).toBeDefined();
expect(calls.extracted).toEqual(INSTALL_ORDER.map(archivePath));
});
it("loadAndRestore without install scripts extracts all archives in one command", async () => {
await writeCache();
const calls: TarCalls = { created: [], extracted: [] };
await createCache(calls).loadAndRestore(KEY, false);
expect(calls.extractCommands).toBe(1);
});
it("loadAndRestore with install scripts extracts archives one at a time in install order", async () => {
await writeCache();
const calls: TarCalls = { created: [], extracted: [] };
await createCache(calls).loadAndRestore(KEY, true);
expect(calls.extracted).toEqual(INSTALL_ORDER.map(archivePath));
expect(calls.extractCommands).toBe(INSTALL_ORDER.length);
});
it("loadAndRestore archive missing returns undefined without extracting", async () => {
await writeCache(INSTALL_ORDER.slice(1));
const calls: TarCalls = { created: [], extracted: [] };
const manifest = await createCache(calls).loadAndRestore(KEY, false);
expect(manifest).toBeUndefined();
expect(calls.extracted).toEqual([]);
});
it("archiveAndSave packages with files creates archives of their files", async () => {
const calls: TarCalls = { created: [], extracted: [] };
const cacheId = await createCache(calls).archiveAndSave(MANIFEST);
expect(cacheId).toBe(CACHE_ID);
expect(calls.created).toEqual(INSTALL_ORDER.map((pkg) => [archivePath(pkg), filesOf(pkg)]));
});
it("archiveAndSave manifest written records package files", async () => {
await createCache().archiveAndSave(MANIFEST);
const written = await Manifest.readFromFile(path.join(CACHE_DIR, MANIFEST_ALL_FILENAME));
expect(written.entries.map((entry) => entry.filepaths)).toEqual(
MANIFEST.entries.map((entry) => entry.filepaths),
);
});
it("loadAndRestore cache hit returns all installed packages", async () => {
await writeCache();
const manifest = await createCache().loadAndRestore(KEY, false);
expect(manifest?.entries.map((entry) => entry.packageName.serialize())).toEqual(
MANIFEST.entries.map((entry) => entry.packageName.serialize()),
);
});
it("archiveAndSave requested package without version records installed entry in main manifest", async () => {
const unversionedKey = new CacheKey(
"v1",
"1",
"x86_64",
ActionPackageNames.fromInput(REQUESTED_PACKAGE.name),
);
await createCache().archiveAndSave(new Manifest(MANIFEST.created, MANIFEST.entries, unversionedKey));
const written = await Manifest.readFromFile(path.join(CACHE_DIR, MANIFEST_MAIN_FILENAME));
expect(written.entries.map((entry) => [entry.packageName.serialize(), entry.filepaths])).toEqual([
[REQUESTED_PACKAGE.serialize(), filesOf(REQUESTED_PACKAGE)],
]);
});
it("loadAndRestore cache miss returns undefined without extracting", async () => {
vi.mocked(ghcache.restoreCache).mockResolvedValue(undefined);
const calls: TarCalls = { created: [], extracted: [] };
const manifest = await createCache(calls).loadAndRestore(KEY, false);
expect(manifest).toBeUndefined();
expect(calls.extracted).toEqual([]);
});
it.each([
["main manifest missing", MANIFEST_MAIN_FILENAME, undefined],
["all manifest missing", MANIFEST_ALL_FILENAME, undefined],
["all manifest empty", MANIFEST_ALL_FILENAME, new Manifest(MANIFEST.created, [], KEY)],
])("loadAndRestore %s returns undefined without extracting", async (_case, filename, replacement) => {
await writeCache();
const filePath = path.join(CACHE_DIR, filename);
fs.rmSync(filePath);
await replacement?.writeToFile(filePath);
const calls: TarCalls = { created: [], extracted: [] };
const manifest = await createCache(calls).loadAndRestore(KEY, false);
expect(manifest).toBeUndefined();
expect(calls.extracted).toEqual([]);
});
it("loadAndRestore cache hit without cache directory logs it as unreadable and returns undefined", async () => {
const logger = winston.createLogger({ silent: true });
const errorSpy = vi.spyOn(logger, "error");
const cache = new Cache({ run: vi.fn() } as never, logger);
const manifest = await cache.loadAndRestore(KEY, false);
expect(manifest).toBeUndefined();
expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining("Unable to read cache directory contents"));
});
it("loadAndRestore only packages without files runs no extract command", async () => {
fs.mkdirSync(CACHE_DIR, { recursive: true });
const metaOnly = new Manifest(MANIFEST.created, [new ManifestEntry(METAPACKAGE)], KEY);
await metaOnly.writeToFile(path.join(CACHE_DIR, MANIFEST_MAIN_FILENAME));
await metaOnly.writeToFile(path.join(CACHE_DIR, MANIFEST_ALL_FILENAME));
const calls: TarCalls = { created: [], extracted: [] };
const manifest = await createCache(calls).loadAndRestore(KEY, false);
expect(manifest?.entries).toHaveLength(1);
expect(calls.extractCommands).toBeUndefined();
});
it("loadAndRestore with install scripts runs each found script after its archive with the dpkg action", async () => {
await writeCache();
const [first, second] = INSTALL_ORDER;
vi.mocked(findInstallScript).mockImplementation(async (packageName, scriptType) =>
packageName.name === first!.name ? `/var/lib/dpkg/info/${packageName.name}.${scriptType}` : undefined,
);
const calls: TarCalls = { created: [], extracted: [], events: [] };
await createCache(calls).loadAndRestore(KEY, true);
expect(calls.events?.slice(0, 4)).toEqual([
`extract ${path.basename(archivePath(first!))}`,
`script ${first!.name}.preinst install`,
`script ${first!.name}.postinst configure`,
`extract ${path.basename(archivePath(second!))}`,
]);
});
it("archiveAndSave cache service unavailable throws without archiving", async () => {
vi.mocked(ghcache.isFeatureAvailable).mockReturnValue(false);
const calls: TarCalls = { created: [], extracted: [] };
await expect(createCache(calls).archiveAndSave(MANIFEST)).rejects.toThrow("not available");
expect(calls.created).toEqual([]);
});
it("archiveAndSave archive already exists skips recreating it", async () => {
const [existing, ...rest] = INSTALL_ORDER;
fs.mkdirSync(CACHE_DIR, { recursive: true });
fs.writeFileSync(archivePath(existing!), "");
const calls: TarCalls = { created: [], extracted: [] };
await createCache(calls).archiveAndSave(MANIFEST);
expect(calls.created.map(([archive]) => archive)).toEqual(rest.map(archivePath));
});
it.each([
["no cache ID", () => Promise.resolve(undefined), "no cache action ID returned"],
["an Error", () => Promise.reject(new Error("upload failed")), "upload failed"],
["a non-Error", () => Promise.reject("upload failed"), "upload failed"],
])("archiveAndSave save returns %s throws a save error", async (_case, saveResult, message) => {
vi.mocked(ghcache.saveCache).mockImplementation(saveResult as never);
await expect(createCache().archiveAndSave(MANIFEST)).rejects.toThrow(
new RegExp(`^Failed to save cache: .*${message}`),
);
});
});