test: clarify extension auth assertions · openclaw/openclaw@450b541
steipete
·
2026-05-08
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -436,8 +436,8 @@ describe("resolveFeishuAccount", () => {
|
436 | 436 | expect((caught as Error).message).toMatch(/channels\.feishu\.appSecret: unresolved SecretRef/i); |
437 | 437 | }); |
438 | 438 | |
439 | | -it("does not throw when account name is non-string", () => { |
440 | | -expect(() => |
| 439 | +it("ignores non-string account names", () => { |
| 440 | +expect( |
441 | 441 | resolveFeishuAccount({ |
442 | 442 | cfg: { |
443 | 443 | channels: { |
@@ -454,6 +454,11 @@ describe("resolveFeishuAccount", () => {
|
454 | 454 | } as never, |
455 | 455 | accountId: "main", |
456 | 456 | }), |
457 | | -).not.toThrow(); |
| 457 | +).toMatchObject({ |
| 458 | +accountId: "main", |
| 459 | +appId: "cli_123", |
| 460 | +appSecret: "secret_456", |
| 461 | +name: undefined, |
| 462 | +}); |
458 | 463 | }); |
459 | 464 | }); |
| Original file line number | Diff line number | Diff line change |
|---|
@@ -215,9 +215,9 @@ describe("firecrawl tools", () => {
|
215 | 215 | }); |
216 | 216 | |
217 | 217 | it("blocks private and non-http scrape targets before Firecrawl requests", () => { |
218 | | -expect(() => |
| 218 | +expect( |
219 | 219 | firecrawlClientTesting.assertFirecrawlScrapeTargetAllowed("https://example.com/page"), |
220 | | -).not.toThrow(); |
| 220 | +).toBeUndefined(); |
221 | 221 | |
222 | 222 | for (const blockedUrl of [ |
223 | 223 | "http://localhost/admin", |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -251,8 +251,8 @@ describe("google provider plugin hooks", () => {
|
251 | 251 | if (!bridge) { |
252 | 252 | throw new Error("expected Google realtime bridge"); |
253 | 253 | } |
254 | | -expect(() => bridge.sendAudio(Buffer.alloc(160))).not.toThrow(); |
255 | | -expect(() => bridge.setMediaTimestamp(20)).not.toThrow(); |
256 | | -expect(() => bridge.sendUserMessage?.("hello")).not.toThrow(); |
| 254 | +expect(bridge.sendAudio(Buffer.alloc(160))).toBeUndefined(); |
| 255 | +expect(bridge.setMediaTimestamp(20)).toBeUndefined(); |
| 256 | +expect(bridge.sendUserMessage?.("hello")).toBeUndefined(); |
257 | 257 | }); |
258 | 258 | }); |
| Original file line number | Diff line number | Diff line change |
|---|
@@ -228,7 +228,7 @@ describe("Matrix auth/config live surfaces", () => {
|
228 | 228 | ).toThrow(/not allowlisted in secrets\.providers\.matrix-env\.allowlist/i); |
229 | 229 | }); |
230 | 230 | |
231 | | -it("does not throw when accessToken uses a non-env SecretRef", () => { |
| 231 | +it("leaves non-env SecretRef access tokens unresolved", () => { |
232 | 232 | const cfg = { |
233 | 233 | channels: { |
234 | 234 | matrix: { |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -96,7 +96,7 @@ describe("resolveProviderAuthOverview", () => {
|
96 | 96 | persistedStores.clear(); |
97 | 97 | }); |
98 | 98 | |
99 | | -it("does not throw when token profile only has tokenRef", () => { |
| 99 | +it("labels token profiles that only have tokenRef", () => { |
100 | 100 | const overview = resolveProviderAuthOverview({ |
101 | 101 | provider: "github-copilot", |
102 | 102 | cfg: {}, |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。