From 0391aaa8eb73094855b89d2f6f61ed3868c3fbf8 Mon Sep 17 00:00:00 2001 From: Omar Abdelwahab Date: Thu, 13 Nov 2025 19:07:37 -0800 Subject: [PATCH] test: Remove skip marker from MCP authentication tests These tests use local in-process MCP servers and don't require external API calls or recordings. They can run in both replay and record modes without issues since they don't depend on pre-recorded API responses. --- tests/integration/responses/test_mcp_authentication.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tests/integration/responses/test_mcp_authentication.py b/tests/integration/responses/test_mcp_authentication.py index e61c7983f..4d469a6a5 100644 --- a/tests/integration/responses/test_mcp_authentication.py +++ b/tests/integration/responses/test_mcp_authentication.py @@ -4,8 +4,6 @@ # This source code is licensed under the terms described in the LICENSE file in # the root directory of this source tree. -import os - import pytest from llama_stack.core.library_client import LlamaStackAsLibraryClient @@ -13,12 +11,6 @@ from tests.common.mcp import make_mcp_server from .helpers import setup_mcp_tools -# Skip these tests in replay mode until recordings are generated -pytestmark = pytest.mark.skipif( - os.environ.get("LLAMA_STACK_TEST_INFERENCE_MODE") == "replay", - reason="No recordings yet for authorization tests. Run with --inference-mode=record-if-missing to generate.", -) - def test_mcp_authorization_bearer(compat_client, text_model_id): """Test that bearer authorization is correctly applied to MCP requests."""