test: tighten release check assertions

This commit is contained in:
Peter Steinberger
2026-05-09 10:47:02 +01:00
parent 446f4b2e2e
commit bd44369a26

View File

@@ -145,7 +145,7 @@ describe("packed CLI smoke", () => {
OPENCLAW_STATE_DIR: "/tmp/smoke-state", OPENCLAW_STATE_DIR: "/tmp/smoke-state",
}, },
), ),
).toMatchObject({ ).toEqual({
PATH: "/usr/bin", PATH: "/usr/bin",
HOME: "/tmp/smoke-home", HOME: "/tmp/smoke-home",
OPENCLAW_STATE_DIR: "/tmp/smoke-state", OPENCLAW_STATE_DIR: "/tmp/smoke-state",
@@ -380,9 +380,9 @@ describe("collectForbiddenPackPaths", () => {
it("keeps local build metadata excluded by package files", () => { it("keeps local build metadata excluded by package files", () => {
const pkg = JSON.parse(readFileSync("package.json", "utf8")) as { files?: string[] }; const pkg = JSON.parse(readFileSync("package.json", "utf8")) as { files?: string[] };
expect(pkg.files).toEqual( for (const entry of LOCAL_BUILD_METADATA_DIST_PATHS) {
expect.arrayContaining(LOCAL_BUILD_METADATA_DIST_PATHS.map((entry) => `!${entry}`)), expect(pkg.files).toContain(`!${entry}`);
); }
}); });
it("blocks legacy runtime dependency stamps from npm pack output", () => { it("blocks legacy runtime dependency stamps from npm pack output", () => {
@@ -483,27 +483,27 @@ describe("collectMissingPackPaths", () => {
"dist/build-info.json", "dist/build-info.json",
]); ]);
expect(missing).toEqual( for (const path of [
expect.arrayContaining([ "dist/channel-catalog.json",
"dist/channel-catalog.json", PACKAGE_DIST_INVENTORY_RELATIVE_PATH,
PACKAGE_DIST_INVENTORY_RELATIVE_PATH, "dist/control-ui/index.html",
"dist/control-ui/index.html", "scripts/npm-runner.mjs",
"scripts/npm-runner.mjs", "scripts/preinstall-package-manager-warning.mjs",
"scripts/preinstall-package-manager-warning.mjs", "scripts/lib/official-external-channel-catalog.json",
"scripts/lib/official-external-channel-catalog.json", "scripts/lib/official-external-plugin-catalog.json",
"scripts/lib/official-external-plugin-catalog.json", "scripts/lib/official-external-provider-catalog.json",
"scripts/lib/official-external-provider-catalog.json", "scripts/lib/package-dist-imports.mjs",
"scripts/lib/package-dist-imports.mjs", "scripts/postinstall-bundled-plugins.mjs",
"scripts/postinstall-bundled-plugins.mjs", "dist/task-registry-control.runtime.js",
"dist/task-registry-control.runtime.js", bundledDistPluginFile("slack", "runtime-api.js"),
bundledDistPluginFile("slack", "runtime-api.js"), bundledDistPluginFile("slack", "openclaw.plugin.json"),
bundledDistPluginFile("slack", "openclaw.plugin.json"), bundledDistPluginFile("slack", "package.json"),
bundledDistPluginFile("slack", "package.json"), bundledDistPluginFile("telegram", "runtime-api.js"),
bundledDistPluginFile("telegram", "runtime-api.js"), bundledDistPluginFile("telegram", "openclaw.plugin.json"),
bundledDistPluginFile("telegram", "openclaw.plugin.json"), bundledDistPluginFile("telegram", "package.json"),
bundledDistPluginFile("telegram", "package.json"), ]) {
]), expect(missing).toContain(path);
); }
}); });
it("accepts the shipped upgrade surface when optional bundled metadata is present", () => { it("accepts the shipped upgrade surface when optional bundled metadata is present", () => {
@@ -535,11 +535,11 @@ describe("collectMissingPackPaths", () => {
}); });
it("requires bundled plugin runtime sidecars that dynamic plugin boundaries resolve at runtime", () => { it("requires bundled plugin runtime sidecars that dynamic plugin boundaries resolve at runtime", () => {
expect(requiredBundledPluginPackPaths).toEqual( expect(requiredBundledPluginPackPaths).toContain(
expect.arrayContaining([ bundledDistPluginFile("slack", "runtime-api.js"),
bundledDistPluginFile("slack", "runtime-api.js"), );
bundledDistPluginFile("telegram", "runtime-api.js"), expect(requiredBundledPluginPackPaths).toContain(
]), bundledDistPluginFile("telegram", "runtime-api.js"),
); );
}); });
}); });