test: guard provider helper assertions · openclaw/openclaw@864f30f
steipete
·
2026-05-12
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -31,9 +31,9 @@ function mockSingleActiveSummary(overrides: Partial<typeof baseActiveAnthropicSu
|
31 | 31 | } |
32 | 32 | |
33 | 33 | function expectModelFields(model: unknown, expected: Record<string, unknown>): void { |
34 | | -expect(model).toBeDefined(); |
35 | | -expect(typeof model).toBe("object"); |
36 | | -expect(model).not.toBeNull(); |
| 34 | +if (!model || typeof model !== "object") { |
| 35 | + throw new Error("Expected model record"); |
| 36 | +} |
37 | 37 | const actual = model as Record<string, unknown>; |
38 | 38 | for (const [key, value] of Object.entries(expected)) { |
39 | 39 | expect(actual[key]).toEqual(value); |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -56,9 +56,9 @@ function installGoogleTtsRequestMock(pcm = Buffer.from([1, 0, 2, 0])) {
|
56 | 56 | } |
57 | 57 | |
58 | 58 | function expectRecordFields(value: unknown, expected: Record<string, unknown>) { |
59 | | -expect(value).toBeDefined(); |
60 | | -expect(typeof value).toBe("object"); |
61 | | -expect(value).not.toBeNull(); |
| 59 | +if (!value || typeof value !== "object") { |
| 60 | + throw new Error("Expected record"); |
| 61 | +} |
62 | 62 | const actual = value as Record<string, unknown>; |
63 | 63 | for (const [key, expectedValue] of Object.entries(expected)) { |
64 | 64 | expect(actual[key]).toEqual(expectedValue); |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -83,7 +83,9 @@ function expectBaseStreamModelFields(baseStream: StreamFn, fields: Record<string
|
83 | 83 | "base stream", |
84 | 84 | ); |
85 | 85 | expectRecordFields(requireRecord(call[0], "base stream model"), fields); |
86 | | -expect(call[1]).toBeDefined(); |
| 86 | +if (call[1] === undefined) { |
| 87 | +throw new Error("Expected base stream context"); |
| 88 | +} |
87 | 89 | expect(call[2]).toBeUndefined(); |
88 | 90 | } |
89 | 91 | |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -24,19 +24,23 @@ function expectReplayPolicyFields(
|
24 | 24 | policy: Record<string, unknown> | undefined, |
25 | 25 | fields: Record<string, unknown>, |
26 | 26 | ): void { |
27 | | -expect(policy).toBeDefined(); |
| 27 | +if (!policy) { |
| 28 | +throw new Error("Expected replay policy"); |
| 29 | +} |
28 | 30 | for (const [key, value] of Object.entries(fields)) { |
29 | | -expect(policy?.[key]).toEqual(value); |
| 31 | +expect(policy[key]).toEqual(value); |
30 | 32 | } |
31 | 33 | } |
32 | 34 | |
33 | 35 | function expectModelFields( |
34 | 36 | model: Record<string, unknown> | undefined, |
35 | 37 | fields: Record<string, unknown>, |
36 | 38 | ): void { |
37 | | -expect(model).toBeDefined(); |
| 39 | +if (!model) { |
| 40 | +throw new Error("Expected provider model"); |
| 41 | +} |
38 | 42 | for (const [key, value] of Object.entries(fields)) { |
39 | | -expect(model?.[key]).toEqual(value); |
| 43 | +expect(model[key]).toEqual(value); |
40 | 44 | } |
41 | 45 | } |
42 | 46 | |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。