mirror of
https://github.com/AstrBotDevs/AstrBot
synced 2026-07-15 17:30:13 +08:00
* refactor: migrate to fastapi * structure refactor * fix: pyright fix * refactor: improve error handling and public messages in plugin services * feat(api): refactor API client integration and enhance request handling - Updated API client configuration to use a dedicated HTTP client. - Introduced utility functions for generating options, queries, and form data for API requests. - Refactored multiple API methods to utilize the new utility functions for improved consistency and readability. - Renamed types for clarity and updated import statements accordingly. feat(docs): add script to update OpenAPI JSON from YAML spec - Created a Python script to convert OpenAPI YAML specification to JSON format. - The script supports customizable input and output paths. - Ensured the script handles directory creation for output paths and validates the YAML structure. * fix * feat(auth): implement rate limiting for v1 login endpoint and enhance request handling * Refactor dashboard API routers to use legacy_router for backward compatibility - Changed all instances of dashboard_router to legacy_router across multiple API modules including platform, plugins, providers, sessions, skills, stats, subagents, t2i, tools, updates, and asgi_runtime. - Updated route definitions to ensure existing endpoints remain functional under the new router structure. - Introduced support for Quart request context in asgi_runtime to enhance compatibility with existing Quart-based plugins. - Added a test case to validate the functionality of the new Quart request context handling in plugin extensions. * chore: remove cli test * fix: update dashboard tests for fastapi migration * chore: satisfy ruff checks * fix: update openapi api key scopes * fix: sync config scope chip selection * fix: restore quart dependency * docs: clarify quart plugin api compatibility * docs: update openapi scope documentation * fix: use singular skill openapi scope * fix: hide update service exception details * fix: address fastapi review comments * fix: address dashboard review findings * docs: revert unrelated package deployment changes * docs: update agent api generation guidance * feat: add plugin page web api helpers * docs: add plugin page bridge demo * fix: type plugin upload files * fix: stabilize plugin page uploads * fix: type plugin web request proxy * docs: remove plugin page docs example * fix: authenticate plugin page SSE bridge
170 lines
5.6 KiB
Python
170 lines
5.6 KiB
Python
from types import SimpleNamespace
|
|
from unittest.mock import AsyncMock, MagicMock
|
|
|
|
import pytest
|
|
|
|
from astrbot.api.message_components import Plain
|
|
from astrbot.builtin_stars.astrbot.main import Main
|
|
|
|
|
|
def make_main_with_conversation_manager(conv_mgr):
|
|
main = Main.__new__(Main)
|
|
main.context = MagicMock()
|
|
main.context.conversation_manager = conv_mgr
|
|
return main
|
|
|
|
|
|
def _make_extras_store():
|
|
"""Return a mutable dict and get_extra / set_extra side_effects bound to it."""
|
|
store: dict[str, object] = {}
|
|
get_extra = lambda key, default=None: store.get(key, default) # noqa: E731
|
|
set_extra = store.__setitem__ # type: ignore[assignment]
|
|
return store, get_extra, set_extra
|
|
|
|
|
|
def make_event(
|
|
umo: str = "aiocqhttp:GroupMessage:user_123_group_456",
|
|
*,
|
|
handlers_parsed_params: dict | None = None,
|
|
):
|
|
event = MagicMock()
|
|
event.unified_msg_origin = umo
|
|
event.get_platform_id.return_value = "aiocqhttp"
|
|
event.message_obj = SimpleNamespace(message=[Plain("hello")])
|
|
event.message_str = "hello"
|
|
event.session_id = "session-1"
|
|
|
|
store, get_extra, set_extra = _make_extras_store()
|
|
# Simulate WakingCheckStage output: an empty dict means no command matched.
|
|
store["handlers_parsed_params"] = (
|
|
{} if handlers_parsed_params is None else handlers_parsed_params
|
|
)
|
|
event.get_extra.side_effect = get_extra
|
|
event.set_extra.side_effect = set_extra
|
|
return event
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_active_reply_does_not_create_conversation_when_current_missing():
|
|
conv_mgr = SimpleNamespace(
|
|
get_curr_conversation_id=AsyncMock(return_value=None),
|
|
new_conversation=AsyncMock(),
|
|
get_conversation=AsyncMock(),
|
|
)
|
|
main = make_main_with_conversation_manager(conv_mgr)
|
|
main.context.get_config.return_value = {
|
|
"provider_ltm_settings": {
|
|
"group_icl_enable": False,
|
|
"active_reply": {"enable": True},
|
|
},
|
|
}
|
|
main.context.get_using_provider.return_value = object()
|
|
main.group_chat_context = SimpleNamespace(
|
|
need_active_reply=AsyncMock(return_value=True),
|
|
handle_message=AsyncMock(),
|
|
)
|
|
event = make_event()
|
|
|
|
results = [item async for item in main.on_message(event)]
|
|
|
|
assert results == []
|
|
conv_mgr.get_curr_conversation_id.assert_awaited_once_with(event.unified_msg_origin)
|
|
conv_mgr.new_conversation.assert_not_called()
|
|
conv_mgr.get_conversation.assert_not_called()
|
|
event.request_llm.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_active_reply_reuses_current_umo_conversation():
|
|
conv = SimpleNamespace(cid="cid-1")
|
|
conv_mgr = SimpleNamespace(
|
|
get_curr_conversation_id=AsyncMock(return_value="cid-1"),
|
|
new_conversation=AsyncMock(),
|
|
get_conversation=AsyncMock(return_value=conv),
|
|
)
|
|
main = make_main_with_conversation_manager(conv_mgr)
|
|
main.context.get_config.return_value = {
|
|
"provider_ltm_settings": {
|
|
"group_icl_enable": False,
|
|
"active_reply": {"enable": True},
|
|
},
|
|
}
|
|
main.context.get_using_provider.return_value = object()
|
|
main.group_chat_context = SimpleNamespace(
|
|
need_active_reply=AsyncMock(return_value=True),
|
|
handle_message=AsyncMock(),
|
|
)
|
|
event = make_event("aiocqhttp:GroupMessage:user_999_group_456")
|
|
llm_request = object()
|
|
event.request_llm.return_value = llm_request
|
|
|
|
results = [item async for item in main.on_message(event)]
|
|
|
|
assert results == [llm_request]
|
|
conv_mgr.get_curr_conversation_id.assert_awaited_once_with(event.unified_msg_origin)
|
|
conv_mgr.new_conversation.assert_not_called()
|
|
conv_mgr.get_conversation.assert_awaited_once_with(
|
|
event.unified_msg_origin,
|
|
"cid-1",
|
|
)
|
|
event.request_llm.assert_called_once_with(
|
|
prompt="hello",
|
|
session_id="session-1",
|
|
image_urls=[],
|
|
conversation=conv,
|
|
)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_on_message_does_not_clear_group_context_on_first_enabled_message():
|
|
main = Main.__new__(Main)
|
|
main.context = MagicMock()
|
|
main.context.get_config.return_value = {
|
|
"provider_ltm_settings": {
|
|
"group_icl_enable": True,
|
|
"active_reply": {"enable": False},
|
|
},
|
|
}
|
|
main.group_chat_context = SimpleNamespace(
|
|
need_active_reply=AsyncMock(return_value=False),
|
|
handle_message=AsyncMock(),
|
|
remove_session=AsyncMock(),
|
|
)
|
|
event = make_event()
|
|
|
|
async for _ in main.on_message(event):
|
|
pass
|
|
|
|
main.group_chat_context.need_active_reply.assert_awaited_once_with(event)
|
|
main.group_chat_context.handle_message.assert_awaited_once_with(event)
|
|
main.group_chat_context.remove_session.assert_not_called()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_on_message_skips_recording_when_command_handler_matched():
|
|
"""A slash-command message (handlers_parsed_params non-empty) must not be
|
|
recorded into the group context buffer."""
|
|
main = Main.__new__(Main)
|
|
main.context = MagicMock()
|
|
main.context.get_config.return_value = {
|
|
"provider_ltm_settings": {
|
|
"group_icl_enable": True,
|
|
"active_reply": {"enable": False},
|
|
},
|
|
}
|
|
main.group_chat_context = SimpleNamespace(
|
|
need_active_reply=AsyncMock(return_value=False),
|
|
handle_message=AsyncMock(),
|
|
)
|
|
event = make_event(
|
|
handlers_parsed_params={
|
|
"astrbot.builtin_stars.builtin_commands.main_reset": {}
|
|
},
|
|
)
|
|
|
|
async for _ in main.on_message(event):
|
|
pass
|
|
|
|
main.group_chat_context.need_active_reply.assert_awaited_once_with(event)
|
|
main.group_chat_context.handle_message.assert_not_awaited()
|