test: tighten agent auth assertions · openclaw/openclaw@e878597

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -751,25 +751,25 @@ describe("anthropic transport stream", () => {`
`751`	`751`	`} as AnthropicStreamOptions,`
`752`	`752`	`);`
`753`	`753`
`754`		`-expect(latestAnthropicRequest().payload).toMatchObject({`
`755`		`- model: "MiniMax-M2.7",`
`756`		`- messages: [`
`757`		`- {`
`758`		`- role: "user",`
`759`		`- content: [`
`760`		`- {`
`761`		`- type: "text",`
`762`		`- text: ".",`
`763`		`- cache_control: { type: "ephemeral" },`
`764`		`- },`
`765`		`- ],`
`766`		`- },`
`767`		`- ],`
`768`		`-});`
`769`		`-expect(guardedFetchMock).toHaveBeenCalledWith(`
`770`		`- "https://api.minimax.io/anthropic/v1/messages",`
`771`		`- expect.objectContaining({ method: "POST" }),`
`772`		`-);`
	`754`	`+const requestPayload = latestAnthropicRequest().payload;`
	`755`	`+expect(requestPayload.model).toBe("MiniMax-M2.7");`
	`756`	`+expect(requestPayload.messages).toEqual([`
	`757`	`+{`
	`758`	`+role: "user",`
	`759`	`+content: [`
	`760`	`+{`
	`761`	`+type: "text",`
	`762`	`+text: ".",`
	`763`	`+cache_control: { type: "ephemeral" },`
	`764`	`+},`
	`765`	`+],`
	`766`	`+},`
	`767`	`+]);`
	`768`	`+const [[url, fetchOptions]] = guardedFetchMock.mock.calls as unknown as Array<`
	`769`	`+ [string, { method?: string }]`
	`770`	`+>;`
	`771`	`+expect(url).toBe("https://api.minimax.io/anthropic/v1/messages");`
	`772`	`+expect(fetchOptions.method).toBe("POST");`
`773`	`773`	`},`
`774`	`774`	`);`
`775`	`775`

Original file line number	Diff line number	Diff line change
`@@ -207,21 +207,21 @@ describe("auth external oauth helpers", () => {`
`207`	`207`	`}),`
`208`	`208`	`);`
`209`	`209`
`210`		`-expect(overlaid.profiles["openai-codex:default"]).toMatchObject({`
`211`		`-access: "fresh-cli-access-token",`
`212`		`-refresh: "fresh-cli-refresh-token",`
`213`		`-accountId: "acct-cli",`
`214`		`-});`
`215`		`-expect(`
`216`		`-readManagedExternalCliCredential({`
`217`		`-profileId: "openai-codex:default",`
`218`		`-credential: tokenlessCredential,`
`219`		`-}),`
`220`		`-).toMatchObject({`
`221`		`-access: "fresh-cli-access-token",`
`222`		`-refresh: "fresh-cli-refresh-token",`
`223`		`-accountId: "acct-cli",`
	`210`	`+const overlaidProfile = overlaid.profiles["openai-codex:default"];`
	`211`	`+expect(overlaidProfile?.type).toBe("oauth");`
	`212`	`+if (!overlaidProfile \|\| overlaidProfile.type !== "oauth") {`
	`213`	`+throw new Error("expected overlaid OAuth profile");`
	`214`	`+}`
	`215`	`+expect(overlaidProfile.access).toBe("fresh-cli-access-token");`
	`216`	`+expect(overlaidProfile.refresh).toBe("fresh-cli-refresh-token");`
	`217`	`+expect(overlaidProfile.accountId).toBe("acct-cli");`
	`218`	`+const managedCredential = readManagedExternalCliCredential({`
	`219`	`+profileId: "openai-codex:default",`
	`220`	`+credential: tokenlessCredential,`
`224`	`221`	`});`
	`222`	`+expect(managedCredential?.access).toBe("fresh-cli-access-token");`
	`223`	`+expect(managedCredential?.refresh).toBe("fresh-cli-refresh-token");`
	`224`	`+expect(managedCredential?.accountId).toBe("acct-cli");`
`225`	`225`	`});`
`226`	`226`
`227`	`227`	`it("keeps healthy local oauth even when external cli has a fresher token", () => {`

Original file line number	Diff line number	Diff line change
`@@ -33,11 +33,11 @@ function expectPersistedOpenAICodexProfileWithoutInlineTokens(`
`33`	`33`	`credential: AuthProfileStore["profiles"][string],`
`34`	`34`	`metadata: Record<string, unknown> = {},`
`35`	`35`	`): void {`
`36`		`-expect(credential).toMatchObject({`
`37`		`- type: "oauth",`
`38`		`- provider: "openai-codex",`
`39`		`-...metadata,`
`40`		`-});`
	`36`	`+expect(credential?.type).toBe("oauth");`
	`37`	`+expect(credential?.provider).toBe("openai-codex");`
	`38`	`+for (const [key, value] of Object.entries(metadata)) {`
	`39`	`+expect((credential as Record<string, unknown> \| undefined)?.[key]).toEqual(value);`
	`40`	`+}`
`41`	`41`	`expect(credential).not.toHaveProperty("access");`
`42`	`42`	`expect(credential).not.toHaveProperty("refresh");`
`43`	`43`	`expect(credential).not.toHaveProperty("idToken");`

Original file line number	Diff line number	Diff line change
`@@ -32,11 +32,11 @@ function expectPersistedOpenAICodexProfileWithoutInlineTokens(`
`32`	`32`	`credential: AuthProfileStore["profiles"][string],`
`33`	`33`	`metadata: Record<string, unknown> = {},`
`34`	`34`	`): void {`
`35`		`-expect(credential).toMatchObject({`
`36`		`- type: "oauth",`
`37`		`- provider: "openai-codex",`
`38`		`-...metadata,`
`39`		`-});`
	`35`	`+expect(credential?.type).toBe("oauth");`
	`36`	`+expect(credential?.provider).toBe("openai-codex");`
	`37`	`+for (const [key, value] of Object.entries(metadata)) {`
	`38`	`+expect((credential as Record<string, unknown> \| undefined)?.[key]).toEqual(value);`
	`39`	`+}`
`40`	`40`	`expect(credential).not.toHaveProperty("access");`
`41`	`41`	`expect(credential).not.toHaveProperty("refresh");`
`42`	`42`	`expect(credential).not.toHaveProperty("idToken");`

Original file line number	Diff line number	Diff line change
`@@ -446,13 +446,12 @@ describe("resolveAuthProfileOrder", () => {`
`446`	`446`	`expect(store.lastGood).toEqual({`
`447`	`447`	`"fixture-provider": "fixture-provider:default",`
`448`	`448`	`});`
`449`		`-expect(store.usageStats?.["fixture-provider:default"]).toMatchObject({`
`450`		`-errorCount: 0,`
`451`		`-blockedUntil: undefined,`
`452`		`-blockedReason: undefined,`
`453`		`-cooldownUntil: undefined,`
`454`		`-cooldownReason: undefined,`
`455`		`-});`
	`449`	`+const usageStats = store.usageStats?.["fixture-provider:default"];`
	`450`	`+expect(usageStats?.errorCount).toBe(0);`
	`451`	`+expect(usageStats?.blockedUntil).toBeUndefined();`
	`452`	`+expect(usageStats?.blockedReason).toBeUndefined();`
	`453`	`+expect(usageStats?.cooldownUntil).toBeUndefined();`
	`454`	`+expect(usageStats?.cooldownReason).toBeUndefined();`
`456`	`455`	`const lastUsed = store.usageStats?.["fixture-provider:default"]?.lastUsed;`
`457`	`456`	`expect(typeof lastUsed).toBe("number");`
`458`	`457`	`expect(Number.isFinite(lastUsed)).toBe(true);`

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ describe("exec security floor", () => {`
`57`	`57`	`ask: "off",`
`58`	`58`	`});`
`59`	`59`
`60`		`-expect(result.content[0]).toMatchObject({ type: "text" });`
	`60`	`+expect(result.content[0]?.type).toBe("text");`
`61`	`61`	`const text = (result.content[0] as { text?: string }).text ?? "";`
`62`	`62`	`expect(text).not.toMatch(/exec denied/i);`
`63`	`63`	`expect(text).not.toMatch(/allowlist miss/i);`

Original file line number	Diff line number	Diff line change
`@@ -1419,7 +1419,9 @@ describe("resolveModel", () => {`
`1419`	`1419`	`contextWindow: 262144,`
`1420`	`1420`	`maxTokens: 65536,`
`1421`	`1421`	`});`
`1422`		`-expect(resolvedModel.compat).toMatchObject({ supportsTools: false });`
	`1422`	`+expect((resolvedModel.compat as { supportsTools?: boolean } \| undefined)?.supportsTools).toBe(`
	`1423`	`+false,`
	`1424`	`+);`
`1423`	`1425`	`});`
`1424`	`1426`
`1425`	`1427`	`it("falls back to text-only when OpenRouter API cache is empty", () => {`

Original file line number	Diff line number	Diff line change
`@@ -463,12 +463,12 @@ describe("runEmbeddedPiAgent overflow compaction trigger routing", () => {`
`463`	`463`	`});`
`464`	`464`	`const harnessParams = pluginRunAttempt.mock.calls[0]?.[0];`
`465`	`465`	`expect(harnessParams?.runtimePlan).toBe(runtimePlan);`
`466`		`-expect(mockedMarkAuthProfileSuccess).toHaveBeenCalledWith(`
`467`		`- expect.objectContaining({`
`468`		`- provider: "openai-codex",`
`469`		`- profileId: "openai-codex:work",`
`470`		`- }),`
`471`		`-);`
	`466`	`+expect(mockedMarkAuthProfileSuccess).toHaveBeenCalledTimes(1);`
	`467`	`+const [[successParams]] = mockedMarkAuthProfileSuccess.mock.calls as unknown as Array<`
	`468`	`+[{ provider?: string; profileId?: string }]`
	`469`	`+>;`
	`470`	`+expect(successParams.provider).toBe("openai-codex");`
	`471`	`+expect(successParams.profileId).toBe("openai-codex:work");`
`472`	`472`	`});`
`473`	`473`
`474`	`474`	`it("keeps auto-selected OpenAI Codex auth profiles for forced codex harness runs", async () => {`

Original file line number	Diff line number	Diff line change
`@@ -53,10 +53,12 @@ describe("sessions_yield orchestration", () => {`
`53`	`53`	`expect(isEmbeddedPiRunActive(sessionId)).toBe(false);`
`54`	`54`
`55`	`55`	`// 4. Steer would fail (message delivery must take direct path, not steer)`
`56`		`-expect(queueEmbeddedPiMessageWithOutcome(sessionId, "subagent result")).toMatchObject({`
`57`		`-queued: false,`
`58`		`-reason: "no_active_run",`
`59`		`-});`
	`56`	`+const queueResult = queueEmbeddedPiMessageWithOutcome(sessionId, "subagent result");`
	`57`	`+expect(queueResult.queued).toBe(false);`
	`58`	`+if (queueResult.queued) {`
	`59`	`+throw new Error("expected queue attempt to fail without an active run");`
	`60`	`+}`
	`61`	`+expect(queueResult.reason).toBe("no_active_run");`
`60`	`62`	`});`
`61`	`63`
`62`	`64`	`it("clientToolCalls takes precedence over yieldDetected", async () => {`

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

推荐订阅源

Recent Commits to openclaw:main