From c3967a560d8c994ebca9e1d895aab3b00627c441 Mon Sep 17 00:00:00 2001 From: PhotoPortfolio Developer Date: Thu, 19 Feb 2026 22:17:01 +0800 Subject: [PATCH] Fix web config and search tool tests --- pkg/config/config_test.go | 12 ++++++------ pkg/tools/web_test.go | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/pkg/config/config_test.go b/pkg/config/config_test.go index 47916d155..c65f8d94d 100644 --- a/pkg/config/config_test.go +++ b/pkg/config/config_test.go @@ -289,14 +289,14 @@ func TestDefaultConfig_WebTools(t *testing.T) { cfg := DefaultConfig() // Verify web tools defaults - if cfg.Tools.Web.Brave.MaxResults != 5 { - t.Error("Expected Brave MaxResults 5, got ", cfg.Tools.Web.Brave.MaxResults) + if cfg.Tools.Web.Search.MaxResults != 5 { + t.Error("Expected Search MaxResults 5, got ", cfg.Tools.Web.Search.MaxResults) } - if cfg.Tools.Web.Brave.APIKey != "" { - t.Error("Brave API key should be empty by default") + if cfg.Tools.Web.Search.Provider == "" { + t.Error("Search provider should not be empty by default") } - if cfg.Tools.Web.DuckDuckGo.MaxResults != 5 { - t.Error("Expected DuckDuckGo MaxResults 5, got ", cfg.Tools.Web.DuckDuckGo.MaxResults) + if cfg.Tools.Web.Search.QueryParam == "" { + t.Error("Search query param should not be empty by default") } } diff --git a/pkg/tools/web_test.go b/pkg/tools/web_test.go index a420d543d..bb9ff0561 100644 --- a/pkg/tools/web_test.go +++ b/pkg/tools/web_test.go @@ -190,7 +190,7 @@ func TestWebTool_WebSearch_NoApiKey(t *testing.T) { // TestWebTool_WebSearch_MissingQuery verifies error handling for missing query func TestWebTool_WebSearch_MissingQuery(t *testing.T) { - tool := NewWebSearchTool(WebSearchToolOptions{Provider: "duckduckgo", Enabled: true, MaxResults: 5}) + tool := NewWebSearchTool(WebSearchToolOptions{Provider: "duckduckgo", MaxResults: 5}) ctx := context.Background() args := map[string]interface{}{}