ensure tests will pass if no mcp dependencies are installed, install the dependencies in ci so the tests will run there

Author: bcallender

src/fenic/core/mcp/types.py

@@ -87,10 +87,10 @@ class BoundToolParam:
     data_type: DataType
     required: bool
     has_default: bool
-    default_value: Optional[ToolParameterType]
-    allowed_values: Optional[List[ToolParameterType]]
-    constraints: Optional[ToolParamConstraints]
-    validators: List[ParamValidator]
+    default_value: Optional[ToolParameterType] = None
+    allowed_values: Optional[List[ToolParameterType]] = None
+    constraints: Optional[ToolParamConstraints] = None
+    validators: List[ParamValidator] = Field(default_factory=list)
 
 
 @dataclass(config=ConfigDict(arbitrary_types_allowed=True))

tests/api/mcp/test_server.py

@@ -2,6 +2,8 @@
 
 import pytest
 
+pytest.importorskip("fastmcp")
+
 from fenic.api.functions import col, tool_param
 from fenic.api.mcp.server import create_mcp_server
 from fenic.api.mcp.tool_generation import auto_generate_system_tools_from_tables
@@ -14,7 +16,6 @@
 
 
 def test_server_generation_with_parameterized_tools(local_session: Session):
-    pytest.importorskip("fastmcp")
     df = local_session.create_dataframe({"city": ["SF"], "age": [10], "user_name": ["Alice"]})
     query = df.filter(
         (col("city") == tool_param("city_name", StringType))
@@ -56,7 +57,6 @@ def test_server_generation_with_parameterized_tools(local_session: Session):
     assert user_names_param['description'] == "User names"
 
 def test_server_generation(local_session: Session):
-    pytest.importorskip("fastmcp")
     create_table_with_rows(local_session, "t1", [1, 2, 3], description="table one")
     create_table_with_rows(local_session, "t2", [10, 20], description="table two")
 

tests/core/mcp/test_binder.py

@@ -80,7 +80,6 @@ def test_bind_parameters_type_mismatch_raises(local_session: Session):
                     required=True,
                     has_default=False,
                     allowed_values=None,
-                    default_value=None,
                 )
             ],
         )