test: tighten command extension helper assertions · openclaw/openclaw@d456dd1

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,6 @@ describe("fetchDiscord", () => {`
`142`	`142`	`});`
`143`	`143`
`144`	`144`	`expect(result).toEqual({ id: "42" });`
`145`		`-expect(request).toBeDefined();`
`146`	`145`	`if (!request) {`
`147`	`146`	`throw new Error("expected Discord request init");`
`148`	`147`	`}`

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@ describe("resolve-allowlist-common", () => {`
`14`	`14`
`15`	`15`	`it("resolves and filters guilds by id or name", () => {`
`16`	`16`	`const mainGuild = findDiscordGuildByName(guilds, "Main Guild");`
`17`		`-expect(mainGuild).toBeDefined();`
`18`	`17`	`if (!mainGuild) {`
`19`	`18`	`throw new Error("expected Main Guild lookup result");`
`20`	`19`	`}`

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,6 @@ describe("firecrawl tools", () => {`
`83`	`83`	`expect(provider.id).toBe("firecrawl");`
`84`	`84`	`expect(provider.credentialPath).toBe("plugins.entries.firecrawl.config.webSearch.apiKey");`
`85`	`85`	`const pluginEntry = applied.plugins?.entries?.firecrawl;`
`86`		`-expect(pluginEntry).toBeDefined();`
`87`	`86`	`if (!pluginEntry) {`
`88`	`87`	`throw new Error("expected Firecrawl plugin entry");`
`89`	`88`	`}`
`@@ -354,7 +353,6 @@ describe("firecrawl tools", () => {`
`354`	`353`	`expect(provider.id).toBe("firecrawl");`
`355`	`354`	`expect(provider.credentialPath).toBe("plugins.entries.firecrawl.config.webFetch.apiKey");`
`356`	`355`	`const pluginEntry = applied.plugins?.entries?.firecrawl;`
`357`		`-expect(pluginEntry).toBeDefined();`
`358`	`356`	`if (!pluginEntry) {`
`359`	`357`	`throw new Error("expected Firecrawl fetch plugin entry");`
`360`	`358`	`}`

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,6 @@ function jsonResponse(payload: unknown, status = 200) {`
`18`	`18`
`19`	`19`	`function requireFirstFetchInput(fetchImpl: ReturnType<typeof vi.fn>): RequestInfo \| URL {`
`20`	`20`	`const input = fetchImpl.mock.calls[0]?.[0] as RequestInfo \| URL \| undefined;`
`21`		`-expect(input).toBeDefined();`
`22`	`21`	`if (!input) {`
`23`	`22`	`throw new Error("expected fetch input");`
`24`	`23`	`}`

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,6 @@ async function expectPostJsonRequest(run: (token: string, fetcher: ZaloFetch) =>`
`18`	`18`	`await run("test-token", fetcher);`
`19`	`19`	`expect(fetcher).toHaveBeenCalledTimes(1);`
`20`	`20`	`const [, init] = fetcher.mock.calls[0] ?? [];`
`21`		`-expect(init).toBeDefined();`
`22`	`21`	`if (!init) {`
`23`	`22`	`throw new Error("expected Zalo request init");`
`24`	`23`	`}`
`@@ -71,11 +70,9 @@ describe("Zalo API request methods", () => {`
`71`	`70`
`72`	`71`	`await rejected;`
`73`	`72`	`const [, init] = fetcher.mock.calls[0] ?? [];`
`74`		`-expect(init).toBeDefined();`
`75`	`73`	`if (!init) {`
`76`	`74`	`throw new Error("expected Zalo chat action request init");`
`77`	`75`	`}`
`78`		`-expect(init.signal).toBeDefined();`
`79`	`76`	`if (!init.signal) {`
`80`	`77`	`throw new Error("expected Zalo chat action abort signal");`
`81`	`78`	`}`

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,6 @@ describe("zalo setup wizard", () => {`
`61`	`61`
`62`	`62`	`expect(result.accountId).toBe("default");`
`63`	`63`	`const zaloConfig = result.cfg.channels?.zalo;`
`64`		`-expect(zaloConfig).toBeDefined();`
`65`	`64`	`if (!zaloConfig) {`
`66`	`65`	`throw new Error("expected Zalo config");`
`67`	`66`	`}`
`@@ -123,15 +122,13 @@ describe("zalo setup wizard", () => {`
`123`	`122`
`124`	`123`	`const next = zaloDmPolicy.setPolicy(cfg, "open");`
`125`	`124`	`const zaloConfig = next.channels?.zalo;`
`126`		`-expect(zaloConfig).toBeDefined();`
`127`	`125`	`if (!zaloConfig) {`
`128`	`126`	`throw new Error("expected Zalo config");`
`129`	`127`	`}`
`130`	`128`	`expect(zaloConfig.dmPolicy).toBe("disabled");`
`131`	`129`	`const workAccount = next.channels?.zalo?.accounts?.work as`
`132`	`130`	`\| { dmPolicy?: string; allowFrom?: Array<string \| number> }`
`133`	`131`	`\| undefined;`
`134`		`-expect(workAccount).toBeDefined();`
`135`	`132`	`if (!workAccount) {`
`136`	`133`	`throw new Error("expected Zalo work account");`
`137`	`134`	`}`
`@@ -157,15 +154,13 @@ describe("zalo setup wizard", () => {`
`157`	`154`	`);`
`158`	`155`
`159`	`156`	`const zaloConfig = next.channels?.zalo;`
`160`		`-expect(zaloConfig).toBeDefined();`
`161`	`157`	`if (!zaloConfig) {`
`162`	`158`	`throw new Error("expected Zalo config");`
`163`	`159`	`}`
`164`	`160`	`expect(zaloConfig.dmPolicy).toBeUndefined();`
`165`	`161`	`const workAccount = next.channels?.zalo?.accounts?.work as`
`166`	`162`	`\| { dmPolicy?: string; allowFrom?: Array<string \| number> }`
`167`	`163`	`\| undefined;`
`168`		`-expect(workAccount).toBeDefined();`
`169`	`164`	`if (!workAccount) {`
`170`	`165`	`throw new Error("expected Zalo work account");`
`171`	`166`	`}`

Original file line number	Diff line number	Diff line change
`@@ -33,12 +33,10 @@ describe("zalouser setup wizard", () => {`
`33`	`33`	`) {`
`34`	`34`	`expect(result.accountId).toBe("default");`
`35`	`35`	`const channelConfig = result.cfg.channels?.zalouser;`
`36`		`-expect(channelConfig).toBeDefined();`
`37`	`36`	`if (!channelConfig) {`
`38`	`37`	`throw new Error("expected Zalo Personal channel config");`
`39`	`38`	`}`
`40`	`39`	`const pluginEntry = result.cfg.plugins?.entries?.zalouser;`
`41`		`-expect(pluginEntry).toBeDefined();`
`42`	`40`	`if (!pluginEntry) {`
`43`	`41`	`throw new Error("expected Zalo Personal plugin entry");`
`44`	`42`	`}`

Original file line number	Diff line number	Diff line change
`@@ -44,11 +44,11 @@ async function writeIdentityFile(workspace: string, lines: string[]) {`
`44`	`44`
`45`	`45`	`function getWrittenMainIdentity() {`
`46`	`46`	`const [written] = configMocks.writeConfigFile.mock.calls[0] ?? [];`
`47`		`-expect(written).toBeDefined();`
`48`	`47`	`if (!written) {`
`49`	`48`	`throw new Error("expected written agent config");`
`50`	`49`	`}`
`51`		`-return written.agents?.list?.find((entry) => entry.id === "main")?.identity;`
	`50`	`+const payload = written as ConfigWritePayload;`
	`51`	`+return payload.agents?.list?.find((entry) => entry.id === "main")?.identity;`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`async function runIdentityCommandFromWorkspace(workspace: string, fromIdentity = true) {`

Original file line number	Diff line number	Diff line change
`@@ -340,7 +340,6 @@ describe("channels command", () => {`
`340`	`340`	`function getWrittenConfig<T>(): T {`
`341`	`341`	`expect(configMocks.writeConfigFile).toHaveBeenCalledTimes(1);`
`342`	`342`	`const [config] = configMocks.writeConfigFile.mock.calls[0] ?? [];`
`343`		`-expect(config).toBeDefined();`
`344`	`343`	`if (config === undefined) {`
`345`	`344`	`throw new Error("expected written channel config");`
`346`	`345`	`}`

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,6 @@ vi.mock("../plugin-sdk/facade-loader.js", () => ({`
`10`	`10`
`11`	`11`	`function requireFirstNoteCall(noteFn: ReturnType<typeof vi.fn>): unknown[] {`
`12`	`12`	`const call = noteFn.mock.calls[0];`
`13`		`-expect(call).toBeDefined();`
`14`	`13`	`if (!call) {`
`15`	`14`	`throw new Error("expected browser doctor note");`
`16`	`15`	`}`

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

推荐订阅源

Recent Commits to openclaw:main