From 3d0234978379568b931193f9db0217176e9666cd Mon Sep 17 00:00:00 2001 From: Omar Abdelwahab Date: Thu, 13 Nov 2025 19:58:31 -0800 Subject: [PATCH] test: Keep skip marker for MCP auth tests (recordings needed) After attempting local recording generation, encountered multiple environment issues: 1. Client/server version mismatches (0.3.x vs 0.4.0.dev0) 2. LlamaStackClient API changes (provider_data parameter removed) 3. Dev server network constraints (HTTP 426 errors with OpenAI API) Server logs from CI confirmed recordings are needed: - RuntimeError: Recording not found for request hash: 56ddb450d... - Tests with authorization parameter create different OpenAI request hashes Local recording generation requires complex environment setup that matches CI. Requesting reviewer assistance to generate recordings via CI infrastructure. --- tests/integration/responses/test_mcp_authentication.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/integration/responses/test_mcp_authentication.py b/tests/integration/responses/test_mcp_authentication.py index 5d79dde32..7140ff44f 100644 --- a/tests/integration/responses/test_mcp_authentication.py +++ b/tests/integration/responses/test_mcp_authentication.py @@ -14,11 +14,12 @@ from .helpers import setup_mcp_tools # Skip these tests in replay mode until recordings are generated # The authorization parameter creates different request hashes than existing MCP tests +# Recordings generation requires properly configured test environment with OpenAI API access pytestmark = pytest.mark.skipif( os.environ.get("LLAMA_STACK_TEST_INFERENCE_MODE") == "replay", reason="No recordings yet for MCP authorization tests. These tests use the authorization parameter " "which creates different OpenAI request hashes than existing MCP tool tests. " - "Recordings can be generated in CI with record mode, or by running locally with OpenAI API key.", + "Recordings need to be generated in CI with proper environment configuration.", )