jerry/OpenBB
1
0
Fork 0
mirror of https://github.com/OpenBB-finance/OpenBB.git synced 2026-10-28 04:52:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

add missing docstring for the list_categories tool and add pytest skip markers on unix-specific path handling tests (#7348)

Browse Source
This commit is contained in:
Danglewood
2026-02-17 08:24:39 -08:00
committed by GitHub
parent 6b12c47f0a
commit 074ba784a4
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
openbb_platform/extensions/mcp_server/openbb_mcp_server/app/app.py
View File

@@ -467,6 +467,7 @@ def create_mcp_server(
@mcp.tool(tags={"admin"})
def available_categories() -> list[CategoryInfo]:
"""List available tool categories and subcategories with tool counts."""
categories = tool_registry.get_categories()
return [
CategoryInfo(

6
openbb_platform/extensions/mcp_server/tests/utils/test_app_import.py
View File

@@ -183,6 +183,7 @@ app = FastAPI(title="Test App")
app_file.write_text(app_content)
return app_file
@pytest.mark.skipif(sys.platform == "win32", reason="Unix-specific test")
def test_unix_absolute_path(self, app_file: Path):
"""Test Unix-style absolute paths (e.g., /home/user/app.py)."""
app = import_app(str(app_file), "app", False)
@@ -282,14 +283,14 @@ custom_app = FastAPI(title="Custom Named App")
@pytest.mark.skipif(sys.platform != "win32", reason="Windows-specific test")
def test_windows_absolute_path(self, app_file: Path):
"""Test Windows-style absolute paths (e.g., C:\\Users\\app.py)."""
"""Test Windows-style absolute paths"""
# On Windows, tmp_path will have a drive letter (e.g., C:\...)
app = import_app(str(app_file), "app", False)
assert isinstance(app, FastAPI)
@pytest.mark.skipif(sys.platform != "win32", reason="Windows-specific test")
def test_windows_path_with_colon_notation(self, app_file: Path):
"""Test Windows path with colon notation (e.g., C:\\path\\app.py:myapp)."""
"""Test Windows path with colon notation."""
# Windows path with colon notation: C:\path\app.py:app
app = import_app(f"{app_file}:app", "app", False)
assert isinstance(app, FastAPI)
@@ -298,6 +299,7 @@ custom_app = FastAPI(title="Custom Named App")
class TestPathDetectionHelpers:
"""Test path detection logic for cross-platform compatibility."""
@pytest.mark.skipif(sys.platform == "win32", reason="Unix-specific test")
def test_is_absolute_path_detection_unix(self):
"""Test absolute path detection for Unix paths."""
# Unix absolute paths