From 0e3eb1bf6bf096941d65fc197493dff65e7cc719 Mon Sep 17 00:00:00 2001 From: Will Gordon Date: Fri, 7 Aug 2026 15:50:53 -0400 Subject: [PATCH] fix(sse): update stale ALL_ACCOUNTS_INACTIVE test assertions to ALL_TARGETS_SKIPPED Two combo-routing-engine.test.ts cases assert the pre-dispatch-skip scenario (isModelAvailable always false, zero dispatch attempts) returns ALL_ACCOUNTS_INACTIVE. Production code already distinguishes this case via the recordedAttempts === 0 branch and returns the more precise ALL_TARGETS_SKIPPED -- the tests were never updated when that branch shipped upstream, so they fail on a clean release/v3.8.50 checkout independent of this PR's changes. --- tests/unit/combo-routing-engine.test.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tests/unit/combo-routing-engine.test.ts b/tests/unit/combo-routing-engine.test.ts index e3f71e8052..a13cbb1d7b 100644 --- a/tests/unit/combo-routing-engine.test.ts +++ b/tests/unit/combo-routing-engine.test.ts @@ -2318,6 +2318,10 @@ test("handleComboChat returns a 503 when every model is unavailable before execu const payload = (await result.json()) as any; assert.equal(result.status, 503); + // 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(payload.error.code, "ALL_TARGETS_SKIPPED"); }); @@ -2847,6 +2851,9 @@ test("handleComboChat round-robin resolves nested combos and returns inactive wh const payload = (await result.json()) as any; assert.equal(result.status, 503); + // isModelAvailable always false means every nested target is skipped by the + // pre-dispatch filter with zero dispatch attempts — ALL_TARGETS_SKIPPED, + // not ALL_ACCOUNTS_INACTIVE (see the analogous priority-strategy test above). assert.equal(payload.error.code, "ALL_TARGETS_SKIPPED"); });