test: clarify boolean membership assertions · openclaw/op...

Recent Commits to openclaw:main

steipete · 2026-05-08 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -495,7 +495,12 @@ describe("Discord model picker rendering", () => {`
`495`	`495`	`throw new Error("models view did not render a provider select");`
`496`	`496`	`}`
`497`	`497`	`expect(providerSelect.options?.length).toBe(2);`
`498`		`-expect(providerSelect.options?.find((option) => option.value === "openai")?.default).toBe(true);`
	`498`	`+expect(providerSelect.options).toContainEqual(`
	`499`	`+expect.objectContaining({`
	`500`	`+value: "openai",`
	`501`	`+default: true,`
	`502`	`+}),`
	`503`	`+);`
`499`	`504`	`const parsedProviderState = parseDiscordModelPickerCustomId(providerSelect.custom_id ?? "");`
`500`	`505`	`expect(parsedProviderState?.action).toBe("provider");`
`501`	`506`
`@@ -506,7 +511,12 @@ describe("Discord model picker rendering", () => {`
`506`	`511`	`throw new Error("models view did not render a model select");`
`507`	`512`	`}`
`508`	`513`	`expect(modelSelect.options?.length).toBe(3);`
`509`		`-expect(modelSelect.options?.find((option) => option.value === "o3")?.default).toBe(true);`
	`514`	`+expect(modelSelect.options).toContainEqual(`
	`515`	`+expect.objectContaining({`
	`516`	`+value: "o3",`
	`517`	`+default: true,`
	`518`	`+}),`
	`519`	`+);`
`510`	`520`
`511`	`521`	`const parsedModelSelectState = parseDiscordModelPickerCustomId(modelSelect.custom_id ?? "");`
`512`	`522`	`expect(parsedModelSelectState?.action).toBe("model");`
`@@ -577,7 +587,12 @@ describe("Discord model picker rendering", () => {`
`577`	`587`	`expect(runtimeSelect.options?.find((option) => option.value === "pi")?.label).toBe(`
`578`	`588`	`"OpenClaw Pi Default",`
`579`	`589`	`);`
`580`		`-expect(runtimeSelect.options?.find((option) => option.value === "codex")?.default).toBe(true);`
	`590`	`+expect(runtimeSelect.options).toContainEqual(`
	`591`	`+expect.objectContaining({`
	`592`	`+value: "codex",`
	`593`	`+default: true,`
	`594`	`+}),`
	`595`	`+);`
`581`	`596`
`582`	`597`	`const submitButton = rows[3]?.components?.at(-1);`
`583`	`598`	`const submitState = requireValue(`

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,12 @@ describe("QQBot framework slash commands", () => {`
`37`	`37`	`expect.arrayContaining(["bot-approve", "bot-clear-storage", "bot-logs", "bot-streaming"]),`
`38`	`38`	`);`
`39`	`39`	`for (const commandName of ["bot-approve", "bot-clear-storage", "bot-logs", "bot-streaming"]) {`
`40`		`-expect(commands.find((command) => command.name === commandName)?.c2cOnly).toBe(true);`
	`40`	`+expect(commands).toContainEqual(`
	`41`	`+expect.objectContaining({`
	`42`	`+name: commandName,`
	`43`	`+c2cOnly: true,`
	`44`	`+}),`
	`45`	`+);`
`41`	`46`	`}`
`42`	`47`	`});`
`43`	`48`
`@@ -60,7 +65,12 @@ describe("QQBot framework slash commands", () => {`
`60`	`65`	`const commands = registry.getFrameworkCommands();`
`61`	`66`
`62`	`67`	`expect(commands.map((command) => command.name)).toEqual(["private-admin", "shared-admin"]);`
`63`		`-expect(commands.find((command) => command.name === "private-admin")?.c2cOnly).toBe(true);`
	`68`	`+expect(commands).toContainEqual(`
	`69`	`+expect.objectContaining({`
	`70`	`+name: "private-admin",`
	`71`	`+c2cOnly: true,`
	`72`	`+}),`
	`73`	`+);`
`64`	`74`	`expect(commands.find((command) => command.name === "shared-admin")?.c2cOnly).toBeUndefined();`
`65`	`75`	`});`
`66`	`76`

Original file line number	Diff line number	Diff line change
`@@ -157,8 +157,18 @@ describe("buildWorkspaceSkillStatus", () => {`
`157`	`157`	`expect(report.agentId).toBe("specialist");`
`158`	`158`	`expect(report.agentSkillFilter).toEqual(["alpha"]);`
`159`	`159`	`expect(report.skills.find((skill) => skill.name === "alpha")?.blockedByAgentFilter).toBe(false);`
`160`		`-expect(report.skills.find((skill) => skill.name === "alpha")?.modelVisible).toBe(true);`
`161`		`-expect(report.skills.find((skill) => skill.name === "beta")?.blockedByAgentFilter).toBe(true);`
	`160`	`+expect(report.skills).toContainEqual(`
	`161`	`+expect.objectContaining({`
	`162`	`+name: "alpha",`
	`163`	`+modelVisible: true,`
	`164`	`+}),`
	`165`	`+);`
	`166`	`+expect(report.skills).toContainEqual(`
	`167`	`+expect.objectContaining({`
	`168`	`+name: "beta",`
	`169`	`+blockedByAgentFilter: true,`
	`170`	`+}),`
	`171`	`+);`
`162`	`172`	`expect(report.skills.find((skill) => skill.name === "beta")?.modelVisible).toBe(false);`
`163`	`173`	`});`
`164`	`174`

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ describe("inter-session lastRoute preservation (fixes #54441)", () => {`
`59`	`59`	`isInterSession: true,`
`60`	`60`	`});`
`61`	`61`	`// No external route existed — falls through to normal resolution (webchat or undefined).`
`62`		`-expect(result === "webchat" \|\| result === undefined).toBe(true);`
	`62`	`+expect(["webchat", undefined]).toContain(result);`
`63`	`63`	`});`
`64`	`64`
`65`	`65`	`it("inter-session on session with no persisted lastTo preserves session route", () => {`
`@@ -73,7 +73,7 @@ describe("inter-session lastRoute preservation (fixes #54441)", () => {`
`73`	`73`	`isInterSession: true,`
`74`	`74`	`});`
`75`	`75`	`// No external route — falls through to normal resolution`
`76`		`-expect(result === "session:somekey" \|\| result === undefined).toBe(true);`
	`76`	`+expect(["session:somekey", undefined]).toContain(result);`
`77`	`77`	`});`
`78`	`78`	`});`
`79`	`79`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main