pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-08-13 10:43:43 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(sse): update stale ALL_ACCOUNTS_INACTIVE test assertion (T24)

Browse Source 
Pre-existing upstream test-drift, same as fix/basered-changelog-integrity-fabricated-docs@58ab721fe: t23-t24-fallback-resilience.test.ts's T24 case asserts the pre-dispatch-skip scenario returns ALL_ACCOUNTS_INACTIVE, but production code returns the more precise ALL_TARGETS_SKIPPED when recordedAttempts === 0.
This commit is contained in:
Will Gordon
2026-08-07 16:14:18 -04:00
parent 86636e3ff3
commit a32aed7383
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tests/unit/t23-t24-fallback-resilience.test.ts
View File

@@ -148,7 +148,11 @@ test("T24: all inactive accounts return 503 service_unavailable (not 406)", asyn
assert.equal(result.status, 503); assert.equal(result.status, 503);
const body = (await result.json()) as any; const body = (await result.json()) as any;
assert.equal(body.error?.code, "ALL_ACCOUNTS_INACTIVE"); // isModelAvailable always false means every target is skipped by the
// pre-dispatch filter with zero dispatch attempts — the more precise
// ALL_TARGETS_SKIPPED classification, not ALL_ACCOUNTS_INACTIVE (which
// implies targets were attempted and their accounts found inactive).
assert.equal(body.error?.code, "ALL_TARGETS_SKIPPED");
}); });
test("combo falls through 400s and reaches the next model", async () => { test("combo falls through 400s and reaches the next model", async () => {