refactor: share skills handler test helper · openclaw/ope...

Recent Commits to openclaw:main

vincentkoc · 2026-06-01 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`import { beforeEach, describe, expect, it, vi } from "vitest";`
	`2`	`+import { callGatewayHandler } from "./skills.test-helpers.js";`
`2`	`3`
`3`	`4`	`const loadConfigMock = vi.fn(() => ({}));`
`4`	`5`	`const resolveDefaultAgentIdMock = vi.fn(() => "main");`
`@@ -46,16 +47,7 @@ vi.mock("../../infra/clawhub.js", () => ({`
`46`	`47`
`47`	`48`	`const { skillsHandlers } = await import("./skills.js");`
`48`	`49`
`49`		`-const makeContext = () => ({ getRuntimeConfig: () => ({}) });`
`50`	`50`	`type SkillsHandlerName = keyof typeof skillsHandlers;`
`51`		`-type SkillHandler = (options: {`
`52`		`-params: unknown;`
`53`		`-req: never;`
`54`		`-client: never;`
`55`		`-isWebchatConnect: () => boolean;`
`56`		`-context: never;`
`57`		`-respond: (success: boolean, result?: unknown, err?: unknown) => void;`
`58`		`-}) => Promise<void> \| void;`
`59`	`51`
`60`	`52`	`function emptySkillStatusReport() {`
`61`	`53`	`return {`
`@@ -65,26 +57,8 @@ function emptySkillStatusReport() {`
`65`	`57`	`};`
`66`	`58`	`}`
`67`	`59`
`68`		`-async function callSkillsHandler(method: SkillsHandlerName, params: unknown) {`
`69`		`-let ok: boolean \| null = null;`
`70`		`-let response: unknown;`
`71`		`-let error: unknown;`
`72`		`-const handler = skillsHandlers[method] as SkillHandler;`
`73`		`-`
`74`		`-await handler({`
`75`		`- params,`
`76`		`-req: {} as never,`
`77`		`-client: null as never,`
`78`		`-isWebchatConnect: () => false,`
`79`		`-context: makeContext() as never,`
`80`		`-respond: (success, result, err) => {`
`81`		`-ok = success;`
`82`		`-response = result;`
`83`		`-error = err;`
`84`		`-},`
`85`		`-});`
`86`		`-`
`87`		`-return { ok, response, error };`
	`60`	`+async function callSkillsHandler(method: SkillsHandlerName, params: Record<string, unknown>) {`
	`61`	`+return callGatewayHandler(skillsHandlers, method, params);`
`88`	`62`	`}`
`89`	`63`
`90`	`64`	`function expectEmptySecurityVerdicts(response: unknown): void {`

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@ import path from "node:path";`
`3`	`3`	`import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";`
`4`	`4`	`import { captureEnv } from "../../test-utils/env.js";`
`5`	`5`	`import { createTrackedTempDirs } from "../../test-utils/tracked-temp-dirs.js";`
	`6`	`+import { callGatewayHandler } from "./skills.test-helpers.js";`
`6`	`7`
`7`	`8`	`const tempDirs = createTrackedTempDirs();`
`8`	`9`	`let envSnapshot: ReturnType<typeof captureEnv>;`
`@@ -49,30 +50,8 @@ vi.mock("../../skills/security/clawhub-verdicts.js", () => ({`
`49`	`50`
`50`	`51`	`const { skillsHandlers } = await import("./skills.js");`
`51`	`52`
`52`		`-function makeContext() {`
`53`		`-return {`
`54`		`-getRuntimeConfig: () => ({}),`
`55`		`-logGateway: vi.fn(),`
`56`		`-};`
`57`		`-}`
`58`		`-`
`59`		`-async function callHandler(method: string, params: Record<string, unknown>) {`
`60`		`-let ok: boolean \| null = null;`
`61`		`-let response: unknown;`
`62`		`-let error: unknown;`
`63`		`-await skillsHandlers[method]({`
`64`		`- params,`
`65`		`-req: {} as never,`
`66`		`-client: null as never,`
`67`		`-isWebchatConnect: () => false,`
`68`		`-context: makeContext() as never,`
`69`		`-respond: (success, result, err) => {`
`70`		`-ok = success;`
`71`		`-response = result;`
`72`		`-error = err;`
`73`		`-},`
`74`		`-});`
`75`		`-return { ok, response, error };`
	`53`	`+function callHandler(method: string, params: Record<string, unknown>) {`
	`54`	`+return callGatewayHandler(skillsHandlers, method, params);`
`76`	`55`	`}`
`77`	`56`
`78`	`57`	`describe("skills proposal gateway handlers", () => {`

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,45 @@`
	`1`	`+import { vi } from "vitest";`
	`2`	`+import type { GatewayRequestContext, GatewayRequestHandlers } from "./types.js";`
	`3`	`+`
	`4`	`+export type CapturedGatewayResponse = {`
	`5`	`+ok: boolean \| null;`
	`6`	`+response: unknown;`
	`7`	`+error: unknown;`
	`8`	`+};`
	`9`	`+`
	`10`	`+function makeGatewayHandlerTestContext(): GatewayRequestContext {`
	`11`	`+return {`
	`12`	`+getRuntimeConfig: () => ({}),`
	`13`	`+logGateway: vi.fn(),`
	`14`	`+} as unknown as GatewayRequestContext;`
	`15`	`+}`
	`16`	`+`
	`17`	`+export async function callGatewayHandler(`
	`18`	`+handlers: GatewayRequestHandlers,`
	`19`	`+method: string,`
	`20`	`+params: Record<string, unknown>,`
	`21`	`+): Promise<CapturedGatewayResponse> {`
	`22`	`+let ok: boolean \| null = null;`
	`23`	`+let response: unknown;`
	`24`	`+let error: unknown;`
	`25`	`+const handler = handlers[method];`
	`26`	`+`
	`27`	`+if (!handler) {`
	`28`	+throw new Error(`unknown gateway handler: ${method}`);
	`29`	`+}`
	`30`	`+`
	`31`	`+await handler({`
	`32`	`+ params,`
	`33`	`+req: {} as never,`
	`34`	`+client: null,`
	`35`	`+isWebchatConnect: () => false,`
	`36`	`+context: makeGatewayHandlerTestContext(),`
	`37`	`+respond: (success, result, err) => {`
	`38`	`+ok = success;`
	`39`	`+response = result;`
	`40`	`+error = err;`
	`41`	`+},`
	`42`	`+});`
	`43`	`+`
	`44`	`+return { ok, response, error };`
	`45`	`+}`

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

推荐订阅源

Recent Commits to openclaw:main