From 793b93b74d691a1ebebc0b480ff538a663ab05d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E9=9B=B7=E7=94=B5=E8=8A=BD=E8=A1=A3?= Date: Tue, 29 Sep 2026 16:53:07 -0400 Subject: [PATCH] test(subagent): the auth-scheme test joins the existing tests module --- .../kigi-shell/src/agent/subagent/mod.rs | 31 +------------------ .../src/agent/subagent/tests/mod.rs | 21 +++++++++++++ 2 files changed, 22 insertions(+), 30 deletions(-) diff --git a/crates/codegen/kigi-shell/src/agent/subagent/mod.rs b/crates/codegen/kigi-shell/src/agent/subagent/mod.rs index ba97f50..62df3b1 100644 --- a/crates/codegen/kigi-shell/src/agent/subagent/mod.rs +++ b/crates/codegen/kigi-shell/src/agent/subagent/mod.rs @@ -879,7 +879,7 @@ fn log_subagent_model_resolution( )), ); } -/// The parent model's auth scheme: its live catalog entry, else a disk lookup. +/// The parent model's auth scheme from the live catalog. fn parent_auth_scheme( available: &indexmap::IndexMap, catalog_id: &acp::ModelId, @@ -2671,32 +2671,3 @@ pub(crate) fn reconcile_orphaned_subagents( } #[cfg(test)] mod tests; - -#[cfg(test)] -mod parent_auth_scheme_tests { - use super::*; - - /// A fetched Messages model exists only in the live catalog. - #[test] - fn a_fetched_entry_gives_its_own_scheme_to_the_child() { - let wire: kigi_models::WireModel = - serde_json::from_value(serde_json::json!({ "id": "claude-x" })).unwrap(); - let cfg = crate::agent::models_fetch::wire_model_to_entry( - "gw/claude-x".into(), - kigi_models::PlatformWireApi::Messages, - kigi_models::PlatformKeyHeader::XApiKey, - None, - true, - wire, - "https://gw.example/v1", - ); - let available = indexmap::IndexMap::from([( - "gw/claude-x".to_owned(), - crate::agent::config::ModelEntry::from_config_entry(&cfg), - )]); - - let scheme = parent_auth_scheme(&available, &acp::ModelId::new("gw/claude-x"), "claude-x"); - - assert_eq!(scheme, kigi_sampler::AuthScheme::XApiKey); - } -} diff --git a/crates/codegen/kigi-shell/src/agent/subagent/tests/mod.rs b/crates/codegen/kigi-shell/src/agent/subagent/tests/mod.rs index 25d29a2..660ef75 100644 --- a/crates/codegen/kigi-shell/src/agent/subagent/tests/mod.rs +++ b/crates/codegen/kigi-shell/src/agent/subagent/tests/mod.rs @@ -3322,3 +3322,24 @@ async fn parent_ack_is_bounded_when_the_parent_never_answers() { .expect("parent acks must complete in bounded time"); assert!(!acked, "an unanswered ack must report a miss"); } +/// A fetched Messages model exists only in the live catalog. +#[test] +fn a_fetched_entry_gives_its_own_scheme_to_the_child() { + let wire: kigi_models::WireModel = + serde_json::from_value(serde_json::json!({ "id": "claude-x" })).unwrap(); + let cfg = crate::agent::models_fetch::wire_model_to_entry( + "gw/claude-x".into(), + kigi_models::PlatformWireApi::Messages, + kigi_models::PlatformKeyHeader::XApiKey, + None, + true, + wire, + "https://gw.example/v1", + ); + let available = indexmap::IndexMap::from([( + "gw/claude-x".to_owned(), + crate::agent::config::ModelEntry::from_config_entry(&cfg), + )]); + let scheme = parent_auth_scheme(&available, &acp::ModelId::new("gw/claude-x"), "claude-x"); + assert_eq!(scheme, kigi_sampler::AuthScheme::XApiKey); +}