fix(provider): omit default model test temperature

This commit is contained in:
elky
2026-08-12 16:56:18 +08:00
parent 29fa4aed19
commit edb8362adc
5 changed files with 21 additions and 6 deletions
@@ -635,7 +635,6 @@ fn provider_query_build_test_request_body_for_api_format_with_search_session(
"model": model, "model": model,
"input": message, "input": message,
"max_output_tokens": 30, "max_output_tokens": 30,
"temperature": 0.7,
"stream": true, "stream": true,
}), }),
"openai:search" => json!({ "openai:search" => json!({
@@ -654,7 +653,6 @@ fn provider_query_build_test_request_body_for_api_format_with_search_session(
"content": message "content": message
}], }],
"max_tokens": 30, "max_tokens": 30,
"temperature": 0.7,
"stream": true, "stream": true,
}), }),
_ => json!({ _ => json!({
@@ -664,7 +662,6 @@ fn provider_query_build_test_request_body_for_api_format_with_search_session(
"content": message "content": message
}], }],
"max_tokens": 30, "max_tokens": 30,
"temperature": 0.7,
"stream": true, "stream": true,
}), }),
} }
@@ -811,7 +808,6 @@ fn provider_query_build_test_request_body_with_model_policy(
"content": provider_query_extract_message(payload) "content": provider_query_extract_message(payload)
.unwrap_or_else(|| DEFAULT_PROVIDER_QUERY_TEST_MESSAGE.to_string()) .unwrap_or_else(|| DEFAULT_PROVIDER_QUERY_TEST_MESSAGE.to_string())
}], }],
"temperature": 0.7,
"stream": true, "stream": true,
}) })
} }
@@ -242,6 +242,26 @@ fn provider_query_default_test_request_body_does_not_set_max_tokens() {
); );
} }
#[test]
fn provider_query_default_test_request_bodies_do_not_set_temperature() {
let payload = json!({});
let default_body = provider_query_build_test_request_body(&payload, "fallback-model");
assert!(default_body.get("temperature").is_none());
for api_format in ["openai:chat", "openai:responses", "claude:messages"] {
let body = provider_query_build_test_request_body_for_api_format(
&payload,
"fallback-model",
"/api/admin/provider-query/test-model",
api_format,
);
assert!(
body.get("temperature").is_none(),
"admin model test must not set temperature for {api_format}"
);
}
}
#[test] #[test]
fn provider_query_failover_request_body_overrides_custom_model() { fn provider_query_failover_request_body_overrides_custom_model() {
let payload = json!({ let payload = json!({
@@ -2825,7 +2825,6 @@ async fn gateway_streams_windsurf_connect_upstream_for_admin_model_test_impl() {
"content": "Hello! This is a test message." "content": "Hello! This is a test message."
}], }],
"max_tokens": 30, "max_tokens": 30,
"temperature": 0.7,
"stream": true "stream": true
} }
})) }))
@@ -71,6 +71,7 @@ describe('buildDefaultModelTestRequestBody', () => {
expect(body.messages).toEqual([{ role: 'user', content: 'Hello! This is a test message.' }]) expect(body.messages).toEqual([{ role: 'user', content: 'Hello! This is a test message.' }])
expect(body.stream).toBe(true) expect(body.stream).toBe(true)
expect(body.temperature).toBeUndefined()
expect(body.input).toBeUndefined() expect(body.input).toBeUndefined()
}) })
@@ -257,7 +257,6 @@ export function buildDefaultModelTestRequestBody(
}, },
], ],
max_tokens: 30, max_tokens: 30,
temperature: 0.7,
stream: true, stream: true,
}, null, 2) }, null, 2)
} }