test: fix tests after adding supports_hot_reload property to PythonAgentTool

- Update test_supports_hot_reload to expect True (PythonAgentTool now supports hot reload)
- Fix test_register_tool_duplicate_name_without_hot_reload to use mocked tools without hot reload support
- Add missing patch import in test_registry.py

Author: cagataycali

tests/strands/tools/test_registry.py

@@ -124,8 +124,16 @@ def function() -> str:
 
 def test_register_tool_duplicate_name_without_hot_reload():
     """Test that registering a tool with duplicate name raises ValueError when hot reload is not supported."""
-    tool_1 = PythonAgentTool(tool_name="duplicate_tool", tool_spec=MagicMock(), tool_func=lambda: None)
-    tool_2 = PythonAgentTool(tool_name="duplicate_tool", tool_spec=MagicMock(), tool_func=lambda: None)
+    # Create mock tools that don't support hot reload
+    tool_1 = MagicMock()
+    tool_1.tool_name = "duplicate_tool"
+    tool_1.supports_hot_reload = False
+    tool_1.is_dynamic = False
+
+    tool_2 = MagicMock()
+    tool_2.tool_name = "duplicate_tool"
+    tool_2.supports_hot_reload = False
+    tool_2.is_dynamic = False
 
     tool_registry = ToolRegistry()
     tool_registry.register_tool(tool_1)

tests/strands/tools/test_tools.py

@@ -487,7 +487,7 @@ def test_tool_type(identity_tool):
 
 @pytest.mark.parametrize("identity_tool", ["identity_invoke", "identity_invoke_async"], indirect=True)
 def test_supports_hot_reload(identity_tool):
-    assert not identity_tool.supports_hot_reload
+    assert identity_tool.supports_hot_reload
 
 
 @pytest.mark.parametrize("identity_tool", ["identity_invoke", "identity_invoke_async"], indirect=True)