phoenix-oss/llama-stack-mirror
0
0
Fork 1
mirror of https://github.com/meta-llama/llama-stack.git synced 2025-12-03 09:53:45 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

test: Mark test_mcp_tools_in_inference as xfail due to deprecated registration API

Browse source 
The test requires register_tool_group() which is deprecated. The new approach
is configuration-based registration in run.yaml files under registered_resources.tool_groups.

Example NEW approach:
  registered_resources:
    tool_groups:
      - toolgroup_id: mcp::calculator
        provider_id: model-context-protocol
        mcp_endpoint:
          uri: http://localhost:3000/sse

The old dynamic registration API (register_tool_group) is marked deprecated with
no runtime replacement yet. Test should be updated to use config-based approach.
This commit is contained in:
Omar Abdelwahab 2025-11-13 16:00:00 -08:00
parent b5395fa5d3
commit 42d5547047
1 changed files with 4 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
tests/integration/inference/test_tools_with_schemas.py
View file

@ -174,6 +174,10 @@ class TestMCPToolsInChatCompletion:
with make_mcp_server(required_auth_token=AUTH_TOKEN, tools={"calculate": calculate}) as server: with make_mcp_server(required_auth_token=AUTH_TOKEN, tools={"calculate": calculate}) as server:
yield server yield server
@pytest.mark.xfail(
reason="Test uses deprecated register_tool_group() API. The new approach is configuration-based registration "
"in run.yaml under registered_resources.tool_groups. Test should be updated to use config-based approach."
)
def test_mcp_tools_in_inference(self, llama_stack_client, text_model_id, mcp_with_schemas): def test_mcp_tools_in_inference(self, llama_stack_client, text_model_id, mcp_with_schemas):
"""Test that MCP tools can be used in inference.""" """Test that MCP tools can be used in inference."""
if not isinstance(llama_stack_client, LlamaStackAsLibraryClient): if not isinstance(llama_stack_client, LlamaStackAsLibraryClient):