From 5cc4086c4ff3add09bd071c3f473b4c892c65d3b Mon Sep 17 00:00:00 2001
From: openhands <openhands@all-hands.dev>
Date: Tue, 22 Apr 2025 14:14:11 +0000
Subject: [PATCH] Add BasicTool to the list of classes to skip in
 test_make_tool_all_tools

---
 test/serena/test_mcp.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/serena/test_mcp.py b/test/serena/test_mcp.py
index 31182f0c..5998d23e 100644
--- a/test/serena/test_mcp.py
+++ b/test/serena/test_mcp.py
@@ -246,7 +246,7 @@ def test_make_tool_all_tools() -> None:
                 continue
                 
             # Skip test mock classes that don't properly implement __init__
-            if tool_class.__name__ in ["BaseMockTool", "BadTool", "NoParamsTool", "NoReturnTool", 
+            if tool_class.__name__ in ["BaseMockTool", "BasicTool", "BadTool", "NoParamsTool", "NoReturnTool", 
                                       "MissingParamTool", "ComplexDocTool", "FormatTool", "NoDescriptionTool"]:
                 continue