mirror of
https://github.com/PrefectHQ/fastmcp.git
synced 2026-08-18 19:44:19 +02:00
96c7a74b27* Move OpenAPI to providers/openapi/ submodule Migrates the OpenAPI implementation to the provider pattern: - Creates src/fastmcp/server/providers/openapi/ submodule with provider.py, components.py, and routing.py - Converts server/openapi/ to deprecated re-export stubs - FastMCPOpenAPI remains as deprecated wrapper that uses OpenAPIProvider - Updates experimental/server/openapi to import from new canonical location * Update tests and FastMCP methods to use OpenAPIProvider - Update FastMCP.from_openapi() and from_fastapi() to use OpenAPIProvider directly, returning FastMCP instead of deprecated FastMCPOpenAPI - Move tests from tests/server/openapi/ to tests/server/providers/openapi/ - Update all test imports to use new canonical location - Add tests/deprecated/openapi/ for backward compatibility testing - Update tests/client/test_openapi.py to use new imports * Fix deprecation warning tests and address CodeRabbit feedback - Use importlib.reload() to ensure deprecation warnings fire in tests - Change logger.error to logger.exception for better stack traces
78 lines
3 KiB
Python
78 lines
3 KiB
Python
"""Tests for OpenAPI-related deprecations in 2.14."""
|
|
|
|
import importlib
|
|
import warnings
|
|
|
|
import pytest
|
|
|
|
import fastmcp
|
|
|
|
pytestmark = pytest.mark.filterwarnings("default::DeprecationWarning")
|
|
|
|
|
|
class TestEnableNewOpenAPIParserDeprecation:
|
|
"""Test enable_new_openapi_parser setting deprecation."""
|
|
|
|
def test_setting_true_emits_warning(self):
|
|
"""Setting enable_new_openapi_parser=True should emit deprecation warning."""
|
|
with pytest.warns(
|
|
DeprecationWarning,
|
|
match=r"enable_new_openapi_parser is deprecated.*now the default",
|
|
):
|
|
fastmcp.settings.experimental.enable_new_openapi_parser = True
|
|
|
|
def test_setting_false_no_warning(self):
|
|
"""Setting enable_new_openapi_parser=False should not emit warning."""
|
|
with warnings.catch_warnings(record=True) as recorded:
|
|
warnings.simplefilter("always")
|
|
fastmcp.settings.experimental.enable_new_openapi_parser = False
|
|
|
|
deprecation_warnings = [
|
|
w for w in recorded if issubclass(w.category, DeprecationWarning)
|
|
]
|
|
assert len(deprecation_warnings) == 0
|
|
|
|
|
|
class TestExperimentalOpenAPIImportDeprecation:
|
|
"""Test experimental OpenAPI import path deprecations."""
|
|
|
|
def test_experimental_server_openapi_import_warns(self):
|
|
"""Importing from fastmcp.experimental.server.openapi should warn."""
|
|
import fastmcp.experimental.server.openapi
|
|
|
|
with pytest.warns(
|
|
DeprecationWarning,
|
|
match=r"Importing from fastmcp\.experimental\.server\.openapi is deprecated",
|
|
):
|
|
importlib.reload(fastmcp.experimental.server.openapi)
|
|
|
|
def test_experimental_utilities_openapi_import_warns(self):
|
|
"""Importing from fastmcp.experimental.utilities.openapi should warn."""
|
|
import fastmcp.experimental.utilities.openapi
|
|
|
|
with pytest.warns(
|
|
DeprecationWarning,
|
|
match=r"Importing from fastmcp\.experimental\.utilities\.openapi is deprecated",
|
|
):
|
|
importlib.reload(fastmcp.experimental.utilities.openapi)
|
|
|
|
def test_experimental_imports_resolve_to_same_classes(self):
|
|
"""Experimental imports should resolve to the same classes as main imports."""
|
|
with warnings.catch_warnings():
|
|
warnings.simplefilter("ignore")
|
|
|
|
from fastmcp.experimental.server.openapi import (
|
|
FastMCPOpenAPI as ExpFastMCPOpenAPI,
|
|
)
|
|
from fastmcp.experimental.server.openapi import MCPType as ExpMCPType
|
|
from fastmcp.experimental.server.openapi import RouteMap as ExpRouteMap
|
|
from fastmcp.experimental.utilities.openapi import (
|
|
HTTPRoute as ExpHTTPRoute,
|
|
)
|
|
from fastmcp.server.openapi import FastMCPOpenAPI, MCPType, RouteMap
|
|
from fastmcp.utilities.openapi import HTTPRoute
|
|
|
|
assert FastMCPOpenAPI is ExpFastMCPOpenAPI
|
|
assert RouteMap is ExpRouteMap
|
|
assert MCPType is ExpMCPType
|
|
assert HTTPRoute is ExpHTTPRoute
|