fix: validate provider retry attempts · openclaw/openclaw@714ff55
steipete
·
2026-05-29
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
|
| 1 | +import { describe, expect, it, vi } from "vitest"; |
| 2 | +import { |
| 3 | +executeProviderOperationWithRetry, |
| 4 | +resolveTransientProviderAttempts, |
| 5 | +} from "./operation-retry.js"; |
| 6 | + |
| 7 | +describe("resolveTransientProviderAttempts", () => { |
| 8 | +it("does not round malformed attempt counts", () => { |
| 9 | +expect(resolveTransientProviderAttempts({ attempts: 1.5 })).toBe(1); |
| 10 | +expect(resolveTransientProviderAttempts({ attempts: Number.NaN })).toBe(1); |
| 11 | +expect(resolveTransientProviderAttempts({ attempts: Number.POSITIVE_INFINITY })).toBe(1); |
| 12 | +expect(resolveTransientProviderAttempts({ attempts: Number.MAX_SAFE_INTEGER + 1 })).toBe(1); |
| 13 | +}); |
| 14 | + |
| 15 | +it("keeps valid attempt counts as integers", () => { |
| 16 | +expect(resolveTransientProviderAttempts({ attempts: 0 })).toBe(1); |
| 17 | +expect(resolveTransientProviderAttempts({ attempts: 3 })).toBe(3); |
| 18 | +}); |
| 19 | +}); |
| 20 | + |
| 21 | +describe("executeProviderOperationWithRetry", () => { |
| 22 | +it("does not turn fractional attempts into an extra execution", async () => { |
| 23 | +const operation = vi.fn(async () => { |
| 24 | +const error = new Error("HTTP 503"); |
| 25 | +Object.assign(error, { status: 503 }); |
| 26 | +throw error; |
| 27 | +}); |
| 28 | + |
| 29 | +await expect( |
| 30 | +executeProviderOperationWithRetry({ |
| 31 | +provider: "test", |
| 32 | +stage: "read", |
| 33 | + operation, |
| 34 | +retry: { |
| 35 | +attempts: 1.5, |
| 36 | +baseDelayMs: 0, |
| 37 | +maxDelayMs: 0, |
| 38 | +}, |
| 39 | +}), |
| 40 | +).rejects.toThrow("HTTP 503"); |
| 41 | + |
| 42 | +expect(operation).toHaveBeenCalledTimes(1); |
| 43 | +}); |
| 44 | +}); |
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。