fix: unit tests (#2760)
* fix:修复了main和plugin_manager部分单元测试 * fix: 修复了dashboard部分测试 * remove: 删除暂无用的配置测试脚本 * perf:拆分插件增查删改为独立的单元测试 * refactor: 重构插件管理器测试，使用临时环境隔离测试实例 * test: 增加对仪表板文件检查的单元测试，涵盖不同情况 * style: format code * remove: 删除未使用的导入语句 * delete: remove unused test file for pipeline --------- Co-authored-by: Soulter <905617992@qq.com>
This commit is contained in:
RC-CHN
GitHub
+105
-43
@@ -1,5 +1,6 @@
|
||||
import pytest
|
||||
import os
|
||||
from unittest.mock import MagicMock
|
||||
from astrbot.core.star.star_manager import PluginManager
|
||||
from astrbot.core.star.star_handler import star_handlers_registry
|
||||
from astrbot.core.star.star import star_registry
|
||||
@@ -8,18 +9,51 @@ from astrbot.core.config.astrbot_config import AstrBotConfig
|
||||
from astrbot.core.db.sqlite import SQLiteDatabase
|
||||
from asyncio import Queue
|
||||
|
||||
event_queue = Queue()
|
||||
|
||||
config = AstrBotConfig()
|
||||
|
||||
db = SQLiteDatabase("data/data_v3.db")
|
||||
|
||||
star_context = Context(event_queue, config, db)
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def plugin_manager_pm():
|
||||
return PluginManager(star_context, config)
|
||||
def plugin_manager_pm(tmp_path):
|
||||
"""
|
||||
Provides a fully isolated PluginManager instance for testing.
|
||||
- Uses a temporary directory for plugins.
|
||||
- Uses a temporary database.
|
||||
- Creates a fresh context for each test.
|
||||
"""
|
||||
# Create temporary resources
|
||||
temp_plugins_path = tmp_path / "plugins"
|
||||
temp_plugins_path.mkdir()
|
||||
temp_db_path = tmp_path / "test_db.db"
|
||||
|
||||
# Create fresh, isolated instances for the context
|
||||
event_queue = Queue()
|
||||
config = AstrBotConfig()
|
||||
db = SQLiteDatabase(str(temp_db_path))
|
||||
|
||||
# Set the plugin store path in the config to the temporary directory
|
||||
config.plugin_store_path = str(temp_plugins_path)
|
||||
|
||||
# Mock dependencies for the context
|
||||
provider_manager = MagicMock()
|
||||
platform_manager = MagicMock()
|
||||
conversation_manager = MagicMock()
|
||||
message_history_manager = MagicMock()
|
||||
persona_manager = MagicMock()
|
||||
astrbot_config_mgr = MagicMock()
|
||||
|
||||
star_context = Context(
|
||||
event_queue,
|
||||
config,
|
||||
db,
|
||||
provider_manager,
|
||||
platform_manager,
|
||||
conversation_manager,
|
||||
message_history_manager,
|
||||
persona_manager,
|
||||
astrbot_config_mgr,
|
||||
)
|
||||
|
||||
# Create the PluginManager instance
|
||||
manager = PluginManager(star_context, config)
|
||||
yield manager
|
||||
|
||||
|
||||
def test_plugin_manager_initialization(plugin_manager_pm: PluginManager):
|
||||
@@ -36,48 +70,76 @@ async def test_plugin_manager_reload(plugin_manager_pm: PluginManager):
|
||||
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_plugin_crud(plugin_manager_pm: PluginManager):
|
||||
"""测试插件安装和重载"""
|
||||
os.makedirs("data/plugins", exist_ok=True)
|
||||
async def test_install_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests successful plugin installation in an isolated environment."""
|
||||
test_repo = "https://github.com/Soulter/astrbot_plugin_essential"
|
||||
plugin_path = await plugin_manager_pm.install_plugin(test_repo)
|
||||
exists = False
|
||||
for md in star_registry:
|
||||
if md.name == "astrbot_plugin_essential":
|
||||
exists = True
|
||||
break
|
||||
assert plugin_path is not None
|
||||
plugin_info = await plugin_manager_pm.install_plugin(test_repo)
|
||||
plugin_path = os.path.join(
|
||||
plugin_manager_pm.plugin_store_path, "astrbot_plugin_essential"
|
||||
)
|
||||
|
||||
assert plugin_info is not None
|
||||
assert os.path.exists(plugin_path)
|
||||
assert exists is True, "插件 astrbot_plugin_essential 未成功载入"
|
||||
# shutil.rmtree(plugin_path)
|
||||
assert any(md.name == "astrbot_plugin_essential" for md in star_registry), (
|
||||
"Plugin 'astrbot_plugin_essential' was not loaded into star_registry."
|
||||
)
|
||||
|
||||
# install plugin which is not exists
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_install_nonexistent_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests that installing a non-existent plugin raises an exception."""
|
||||
with pytest.raises(Exception):
|
||||
plugin_path = await plugin_manager_pm.install_plugin(test_repo + "haha")
|
||||
await plugin_manager_pm.install_plugin(
|
||||
"https://github.com/Soulter/non_existent_repo"
|
||||
)
|
||||
|
||||
# update
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_update_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests updating an existing plugin in an isolated environment."""
|
||||
# First, install the plugin
|
||||
test_repo = "https://github.com/Soulter/astrbot_plugin_essential"
|
||||
await plugin_manager_pm.install_plugin(test_repo)
|
||||
|
||||
# Then, update it
|
||||
await plugin_manager_pm.update_plugin("astrbot_plugin_essential")
|
||||
|
||||
with pytest.raises(Exception):
|
||||
await plugin_manager_pm.update_plugin("astrbot_plugin_essentialhaha")
|
||||
|
||||
# uninstall
|
||||
@pytest.mark.asyncio
|
||||
async def test_update_nonexistent_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests that updating a non-existent plugin raises an exception."""
|
||||
with pytest.raises(Exception):
|
||||
await plugin_manager_pm.update_plugin("non_existent_plugin")
|
||||
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_uninstall_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests successful plugin uninstallation in an isolated environment."""
|
||||
# First, install the plugin
|
||||
test_repo = "https://github.com/Soulter/astrbot_plugin_essential"
|
||||
await plugin_manager_pm.install_plugin(test_repo)
|
||||
plugin_path = os.path.join(
|
||||
plugin_manager_pm.plugin_store_path, "astrbot_plugin_essential"
|
||||
)
|
||||
assert os.path.exists(plugin_path) # Pre-condition
|
||||
|
||||
# Then, uninstall it
|
||||
await plugin_manager_pm.uninstall_plugin("astrbot_plugin_essential")
|
||||
|
||||
assert not os.path.exists(plugin_path)
|
||||
exists = False
|
||||
for md in star_registry:
|
||||
if md.name == "astrbot_plugin_essential":
|
||||
exists = True
|
||||
break
|
||||
assert exists is False, "插件 astrbot_plugin_essential 未成功卸载"
|
||||
exists = False
|
||||
for md in star_handlers_registry:
|
||||
if "astrbot_plugin_essential" in md.handler_module_path:
|
||||
exists = True
|
||||
break
|
||||
assert exists is False, "插件 astrbot_plugin_essential 未成功卸载"
|
||||
assert not any(md.name == "astrbot_plugin_essential" for md in star_registry), (
|
||||
"Plugin 'astrbot_plugin_essential' was not unloaded from star_registry."
|
||||
)
|
||||
assert not any(
|
||||
"astrbot_plugin_essential" in md.handler_module_path
|
||||
for md in star_handlers_registry
|
||||
), (
|
||||
"Plugin 'astrbot_plugin_essential' handler was not unloaded from star_handlers_registry."
|
||||
)
|
||||
|
||||
|
||||
@pytest.mark.asyncio
|
||||
async def test_uninstall_nonexistent_plugin(plugin_manager_pm: PluginManager):
|
||||
"""Tests that uninstalling a non-existent plugin raises an exception."""
|
||||
with pytest.raises(Exception):
|
||||
await plugin_manager_pm.uninstall_plugin("astrbot_plugin_essentialhaha")
|
||||
|
||||
# TODO: file installation
|
||||
await plugin_manager_pm.uninstall_plugin("non_existent_plugin")
|
||||
|
||||
Reference in New Issue
Block a user