test: tighten doctor empty config assertions

This commit is contained in:
Shakker
2026-05-09 03:55:40 +01:00
parent c9f3b099b2
commit 8989d0a777
3 changed files with 5 additions and 5 deletions

View File

@@ -50,10 +50,10 @@ describe("doctor config flow safe bins", () => {
}>;
};
};
expect(cfg.tools?.exec?.safeBinProfiles?.myfilter).toEqual({});
expect(cfg.tools?.exec?.safeBinProfiles?.myfilter).toStrictEqual({});
expect(cfg.tools?.exec?.safeBinProfiles?.python3).toBeUndefined();
const ops = cfg.agents?.list?.find((entry) => entry.id === "ops");
expect(ops?.tools?.exec?.safeBinProfiles?.mytool).toEqual({});
expect(ops?.tools?.exec?.safeBinProfiles?.mytool).toStrictEqual({});
expect(ops?.tools?.exec?.safeBinProfiles?.node).toBeUndefined();
});

View File

@@ -157,7 +157,7 @@ describe("doctor config flow steps", () => {
});
expect(result.removed).toEqual(["bogus"]);
expect(result.state.candidate).toEqual({});
expect(result.state.candidate).toStrictEqual({});
expect(result.state.fixHints).toContain('Run "openclaw doctor --fix" to remove these keys.');
});

View File

@@ -97,7 +97,7 @@ describe("doctor exec safe bin helpers", () => {
"- tools.exec.safeBins includes interpreter/runtime 'awk' without profile; remove it from safeBins or use explicit allowlist entries.",
"- tools.exec.safeBins includes interpreter/runtime 'sed' without profile; remove it from safeBins or use explicit allowlist entries.",
]);
expect(result.config.tools?.exec?.safeBinProfiles).toEqual({});
expect(result.config.tools?.exec?.safeBinProfiles).toStrictEqual({});
});
it("warns on busybox/toybox safeBins instead of scaffolding them", () => {
@@ -114,7 +114,7 @@ describe("doctor exec safe bin helpers", () => {
"- tools.exec.safeBins includes interpreter/runtime 'busybox' without profile; remove it from safeBins or use explicit allowlist entries.",
"- tools.exec.safeBins includes interpreter/runtime 'toybox' without profile; remove it from safeBins or use explicit allowlist entries.",
]);
expect(result.config.tools?.exec?.safeBinProfiles).toEqual({});
expect(result.config.tools?.exec?.safeBinProfiles).toStrictEqual({});
});
it("flags safeBins that resolve outside trusted directories", () => {