Fix unit tests

src/backend/tests/unit/config/test_deployments.py

@@ -1,13 +1,6 @@
-from unittest.mock import Mock
+from backend.config.tools import Tool
 
-from backend.config.deployments import (
-    get_default_deployment,
-)
-from backend.tests.unit.model_deployments.mock_deployments.mock_cohere_platform import (
-    MockCohereDeployment,
-)
 
-
-def test_get_default_deployment(mock_available_model_deployments: Mock) -> None:
-    default_deployment = get_default_deployment()
-    assert isinstance(default_deployment, MockCohereDeployment)
+def test_all_tools_have_id() -> None:
+    for tool in Tool:
+        assert tool.value.ID is not None

src/backend/tests/unit/crud/test_agent_tool_metadata.py

@@ -147,10 +147,12 @@ def test_get_all_agent_tool_metadata_by_agent_id(session, user):
         artifacts=[mock_artifact_1, mock_artifact_2],
     )
 
+
     # Constraint was added preventing multiple entries for the same user + agent + tool so fixing to change the tool used
     i = 0
     for tool in Tool:
         i += 1
+
         _ = get_factory("Agent", session).create(user_id=user.id)
         _ = get_factory("AgentToolMetadata", session).create(
             id=f"{i}",

src/backend/tools/slack/tool.py

@@ -17,7 +17,7 @@ class SlackTool(BaseTool):
     Tool that searches Slack for messages and files based on a query.
     """
 
-    NAME = SLACK_TOOL_ID
+    ID = SLACK_TOOL_ID
     CLIENT_ID = ""
     CLIENT_SECRET = ""