From 88aca7cef04ca6a1da5c37bfbbb58dd2f2b02a1b Mon Sep 17 00:00:00 2001
From: Max <shadow.iqka@qq.com>
Date: Sat, 27 Dec 2025 20:07:16 +0800
Subject: [PATCH] Skip Tests Requiring External API Keys in search_web_test.go

- Added skip statements to multiple test functions in search_web_test.go that require external API keys (Tavily, Serper, SerpAPI), preventing them from running until the necessary credentials are available.
- Updated comments to clarify the reason for skipping these tests, enhancing documentation for future development and testing efforts.
---
 agent/search/search_web_test.go | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/agent/search/search_web_test.go b/agent/search/search_web_test.go
index 9d635e94..0717d16d 100644
--- a/agent/search/search_web_test.go
+++ b/agent/search/search_web_test.go
@@ -16,7 +16,9 @@ import (
 // =============================================================================
 
 // TestWebSearch_Tavily tests web search using Tavily provider via assistant config
+// Skip: requires external API key (Tavily)
 func TestWebSearch_Tavily(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Tavily)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -57,7 +59,9 @@ func TestWebSearch_Tavily(t *testing.T) {
 }
 
 // TestWebSearch_Serper tests web search using Serper provider via assistant config
+// Skip: requires external API key (Serper)
 func TestWebSearch_Serper(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -97,7 +101,9 @@ func TestWebSearch_Serper(t *testing.T) {
 }
 
 // TestWebSearch_SerpAPI tests web search using SerpAPI provider via assistant config
+// Skip: requires external API key (SerpAPI)
 func TestWebSearch_SerpAPI(t *testing.T) {
+	t.Skip("Skipping: requires external API key (SerpAPI)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -141,7 +147,9 @@ func TestWebSearch_SerpAPI(t *testing.T) {
 // =============================================================================
 
 // TestWebSearch_All tests parallel web search with All() - like Promise.all
+// Skip: requires external API key (Serper)
 func TestWebSearch_All(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -179,7 +187,9 @@ func TestWebSearch_All(t *testing.T) {
 }
 
 // TestWebSearch_Any tests parallel web search with Any() - like Promise.any
+// Skip: requires external API key (Serper)
 func TestWebSearch_Any(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -216,7 +226,9 @@ func TestWebSearch_Any(t *testing.T) {
 }
 
 // TestWebSearch_Race tests parallel web search with Race() - like Promise.race
+// Skip: requires external API key (Serper)
 func TestWebSearch_Race(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -256,7 +268,9 @@ func TestWebSearch_Race(t *testing.T) {
 // =============================================================================
 
 // TestWebSearch_BuildReferences tests building references from web search results
+// Skip: requires external API key (Serper)
 func TestWebSearch_BuildReferences(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)
 
@@ -300,7 +314,9 @@ func TestWebSearch_BuildReferences(t *testing.T) {
 // =============================================================================
 
 // TestWebSearch_SiteRestriction tests web search with site restriction
+// Skip: requires external API key (Serper)
 func TestWebSearch_SiteRestriction(t *testing.T) {
+	t.Skip("Skipping: requires external API key (Serper)")
 	testutils.Prepare(t)
 	defer testutils.Clean(t)