From f3e60f994dbb2a2a390ccd543c04151c26974b95 Mon Sep 17 00:00:00 2001 From: hugocasa Date: Mon, 14 Sep 2026 10:43:52 +0200 Subject: [PATCH] test: pin that an mcp summary is rejected against a name that is not Co-Authored-By: Claude Opus 5 (1M context) --- backend/windmill-worker/src/ai_executor.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/backend/windmill-worker/src/ai_executor.rs b/backend/windmill-worker/src/ai_executor.rs index 0ee9226347..f00bac17c5 100644 --- a/backend/windmill-worker/src/ai_executor.rs +++ b/backend/windmill-worker/src/ai_executor.rs @@ -2008,9 +2008,12 @@ mod tests { ["github"] ); assert!(unmatched_enabled_tools_message(&named_server, &["u/test/gh"]).is_none()); - assert!(narrow_roster(roster(), Some(&["github".to_string()])) - .iter() - .all(|t| !matches!(t.value, ToolValue::Mcp(_)))); + // Alongside a name that does match, so the summary being rejected is what empties it. + let summary_and_tool = ["get_user".to_string(), "github".to_string()]; + assert_eq!( + names(&narrow_roster(roster(), Some(&summary_and_tool))), + ["get_user"] + ); assert!(narrow_roster(roster(), Some(&["u/test/other".to_string()])).is_empty()); }