From 1d5ca910c0abb6d4eef2851baaf0b91ae2b10468 Mon Sep 17 00:00:00 2001 From: mlsmaycon Date: Tue, 11 Aug 2026 13:23:10 +0000 Subject: [PATCH] [proxy] Name what the warm-probe assertions are pinning The two assertions in the HEAD subtest carried no failure message, unlike the rest of the file. Say which half failed. --- .../middleware/builtin/llm_router/middleware_test.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/proxy/internal/middleware/builtin/llm_router/middleware_test.go b/proxy/internal/middleware/builtin/llm_router/middleware_test.go index a0ce265f5..7a9951a33 100644 --- a/proxy/internal/middleware/builtin/llm_router/middleware_test.go +++ b/proxy/internal/middleware/builtin/llm_router/middleware_test.go @@ -1079,9 +1079,11 @@ func TestRouter_NonInferenceRequiresReadMethod(t *testing.T) { out, err := mw.Invoke(context.Background(), in) require.NoError(t, err) - assert.Equal(t, middleware.DecisionAllow, out.Decision) + assert.Equal(t, middleware.DecisionAllow, out.Decision, + "the HEAD warm probe must still reach the upstream") nonInference, _ := metaValue(t, out.Metadata, middleware.KeyLLMNonInference) - assert.Equal(t, "true", nonInference) + assert.Equal(t, "true", nonInference, + "the HEAD warm probe carries no model to meter") }) }