From 8d73f553cad029a82494c5cf806d827b9bafce37 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Wed, 27 May 2026 18:10:06 -0400 Subject: [PATCH] test: drop test_tools_calendar_always_available Tested that services/tools/_registry exposed event tools to the LLM tool layer. That layer was removed in Phase 8 commit 91bafb6; event CRUD is now covered via the MCP tools in test_mcp_tool_events.py. --- tests/test_events_service.py | 18 +++--------------- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/tests/test_events_service.py b/tests/test_events_service.py index 0089bd5..8649d91 100644 --- a/tests/test_events_service.py +++ b/tests/test_events_service.py @@ -321,18 +321,6 @@ async def test_list_events_excludes_timed_event_that_already_ended(): assert results == [] -@pytest.mark.asyncio -async def test_tools_calendar_always_available(): - """Calendar tools must appear in get_tools_for_user even without CalDAV.""" - with patch("fabledassistant.services.tools._registry.is_caldav_configured", new_callable=AsyncMock) as mock_configured, \ - patch("fabledassistant.services.settings.get_setting", new_callable=AsyncMock) as mock_setting: - mock_configured.return_value = False - mock_setting.return_value = "false" - from fabledassistant.services.tools import get_tools_for_user - tools = await get_tools_for_user(user_id=1) - tool_names = {t["function"]["name"] for t in tools} - assert "create_event" in tool_names - assert "list_events" in tool_names - assert "search_events" in tool_names - assert "update_event" in tool_names - assert "delete_event" in tool_names +# test_tools_calendar_always_available removed in Phase 8 along with the +# services/tools/ LLM-tool layer. Event CRUD is now exposed via MCP tools +# (see tests/test_mcp_tool_events.py for coverage).