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

fix(combo): align combo-routing tests with #9630 pre-dispatch skip contract

Browse Source 
#9630 (976d670ff3) intentionally changed the pre-dispatch skip behavior: when
recordedAttempts === 0 (all targets filtered before any dispatch), handleComboChat
now returns 503 ALL_TARGETS_SKIPPED instead of the misleading ALL_ACCOUNTS_INACTIVE.

The two combo-routing-engine tests covering the 'every target skipped before
execution' scenario still asserted the old code. Align both assertions to
ALL_TARGETS_SKIPPED (the tests still verify the 503 + meaningful error code).
This commit is contained in:
diegosouzapw
2026-08-08 08:44:52 -03:00
parent 2e1320796e
commit 5199f464c1
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/unit/combo-routing-engine.test.ts
View File

@@ -2318,7 +2318,7 @@ test("handleComboChat returns a 503 when every model is unavailable before execu
const payload = (await result.json()) as any;
assert.equal(result.status, 503);
assert.equal(payload.error.code, "ALL_ACCOUNTS_INACTIVE");
assert.equal(payload.error.code, "ALL_TARGETS_SKIPPED");
});
test("handleComboChat treats provider circuit breaker responses as ordinary target failures", async () => {
@@ -2847,7 +2847,7 @@ test("handleComboChat round-robin resolves nested combos and returns inactive wh
const payload = (await result.json()) as any;
assert.equal(result.status, 503);
assert.equal(payload.error.code, "ALL_ACCOUNTS_INACTIVE");
assert.equal(payload.error.code, "ALL_TARGETS_SKIPPED");
});
test("handleComboChat round-robin treats provider circuit breaker responses as ordinary target failures", async () => {