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: Remove xfail marker from test_mcp_tools_in_inference

Browse source 
The register_tool_group() issue was due to a temporary bug in llama-stack-client-python that has been resolved. The test should now pass without issues.
This commit is contained in:
Omar Abdelwahab 2025-11-13 17:21:34 -08:00
parent 42d5547047
commit fa8d3f9ca2
1 changed files with 0 additions and 4 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,10 +174,6 @@ 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):