vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-27 02:07:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: tighten doctor repair sequencing assertions

Browse Source
This commit is contained in:
Peter Steinberger
2026-05-10 01:35:32 +01:00
parent 77d608d02a
commit 5d0045786b
1 changed files with 16 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
src/commands/doctor/repair-sequencing.test.ts
View File

@@ -251,21 +251,17 @@ describe("doctor repair sequencing", () => {
expect(result.state.pendingChanges).toBe(true);
expect(result.state.candidate.channels?.discord?.allowFrom).toEqual(["123"]);
expect(result.changeNotes).toEqual(
expect.arrayContaining([
expect.stringContaining("channels.discord.allowFrom: converted 1 numeric ID to strings"),
expect.stringContaining(
"channels.tools.exec.toolsBySender: migrated 1 legacy key to typed id: entries",
),
]),
expect(result.changeNotes.join("\n")).toContain(
"channels.discord.allowFrom: converted 1 numeric ID to strings",
);
expect(result.changeNotes.join("\n")).toContain(
"channels.tools.exec.toolsBySender: migrated 1 legacy key to typed id: entries",
);
expect(result.changeNotes.join("\n")).toContain("bad-keynext -> id:bad-keynext");
expect(result.changeNotes.join("\n")).not.toContain("\u001B");
expect(result.changeNotes.join("\n")).not.toContain("\r");
expect(result.warningNotes).toEqual(
expect.arrayContaining([
expect.stringContaining("channels.signal.accounts.ops-teamnext.dmPolicy"),
]),
expect(result.warningNotes.join("\n")).toContain(
"channels.signal.accounts.ops-teamnext.dmPolicy",
);
expect(result.warningNotes.join("\n")).not.toContain("\u001B");
expect(result.warningNotes.join("\n")).not.toContain("\r");
@@ -305,18 +301,10 @@ describe("doctor repair sequencing", () => {
});
expect(events).toEqual(["cleanup", "missing-installs"]);
expect(mocks.maybeRepairStaleManagedNpmBundledPlugins).toHaveBeenCalledWith(
expect.objectContaining({
config: expect.objectContaining({
plugins: expect.objectContaining({
entries: expect.objectContaining({
"google-meet": { enabled: true },
}),
}),
}),
prompter: { shouldRepair: true },
}),
);
expect(mocks.maybeRepairStaleManagedNpmBundledPlugins).toHaveBeenCalledOnce();
const cleanupCall = mocks.maybeRepairStaleManagedNpmBundledPlugins.mock.calls[0]?.[0];
expect(cleanupCall?.config.plugins?.entries?.["google-meet"]).toEqual({ enabled: true });
expect(cleanupCall?.prompter).toEqual({ shouldRepair: true });
});
it("emits Discord warnings when unsafe numeric ids block repair", async () => {
@@ -389,11 +377,11 @@ describe("doctor repair sequencing", () => {
expect(result.state.pendingChanges).toBe(true);
expect(result.state.candidate.plugins?.allow).toEqual(["telegram", "brave"]);
expect(result.state.candidate.plugins?.entries?.brave?.enabled).toBe(true);
expect(result.changeNotes).toEqual(
expect.arrayContaining([
'Installed missing configured plugin "brave" from @openclaw/brave-plugin.',
"brave web search provider selected, enabled automatically.",
]),
expect(result.changeNotes).toContain(
'Installed missing configured plugin "brave" from @openclaw/brave-plugin.',
);
expect(result.changeNotes).toContain(
"brave web search provider selected, enabled automatically.",
);
});